Notepad++ v8.7.2 Release Candidate

Alan Kilborn

Add Pin tab feature.

I don’t see the Pin Tab menu option anywhere

Settings > Preferences > General > ☐ Enable pin tab feature is not checkmarked by default; you have to checkmark the option to enable that feature.

@donho Is there a reason this isn’t checkmarked by default? It is a good feature and if it isn’t on by default, 99% of Notepad++ won’t know it exists, even years from now – you’ve got to advertise your features. Users can always come here to ask how to turn it off, and some small percentage will want to do this (it’s a guarantee)…people hate change, even good changes.

Lycan Thrope

@Snabel42 ,
I can’t attest to what it looks like in a standard install, as I tend to use portables only for RC’s, since I don’t want my main app being ‘tested’. My bad, maybe, but…in the portable, as soon as you select the Preferences option to show it, it works quite well. I didn’t understand what the pin tab issue was about, but after playing with it, I see it’s not so much about keeping a document open when shutting down, it’s about not allowing the tabs to move it out of place from the left. I figured that out when I tried changing their position manually, and it wouldn’t move. I see the value in it now, if you move tabs a lot and want the left most kept where it is…but…it’s not my use case so I’m little affected by it, but I see the pin tab after enabling it.
Perhaps one of your plugins could be giving it a problem? My portable is a standard plugin setup, no extras. Your list is extensive.

norrisgc

@Alan-Kilborn said in Notepad++ v8.7.2 Release Candidate:

@donho said:

Add Pin tab feature.
@donho Is there a reason this isn’t checkmarked by default? It is a good feature and if it isn’t on by default, 99% of Notepad++ won’t know it exists, even years from now – you’ve got to advertise your features. Users can always come here to ask how to turn it off, and some small percentage will want to do this (it’s a guarantee)…people hate change, even good changes.

You might see this as a good change, but there is no good reason to force on an option for existing users who then waste time trying to figure out either what it does or how to turn it back off again. A text editor is not supposed to be a game of adventure! It should just work, and not change how it behaves simply due to what someone else sees as an improvement. I’m perfectly capable of reading release notes, and I’m pretty sure other people are too, so if I want to use something new, I can choose to do so.

Alan Kilborn

A bug exists in the 8.7.2 RC relating to the feature enhancement:

Add ability to open/copy selected files from Search-results.

I authored (well, mostly) this enhancement, and while the code was “fresh” it was pointed out that it had a bug. Rather than giving me time to consider and fix that bug, a quick fix was introduced (by someone else) that caused another bug, and that went unnoticed and made it into the 8.7.2 RC. I didn’t notice it sooner because I felt my efforts were hijacked, and I basically lost interest.

Bug summary:

Result once clipboard contents are pasted somewhere:

new 1
new 1
new 1
new 2
new 2
new 2
new 3

Instead of the correct:

new 1
new 2
new 3

I have suggested a fix HERE.

donho

@Alan-Kilborn said in Notepad++ v8.7.2 Release Candidate:

Is there a reason this isn’t checkmarked by default? It is a good feature and if it isn’t on by default, 99% of Notepad++ won’t know it exists, even years from now – you’ve got to advertise your features. Users can always come here to ask how to turn it off, and some small percentage will want to do this (it’s a guarantee)…people hate change, even good changes.

Makes sense. It’ll be shown for people update from the old version:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/09332d6f98fc46b4139ff4a7a091c50c1c684d81

donho

@Alan-Kilborn said in Notepad++ v8.7.2 Release Candidate:

I authored (well, mostly) this enhancement, and while the code was “fresh” it was pointed out that it had a bug. Rather than giving me time to consider and fix that bug, a quick fix was introduced (by someone else) that caused another bug, and that went unnoticed and made it into the 8.7.2 RC. I didn’t notice it sooner because I felt my efforts were hijacked, and I basically lost interest.

The fix is committed in master:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/7af3c104d51f889ced48bc9f51e48bc0ee1876df

I apology for not waiting for you to fix it. The reason is there was a solution immediately and the algorithm of the solution uses the same mechanism of the original code.
Finally, it was me who have introduced this bug by the commit af2b90f.

Nevertheless, thank you for spotting the bug & thank you for fixing it!

donho

FYI:
Notepad++ v8.7.2 Release Candidate 2, in which a bug-fix of “Copy selected PathName(s)” command and a new feature (Promote Pin Tab feature) are added, is updated to:
http://download.notepad-plus-plus.org/repository/8.x/8.7.2.RC2/

mkupper

@Alan-Kilborn said in Notepad++ v8.7.2 Release Candidate:

I think the idea is, get it released initially, and then improve upon it as needed in future releases.
I will be creating some “issues” relating to these possible improvements, once 8.7.2 is released.

Where will you be collecting comments about the pinned tab feature?

–

moderator moved most of the post to here; see @PeterJones reply for more

PeterJones

@mkupper and everyone:

Top keep the release topic focused on regressions/bugs , let’s move the “Pinned Tabs” discussion – especially things about how you feel about the current implementation, and where that implementation might go in the future – in this new Topic that I spawned.

(I copied the bulk of @mkupper’s contents into that new discussion, and edited out the “conversation starter” portion of his reply from this Topic)

Coises

@donho said in Notepad++ v8.7.2 Release Candidate:

Notepad++ v8.7.2 Release Candidate:
http://download.notepad-plus-plus.org/repository/8.x/8.7.2.RC2/

Change log:

Add Pin tab feature.

Very minor detail, but if Enable pin tab feature is checked and you toggle the Reduce option for the Tab Bar (whether checking or unchecking), tabs lose the space that’s supposed to be reserved for the pin, and the pin overlaps the file name. Toggling Enable pin tab feature off and then on again fixes this.

donho

@Coises said in Notepad++ v8.7.2 Release Candidate:

Very minor detail, but if Enable pin tab feature is checked and you toggle the Reduce option for the Tab Bar (whether checking or unchecking), tabs lose the space that’s supposed to be reserved for the pin, and the pin overlaps the file name. Toggling Enable pin tab feature off and then on again fixes this.

Thank you for spotting this bug. It’s fixed and the bug-fix has been merged into master:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/ae5675247220ebe35fbe2240533a33bf0ef0f1bb

Also the RC3 is updated here:
http://download.notepad-plus-plus.org/repository/8.x/8.7.2.RC3/

schnurlos

Small comment to the change.log:
Shouldn’t it show “Notepad++ v8.7.2 new features & bug-fixes:”?
I’m missing the “s” at feature (features & fixes)?