Middle-clicking links from reading list doesn't work anymore

Issue #30 resolved
krisu created an issue

Can't open links to new tabs, it was possible before the update.

EDIT: I noticed that there was "open in new tab" option, still I kinda refer my method :)

Comments (5)

  1. Pierre-Adrien Buisson repo owner

    Indeed, I reproduce it on my machine. I'll be checking this out tonight. Thanks for noticing.

    I really need to add an integration test suite to the addon codebase.

  2. JJ

    It’s been a while, should I take a look at this?

    I'd use a element instead of click handler to make it Chromium compatible (chromium doesn’t trigger click for mouse scroll)

    EDIT: okay, I found an issue in your code, but as I mentioned earlier instead of fixing your attempt I’d prefer to rework it to make it Chromium compatible, though first I’ll submit just the quick fix

    EDIT2: mousedown/up events work in Chromium properly, that was probably your reasoning, so switching to a isn’t required

  3. Log in to comment