Notepad++ v7.9.2 Released

Alan Kilborn

I’m noticing the following “weirdness” with the new-in-7.9.2 URI customized schemes :

Note that the built-in file protocol has no problem underlining paths with either types of slashing, / or \.

But with the customized foo protocol, it doesn’t want to underline paths with the most-typical “slashing”, that of Windows’ \.
Curiously, it has no trouble with the underlining, as long as the first slash after the colon of the drive letter is a forward-slash (and subsequent slashes are of the backslash variety).

guy038

Hello, @don-ho, @alan-kilborn and All,

@Alan-kilborn :

I’m quite late in that discussion but I don’t really understand your problem with the new Copy link, in the Context menu !

On a fresh portable v7.9.2 version with the default Context menu, without any customization :

If you previously moved the caret at any position of a link, even right before the first character ( generally the letter h of https?), but not right after the last character, and without any selection, you correctly see the entry Copy link, instead of Copy, in the Context menu
If you previously moved the caret at any position of a link, even right before the first character and that you selected some characters, the normal Copy entry is displayed and functional, in the Context menu
If you previously moved the caret at any position outside a link, without any selection the normal Copy entry is simply greyed in the Context menu. Logical !
If you previously moved the caret at any position outside a link, and did a selection of some characters, the normal Copy entry is displayed and functional, in the Context menu.

So, the N++ behaviour, in these four cases, seems quite coherent !?

Cheers,

guy038

Ekopalypse

@guy038
My understanding is that the fresh portable is not the problem, Alan has the same results with it, but the daily used one is.

guy038

Hi, @don-ho, @alan-kilborn, @ekopalypse and All,

BTW, Alan, is the thick red vertical bar, in your first screenshot, showing the contextMenu.xml file, your current caret ?

If so, it’s logical that the copy menu is displayed in the contextual menu !

BR

guy038

Alan Kilborn

@guy038 said in Notepad++ v7.9.2 Released:

BTW, Alan, is the thick red vertical bar, in your first screenshot, showing the contextMenu.xml file, your current caret ?

Guy to the rescue!
Thanks to that, I now know what is going on!
(Yes, the “thick red vertical bar” is indeed my caret.)

I have this setting enabled:

The code that changes the Copy menu item into Copy Link must be looking at the current caret position instead of the position corresponding to the right-click (x,y) point, in order to determine what it should do.

guy038

Hello, @don-ho, @linpengcheng, @ufcoxd and All,

Don, in this post, you spoke about creating this issue, saying :

An invalid xml could make disfunction in Notepad++ as problem in #9329 .

To avoid such problem, it’s necessary to check the whole set of XML files in Notepad++'s AppVeyor settings to ensure the validity of these files.

Of course, I have no doubt that there are numerous tools to evaluate the relevance of XML files ;-))

Now, just for fun, here is a basic XML validation method that can be processed from within Notepad++, only, which could interest some people !

Start Notepad++
Open the faulty chineseSimplified configuration file, of the v7.9.2 release !
Open the Mark dialog ( Ctrl + M )
- MARK (?x) </? [\w-]+? (?= (?: \x20 (?-s).+? )? (?<![/-]) > )
- Tick the Wrap around option
- Select the Regular expression search mode
- Click on the Mark All button
- Click on the new Copy Marked Text button ( Many thanks @scott-sumner !! )
- Close the Mark dialog ( ESC )
Open a new tab ( Ctrl + N )
Paste all the marked text in this new file ( Ctrl + V )

So, a list of 142 start or end tags !

Now, open the Replace dialog ( Ctrl + H )
- SEARCH (?x-s) ^ < (.+\R) (?s) (.*?) </ \1
- REPLACE \2
- Tick the Wrap around option
- Select the Regular expression search mode
- Click several times, consecutively, on the Replace All button, until you see the message Replace All: 0 occurrence were replaced ( successively 1, 12, 24, 32, 1 and 0 occurrence replaced )

=> And you should get these two lines only :

<MarginsBorderEdge
</Scintillas

After a brief examination of the chineseSimplified file, it clearly appears that the end tag should be rewritten as </MarginsBorderEdge> ( and not </Scintillas> ) !

Best Regards,

guy038

donho

@guy038 said in Notepad++ v7.9.2 Released:

Now, just for fun, here is a basic XML validation method that can be processed from within Notepad++, only, which could interest some people !

Brilliant!

However @chcg has integrated a python script for XML files’ validation in the build system:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/d17fa02a6f40f4650e8e28e94e42078f13ce59cc

The script is efficient - my 2 commits have been rejected due to the invalid XML files. So there’ll be no more such regression. :)

Artur Harison

@donho
Bug with field of Search Result #9341 which I already wrote in CR theme.
Will this be fixed? It’s important for me.

Alan Kilborn

No comments at all on my “new-in-7.9.2 URI customized schemes” issue from above?
It is really getting in the way of something I’d like to do with this new feature.

donho

@Alan-Kilborn said in Notepad++ v7.9.2 Released:

I’m noticing the following “weirdness” with the new-in-7.9.2 URI customized schemes

There’s no way to satisfy all the custom URI’s rule, so it has to be generic. Currently custom URI follows URL’s pattern. What you see the exception of your foo:// is not accepted in URL scheme as well. Feel free to create an issue on Github Notepad++ if you think it’s a bug (for URL).

donho

@Artur-Harison said in Notepad++ v7.9.2 Released:

Bug with field of Search Result #9341 which I already wrote in CR theme.
Will this be fixed?

I’ll take a look on it.

Alan Kilborn

@donho said in Notepad++ v7.9.2 Released:

