Cimbali / CleanLinks

Converts obfuscated/nested links to genuine clean links.
https://addons.mozilla.org/en-GB/firefox/addon/clean-links-webext/
Mozilla Public License 2.0
76 stars 2 forks source link

javascript:; links are broken #111

Closed birdie-github closed 4 years ago

birdie-github commented 4 years ago

What I see in the log:

null; 
-> javascript:;
Cimbali commented 4 years ago

Can you give an example of where this happens? We should be avoiding most of those links already so I need the exact value of the href to know why it doesn't work. Also which version are you using?

birdie-github commented 4 years ago

Looks like you've already fixed this in version 4.1.5. Thank you!

birdie-github commented 4 years ago

Found another instance where such links get broken and there's nothing in the log:

https://bugzilla.kernel.org/show_bug.cgi?id=205227

When I click "reply" nothing happens.

Cimbali commented 4 years ago

Alright so the bug here is really that it doesn’t appear in the log:

So CleanLinks prevents the modification on that link, as it should. You can add a rule to allow javascript modifying links, either just for bugzilla.kernel.org or for *.*

birdie-github commented 4 years ago

You can add a rule to allow JavaScript modifying links

Sorry, I've no idea how to do that.

Cimbali commented 4 years ago

From the settings page,

birdie-github commented 4 years ago

To be honest I frequent at least a dozen of websites which use such links and I'm not a fan of this solution :-( It would be great to have an icon in the CL popup window which allowed to do that automatically.

Cimbali commented 4 years ago

Yes, that’s on its way! See https://github.com/Cimbali/CleanLinks/issues/104#issuecomment-612649837