Links aus Verlauf in neuem Tab öffnen/Tablock-Skript

Speravir

Vielleicht kann man das sogar als Bug oder fehlende Funktionalität in Firefox ansehen:

Ich lasse mir Lesezeichen in einem neuen Tab öffnen (Browsereinstellung browser.tabs.loadBookmarksInTabs). Das funktioniert dann auch so aus der Seitenleiste. Will ich aber eine Seite aus der Historie aufrufen, ebenfalls aus der Seitenleiste heraus, wird diese mit Linksklick immer im aktuellen Tab geöffnet, was besonders ärgerlich ist, wenn man kurz vorher noch in der Lesezeichenleiste war und sich dort mit Linksklick neue Tabs öffneten. Grundsätzlich ist es aber sehr unschön, jedes Mal erst überlegen zu müssen, welche Maustaste man drücken muss.

Leider hilft das Tablock-Skript dabei überhaupt nicht, die Seiten werden trotzdem im aktuellen Tab geöffnet. Kann man an diesem Verhalten etwas im Skript ändern?

aborix

Mit folgendem Skript werden auch Chronik-Einträge aus der Sidebar in neuen Tabs geöffnet. Das Tablock-Skript bleibt unverändert.

Code

function() {
  if (location != 'chrome://browser/content/browser.xul')
    return;
  eval('PlacesUIUtils.openNodeWithEvent = ' + PlacesUIUtils.openNodeWithEvent.toString()
    .replace('PlacesUtils.nodeIsBookmark(aNode) && ', ''));
})();

Das Skript hat folgende Nebenwirkungen:
Auch Chronik-Einträge aus der Bibliothek werden in neuen Tabs geöffnet. Bei Einträgen im Chronik-Menü hingegen erfolgt keine Änderung.
Steht browser.tabs.loadBookmarksInBackground auf true, werden nicht nur alle Lesezeichen, sondern auch die Chronik-Einträge aus der Sidebar und der Bibliothek im Hintergrund geöffnet.

Endor

Hallo aborix.
Ich habe das hier mal i aktuellen 61.0.2er getestet, funktioniert nicht.
Beim öffnen eines Links in der Sidebar aus der Chronik wird hier der aktuelle Tab überschrieben.
Es wird kein neuer geöffnet.
browser.tabs.loadBookmarksInBackground ist auf false gesetzt.

Konsole sagt:
SyntaxError: function statement requires a name[Weitere Informationen] newtabfromhistory.uc.js:1:8

Code

function() {
  if (location != 'chrome://browser/content/browser.xul')
    return;
  eval('PlacesUIUtils.openNodeWithEvent = ' + PlacesUIUtils.openNodeWithEvent.toString()
    .replace('PlacesUtils.nodeIsBookmark(aNode) && ', ''));
})();

Ist da das von Speravir erwähnte tablock Script auch notwendig?
Mfg.
Endor

aborix

Das Tablock-Skript braucht man nicht.

Das Skript verträgt sich wahrscheinlich nicht mit newtabfromhistory.uc.js. Ich kenne das nicht, aber dem Namen nach bewirkt das das Gleiche oder Ähnliches.

Endor

Das ist dein Script von oben, habe es mal so genannt. :wink:
Mfg.
Endor

aborix

Ach so!

Beim Kopieren ist am Anfang eine Klammer verlorengegangen.
Jetzt müsste es passen:

Code

(function() {
  if (location != 'chrome://browser/content/browser.xul')
    return;
  eval('PlacesUIUtils.openNodeWithEvent = ' + PlacesUIUtils.openNodeWithEvent.toString()
    .replace('PlacesUtils.nodeIsBookmark(aNode) && ', ''));
})();

Endor

Und ob es jetzt funktioniert. :klasse:
Wenn es Dir recht ist, gebe ich es weiter an Mithrandir.
Finde ich nämlich sehr praktisch.

Vielen Dank für das Script.

Ps. Eine Frage, beseht eventuell auch die Möglichkeit, dass es auch in der Menüleiste bei Chronik
funktioniert?

Mfg.
Endor

Speravir

Zitat von aborix

Code

(function() {
  if (location != 'chrome://browser/content/browser.xul')
    return;
  eval('PlacesUIUtils.openNodeWithEvent = ' + PlacesUIUtils.openNodeWithEvent.toString()
    .replace('PlacesUtils.nodeIsBookmark(aNode) && ', ''));
})();

Danke, Aborix, funktioniert prima!

Zitat von Endor

Wenn es Dir recht ist, gebe ich es weiter an Mithrandir.
Finde ich nämlich sehr praktisch.

Mach das, bitte.

