Notepad3 icon indicating copy to clipboard operation
Notepad3 copied to clipboard

Default Markdown Header highlight interferes with URLs on the header

Open stdedos opened this issue 2 years ago • 3 comments

image

I assume I am able to fix it myself somehow; I am only wondering if that would be interesting enough to be fixed globally

stdedos avatar Mar 13 '22 07:03 stdedos

Somewhat related: enclosing URLs in angle brackets (i.e., "Lesser than" and "Greater than" character/entities: "<" and ">") is Markdown shorthand for creating active links.

While this works in Notepad3, when clicking the URL, the end bracket (i.e., "Greater than") gets appended to the invoked URL and thus makes it fail to open in the associated browser.

This never happens with the heading "Lesser than" character. To solve this, Notepad3 should exclude any such non-encoded characters at URL end (e.g. anything that isn't &gt; or &#62;) from URL activation.

midaspt avatar Sep 13 '22 13:09 midaspt

No, it's not related - I am saying that the default theme color effectively hides if you'd happen to have a URL on the header line

stdedos avatar Sep 15 '22 09:09 stdedos

While this works in Notepad3, when clicking the URL, the end bracket (i.e., "Greater than") gets appended to the invoked URL and thus makes it fail to open in the associated browser.

Hello @midaspt , One of the latest beta versions has fixed the issue with the end bracket ">" in the URL. (see test file test.md)

2022-09-15_140307


Feel free to test the "BETA/RC PortableApps", version "Notepad3Portable_5.22.902.1_beta.paf" or newer, see 1st list in issue #1129.

"Notepad3Portable BETA/RC PortableApps" version can be used with or without ".7z" extension.

Also, feel free to test the "BETA/RC Setup", version "Notepad3_5.22.902.1_beta_Setup" or newer, see the 2nd list in issue #1129.

Comments and suggestions are welcome... 😃

hpwamr avatar Sep 15 '22 12:09 hpwamr

(Finally...!) My issue confirmed solved in both Notepad3 'v5.22.902.1_beta.paf' linked above and 'v5.22.1119.1_rc.paf'.

midaspt avatar Nov 21 '22 15:11 midaspt

I'm sorry @hpwamr; you didn't solve my issue. Or, I don't see how/where did you solve it.

Idk why @midaspt decided to high-jack my issue for his (otherwise valid) markdown syntax parsing issue. This issue is specifically about "being able to read the URL text inside a Markdown header".

stdedos avatar Dec 26 '22 10:12 stdedos

This issue is specifically about "being able to read the URL text inside a Markdown header".

Hello @stdedos ,

Feel free to test the "BETA/RC PortableApps", version "Notepad3Portable_5.22.1230.1_rc2.paf" or newer, see in issue #1129.

"Notepad3Portable BETA/RC PortableApps" version can be used with or without ".7z" extension.

Also, feel free to test the "BETA/RC Setup", version "Notepad3_5.22.1230.1_rc2_Setup" or newer, see the 2nd list in issue #1129.

Comments and suggestions are welcome... 😃

hpwamr avatar Dec 30 '22 17:12 hpwamr

Great, thanks @hpwamr! I'll have to do it in 3w though 😕 (no access to Windows)

stdedos avatar Dec 30 '22 17:12 stdedos