ferdium-recipes icon indicating copy to clipboard operation
ferdium-recipes copied to clipboard

franz-custom-website: fix click events when trapLinkClicks is enabled

Open s-ol opened this issue 11 months ago • 1 comments

Pre-flight Checklist

Please ensure you've completed all of the following.

Description of Change

Fully deactivates messing with click events when "Open external URLs in Ferdium" is enabled. Previously Ferdium would eat all click events on <a href="http://... and <button title="http://... tags, breaking sites that override click events to provide in-page navigation.

Close ferdium/ferdium-app#2028

s-ol avatar Jan 26 '25 13:01 s-ol

Hi @s-ol, the use case you are mentioning should not warrant for a complete suppression of this feature, however we would be happy to see the code improved. The problem described in the issue linked is about the question of whether the event should be stopped from propagating or not. Indeed, on some websites if this event is not stopped when trapLink is on, then the webpage will open in the browser anyway, which goes against what the goal of trapLink was in the first place. So if there is a way to control than when this were to happen we block the event from happening, and we let it be if it is instead helping to move in the page, then that would be a perfect change. At the current status, we do not want to merge such a change that simply disable it entirely.

Alphrag avatar Apr 18 '25 19:04 Alphrag