Disclaimer: Weil so lange keine Antwort kam, hatte ich Aborix per PN gefragt, ob er vielleicht doch eine Möglichkeit kenne. Diese ist ihm nun in den Sinn gekommen und er hat mir das Skript gleichfalls per PN gesendet. Weil es funktioniert, habe ich ihn gebeten, das auch öffentlich zu posten. Mein Hintergedanke war genau der, dass es für andere ebenfalls sinnvoll sein könne.

Zitat von Endor

Ps. Eine Frage, beseht eventuell auch die Möglichkeit, dass es auch in der Menüleiste bei Chronik
funktioniert?

Oh, da ich die Chronik von dort aus nicht nutze, ist mir das gar nicht aufgefallen. Aber Du hast Recht, wenn es denn skripttechnisch möglich ist, sollte die Funktion möglichst ebenso eingebaut werden. (Ich schenk dir übrigens ein t. :wink: )

Edit: Ich merke aber gerade, dass bei mir von der Chronik im Menü aus die Seiten in einem neuen Tab geöffnet werden.

aborix

Zitat von Endor

Eine Frage, beseht eventuell auch die Möglichkeit, dass es auch in der Menüleiste bei Chronik
funktioniert?

Bitte testet:

Code

(function() {


  if (location != 'chrome://browser/content/browser.xul')
    return;


  eval('PlacesUIUtils.openNodeWithEvent = '  + PlacesUIUtils.openNodeWithEvent.toString()
    .replace('PlacesUtils.nodeIsBookmark(aNode) && ', ''));


  let onPopupshowing = function() {
    let historyMenu = document.getElementById('history-menu');
    if (!historyMenu._placesView) {
      new HistoryMenu(event);
      historyMenu._placesView._onCommand = function HM__onCommand(aEvent) {
        let placesNode = aEvent.target._placesNode;
        if (placesNode) {
          PlacesUIUtils.openNodeWithEvent(placesNode, aEvent);
        };
      };
    };
  };


  let historyPopup = document.getElementById('goPopup');
  historyPopup.setAttribute('onpopupshowing', '(' + onPopupshowing.toString() + ')()');


})();

Alles anzeigen

Zitat von Endor

Wenn es Dir recht ist, gebe ich es weiter an Mithrandir.

Ist mir recht.

Endor

Hallo aborix.
Funktioniert hier bestens.
:klasse:

Vielen Dank!
Mfg.
Endor

Endor

So habe das Script jetzt bei mir hoch geladen, Info am Mithrandir
geht demnächst ab.

https://github.com/Endor8/userChr…omhistory.uc.js

Mfg.
Endor

aborix

Das Skript lässt sich kurz so beschreiben, vielleicht auch im Readme:

Die Einstellungen
browser.tabs.loadBookmarksInTabs
und
browser.tabs.loadBookmarksInBackground
gelten gleichermaßen für Lesezeichen und Chronik-Einträge, egal von wo sie geöffnet werden.

Boersenfeger

Wenn das Script nichts weiter macht, als die beiden Einstellungen einzutragen bzw. zu ändern, kann man sich das auch sparen und in der user.js entsprechend hinterlegen...

aborix

Ohne das Skript gelten die Einstellungen nur für Lesezeichen und nicht für Chronik-Einträge.

Boersenfeger

Ahh, ok... Danke!
Ich arbeite nicht mit der Chronik..

aborix

Das Skript aus #9 funktioniert nicht mit Fx 62. Lesezeichen und Chronik-Einträge lassen sich nicht aufrufen.

Mit einer kleinen Änderung funktioniert es wieder:

Code

(function() {


  if (location != 'chrome://browser/content/browser.xul')
    return;


  eval('PlacesUIUtils.openNodeWithEvent = '  + PlacesUIUtils.openNodeWithEvent.toString()
    .replace(' && PlacesUtils.nodeIsBookmark(aNode)', '')
    .replace('getBrowserWindow(window)', 
      '(window && window.document.documentElement.getAttribute("windowtype") == "navigator:browser") ? window : BrowserWindowTracker.getTopWindow()')
  );


  let onPopupshowing = function() {
    let historyMenu = document.getElementById('history-menu');
    if (!historyMenu._placesView) {
      new HistoryMenu(event);
      historyMenu._placesView._onCommand = function HM__onCommand(aEvent) {
        let placesNode = aEvent.target._placesNode;
        if (placesNode) {
          PlacesUIUtils.openNodeWithEvent(placesNode, aEvent);
        };
      };
    };
  };


  let historyPopup = document.getElementById('goPopup');
  historyPopup.setAttribute('onpopupshowing', '(' + onPopupshowing.toString() + ')()');


})();

Alles anzeigen

Speravir

aborix Es scheint sich in Fx70 etwas geändert zu haben. Das Skript funktioniert jedenfalls nicht mehr. In Fx69 ging es noch. browser.xul und eval waren bereits dort ersetzt, das Skript sieht jetzt bei mir so aus:

JavaScript: NewTabFromSidebarMenu.uc.js

(function() {    if (location != AppConstants.BROWSER_CHROME_URL)        return;    function PUIU_openNodeWithEvent(aNode, aEvent) {        let window = aEvent.target.ownerGlobal;        let browserWindow = (window && window.document.documentElement.getAttribute("windowtype") == "navigator:browser") ? window : BrowserWindowTracker.getTopWindow();        let where = window.whereToOpenLink(aEvent, false, true);        if (this.loadBookmarksInTabs) {            if (where == "current" && !aNode.uri.startsWith("javascript:")) {                where = "tab";            }            if (where == "tab" && browserWindow.gBrowser.selectedTab.isEmpty) {                where = "current";            }        }        this._openNodeIn(aNode, where, window);    }    let onPopupshowing = function() {        let historyMenu = document.getElementById('history-menu');        if (!historyMenu._placesView) {            new HistoryMenu(event);            historyMenu._placesView._onCommand = function HM__onCommand(aEvent) {                let placesNode = aEvent.target._placesNode;                if (placesNode) {                    PlacesUIUtils.openNodeWithEvent(placesNode, aEvent);                };            };        };    };    let historyPopup = document.getElementById('goPopup');    historyPopup.setAttribute('onpopupshowing', '(' + onPopupshowing.toString() + ')()');})();

Nachtrag: Ach, ärgerlich, dass beim Kopieren die Zeilenumbrüche nicht beachtet werden.

aborix

So funktioniert es:

JavaScript

(function() {

  if (location != AppConstants.BROWSER_CHROME_URL)
    return;

  PlacesUIUtils.openNodeWithEvent = function PUIU_openNodeWithEvent(aNode, aEvent) {
    let window = aEvent.target.ownerGlobal;
    let browserWindow = (window && window.document.documentElement.getAttribute('windowtype') == 'navigator:browser') ? window : BrowserWindowTracker.getTopWindow();
    let where = window.whereToOpenLink(aEvent, false, true);
    if (this.loadBookmarksInTabs) {
      if (where == 'current' && !aNode.uri.startsWith('javascript:')) {
        where = 'tab';
      }
      if (where == 'tab' && browserWindow.gBrowser.selectedTab.isEmpty) {
        where = 'current';
      }
    }
    this._openNodeIn(aNode, where, window);
  }

  let onPopupshowing = function() {
    let historyMenu = document.getElementById('history-menu');
    if (!historyMenu._placesView) {
      new HistoryMenu(event);
      historyMenu._placesView._onCommand = function HM__onCommand(aEvent) {
        let placesNode = aEvent.target._placesNode;
        if (placesNode) {
          PlacesUIUtils.openNodeWithEvent(placesNode, aEvent);
        };
      };
    };
  };

  let historyPopup = document.getElementById('goPopup');
  historyPopup.setAttribute('onpopupshowing', '(' + onPopupshowing.toString() + ')()');

})();

Alles anzeigen

In Zeile 6 fehlte am Anfang PlacesUIUtils.openNodeWithEvent =

Ohne dem funktioniert es auch in Fx 69 nicht. Sonst habe ich nichts verändert.

Speravir

Zitat von aborix

In Zeile 6 fehlte am Anfang PlacesUIUtils.openNodeWithEvent =
Ohne dem funktioniert es auch in Fx 69 nicht. Sonst habe ich nichts verändert.

Danke!

Also habe ich einen Fehler beim Ersetzen von eval gemacht. Allerdings bin ich mir sicher, das Skript in Fx69 genutzt zu haben, dann aber wohl vor der Ersetzung <grybel>.

Speravir

milupo hat auf eine notwendige Codeänderung ab Fx 127 hingewiesen, siehe Posting vom 9. Mai 2024 um 20:15.

Im von Aborix oben in Beitrag #18 präsentierten Skript ist es Zeile 9:

alt:

JavaScript

    let where = window.whereToOpenLink(aEvent, false, true);

neu:

JavaScript

let where = window.BrowserUtils.whereToOpenLink(aEvent, false, true);

Und wenn man in Zeile 34 gopopup durch historyMenuPopup ersetzt, funktioniert das Skript auch wieder im Chronikmenü:

alt:

JavaScript

let historyPopup = document.getElementById('goPopup');

neu:

JavaScript

let historyPopup = document.getElementById('historyMenuPopup');

Von wem diese Korrektur ursprünglich stammt, ist mir unbekannt.

Vollständig ist das Skript nun bei Endor zu finden: userChrome.js/Firefox 127/newtabfromhistory.uc.js at master · Endor8/userChrome.js.