Feedly-Notifier-Firefox icon indicating copy to clipboard operation
Feedly-Notifier-Firefox copied to clipboard

Fix middle mouse on links in the feed popup

Open ThiefMaster opened this issue 9 years ago • 4 comments

When viewing a feed in the popup, middle-clicking a link in the content of a feed's entry doesn't do anything. It should open the link in a new (background) tab though.

Also I noticed some cases where left-clicking a link caused the page to load inside the popup, breaking it and requiring me to disable and re-enable the whole extension (or restart firefox).

ThiefMaster avatar Sep 10 '14 10:09 ThiefMaster

Also I noticed some cases where left-clicking a link caused the page to load inside the popup, breaking it and requiring me to disable and re-enable the whole extension (or restart firefox).

Could you please provide the feeds for testing?

olsh avatar Feb 27 '15 00:02 olsh

It happens for me with this feed: http://blog.fefe.de/rss.xml?html

ThiefMaster avatar Mar 07 '15 09:03 ThiefMaster

When viewing a feed in the popup, middle-clicking a link in the content of a feed's entry doesn't do anything. It should open the link in a new (background) tab though.

For now all links are opened in a new tab with left mouse click. You don't need to use the middle mouse button.

It happens for me with this feed: http://blog.fefe.de/rss.xml?html

Unable to reproduce the problem, all link works fine for me. Could you please specify an exact entry on which the bug can be reproduced?

olsh avatar Apr 13 '15 22:04 olsh

Happened on all entries.

Could it be related to this GreaseMonkey script I'm running?

// ==UserScript==
// @name           _blank Must Die
// @namespace      http://maxkueng.com/gmscripts
// @description    Removes the target attribute from hyperlinks that open a new browser window
// @version        1.3
// @author         Max Kueng
// @homepage       http://maxkueng.com/
// @include        *
// ==/UserScript==

(function (){
    var a = window.document.getElementsByTagName('a');
    var regTarget = /^(_blank|blank|_new|new|_neu|neu|_newwin|newwin)$/i;
    for (var i = 0; i < a.length; i++) {
        if (a[i].target.match(regTarget)) {
            //a[i].removeAttribute("target");
            a[i].target = '_top';
        }
    }
}());

If that's the case, maybe you can somehow prevent user scripts from running on your extension's panel?

ThiefMaster avatar Apr 14 '15 05:04 ThiefMaster