There’s no way to satisfy all the custom URI’s rule, so it has to be generic. Currently custom URI follows URL’s pattern. What you see the exception of your foo:// is not accepted in URL scheme as well. Feel free to create an issue on Github Notepad++ if you think it’s a bug (for URL).

I’m totally unsure what any of that means.
Can someone “translate” it for me? :-)
Meanwhile I will guess…

It would be my hope and expectation that if Notepad++ considers something like file://w:\testing\test as a link and underlines it (and opens the file in the associated application and the file when double-clicked – AND IT DOES!) it would also do it – the underlining, not the opening – for the same argument for a custom “protocol” such as my foo://w:\testing\test.

I can create an issue, but don’t want to waste effort if this is an unrealistic expectation. Thinking about it in light of what @donho said, maybe an issue I open should be a “counter-issue”, saying that Notepad++ should NOT underline/open file://w:\testing\test because backslashes in this are “not correct”?

I can still do what I want with the custom protocols, but (ugh!) it means I would have to convert every Windows path I would ever use from:

c:\myfolder\mysubfolder\myfile.txt

to

c:/myfolder/mysubfolder/myfile.txt

in order to get it ( foo://c:/myfolder/mysubfolder/myfile.txt ) to show in Notepad++ as a link.
I can make a macro to do that, but, again, ugh!, it means selecting a pathname and running the macro each time I need a pathname.

Side note: Chrome has no issue if I put file://c:\myfolder\mysubfolder\myfile.txt into its address box and press Enter.

But in the end, technically I don’t know if backslashes here are “spec” or not; I didn’t look it up, I just know which way I want it to work. :-)

ArkadiuszMichalski

@Alan-Kilborn
file:// is processed slightly differently than http(s) and customs schemes (with follow http rules) so that your example not works. This is how it was written by the author of the new URL parser (Uhf7). You can open an bug, if the changes are not big, it may take them into account (plus add new tests), who knows.

Bassically on Windows for all urls/paths we can use \ or / by mixing them and it works (browsers also correct such url to correct format - try load such adress and check how it looks like).

Alan Kilborn

@ArkadiuszMichalski said in Notepad++ v7.9.2 Released:

Bassically on Windows for all urls/paths we can use \ or / by mixing them and it works (browsers also correct such url to correct format - try load such adress and check how it looks like).

Sure. That’s when we are doing the sourcing. When something else is doing the sourcing (e.g. Notepad++), we typically always get \ – think of right-click-tab’s Full File Path to Clipboard.

For what I want to in this circumstance, it is a (minor) stopper. I’ll be elaborating on exactly what I’m doing soon in another thread. Key to what I’m doing is Notepad++ underlining-as-link the “right” thing for me, scripting can take it from there. I don’t want to have to reinvent the wheel and do my own “underlining”.

Michael Vincent

@Alan-Kilborn said in Notepad++ v7.9.2 Released:

When something else is doing the sourcing (e.g. Notepad++), we typically always get \ – think of right-click-tab’s Full File Path to Clipboard.

I think the file:// URI typically has another / to start with to indicate the root directory, fully qualified path. So from your examples, the actual URI should be:

file:///w:\testing\test.txt
file:///w:/testing/test.txt
...

Typically, we “ignore” the leading / on Windows since the driver letter C: in this example acts like the root directory in linux operating systems. But, if we consider the “extra” / mandatory, then the foo:// example works for me:

See the first one doesn’t work using just the foo:// leader, but adding the extra / in the second example to indicate a file-like URI, it works. I’ve only added foo:// (with only 2 /'s to my URI custom schemes:

Cheers.

Alan Kilborn

@Michael-Vincent said in Notepad++ v7.9.2 Released:

See the first one doesn’t work using just the foo:// leader, but adding the extra / in the second example to indicate a file-like URI, it works.

Ah, Michael to the rescue now. Thanks!
Typing that third / is way less of a burden than converting all of the \ in the following path. (Note: it was truly only necessary to convert the one immediately after the colon of the drive letter, but that mixing of slashes was just too wacky to propagate through a bunch of data files)

I’ve only added foo:// (with only 2 /'s to my URI custom schemes

Which raises another point: in the box in the Preferences, is it correct to do foo:// or foo: alone? Maybe I don’t need the // but I am so used to seeing it as part of file:// or http:// with Notepad++ links that I just assumed… I suppose it will become more apparent as the experimentation goes on.

If I’d have tried foo: without the // from the beginning, I wouldn’t have “whined”, because foo:w:\blah\blah.txt highlights as a link just fine! Drat!

Michael Vincent

@Alan-Kilborn said in Notepad++ v7.9.2 Released:

I wouldn’t have “whined”, because foo:w:\blah\blah.txt highlights as a link just fine! Drat!

Interesting. There are some URI’s that don’t use the // - I’m thinking mailto:user@domain.com so maybe foo: alone is fine for your use case. I think I pulled my extensive list from an issue or maybe the Notepad++ source code - I certainly didn’t type it all myself - that was cut and paste - from where, I can’t remember.

Cheers.

Alan Kilborn

@Michael-Vincent said in Notepad++ v7.9.2 Released:

I think I pulled my extensive list … I certainly didn’t type it all myself

Are you talking about then contents of the URI customized schemes you showed? :

That list?
If so, then no, you didn’t do it: With the exception of the foo:// at the end, that is all default text content for that box in 7.9.2 !

Michael Vincent

@Alan-Kilborn said in Notepad++ v7.9.2 Released:

that is all default text content for that box in 7.9.2 !

Oh… :-) that makes sense.

Ekopalypse

See https://community.notepad-plus-plus.org/topic/20563/notepad-v-7-9-2-ignores-print-line-number-preference

Does not happen with 7.9.1