Eintrag im (Tab)Kontextmenü verschieben

Tja, meinereiner sollte vorm Testen mal alles lesen

Vielleicht schaffe ich es ja mit Deiner Vorlage "In neues Fenster verschieben" aus "Tab verschieben" hochzuholen.

Schönes Wochenende, ich widme mich jetzt primär der Damen-Nationalelf gegen die Französinnen...

Um das noch zu beantworten:

Zitat von Mira_Belle

mit oder ohne ' , oder eher mit " ?

Ob einfache oder doppelte Anführungszeichen kannst du dir aussuchen. Aus meiner Sicht sind einfache Anführungszeichen vorzuziehen, aber das ist eine Geschmacksfrage. Nur sollte man sich entscheiden. Beides zu mischen, ist nicht hübsch.

PS: In deiner letzten Version hast du in den Zeilen 5 und 6 noch jeweils überflüssige Klammern. Und let kann vermutlich durch const ersetzt werden.

Zitat von Mira_Belle

Habe ich auch gelesen! Niemals mischen, sonst funktioniert es nicht.

Du musst natürlich mit dem gleichen Anführungszeichen schließen, wie du geöffnet hast, richtig. Ich meinte aber auch, dass man im gesamten Script konsistent sein sollte, damit es nicht so willkürlich ist. In Zeile 2 verwendest du einfache Anführungszeichen, in den Zeilen 5 und 6 doppelte. Das macht funktional überhaupt nichts, meinte ich aber mit nicht hübsch.

Zitat von Mira_Belle

Was die Variablen angeht, ist mir immer noch nicht klar wann und warum man mal let

und mal const benutzt.

Wenn du eine Variable nach der Deklaration nicht noch einmal überschreibst, kannst du const verwenden, sonst muss es let sein (oder eben var). Es ist aber nicht so, dass es falsch wäre, immer let zu verwenden. Mein Ansatz ist halt, so restriktiv wie möglich zu starten und dann zu lockern, wo notwendig. Die Idee dahinter ist es, unerwartetes Verhalten durch versehentliches Überschreiben von Variablen bestmöglich zu vermeiden. Bei nur so wenigen Zeilen Code wird das zugegeben kein Thema sein. Andererseits kann man sich ja beim kleinen Code schon ein paar Dinge angewöhnen, über die man dann später bei komplexeren Aufgaben gar nicht mehr bewusst nachdenkt, sondern schon verinnerlicht hat.

Zitat von Mira_Belle

Kannst Du mir noch zeigen, wie man das Label, also den angezeigten Text eines Eintrags ändert?

Bei #context_openTabInWindow wird ja "In neues Fenster verschieben" angezeigt.

Es macht aber Sinn, wenn der Eintrag eine Ebene nach Oben verschoben wurde,

daraus "Tab in neues Fenster verschieben" zu machen.

cs_openTabs.setAttribute('label', 'Tab in neues Fenster verschieben');

Zitat von Mira_Belle

Teste mal.

klappt...

Zitat von Mira_Belle

Wenn Du den Eintrag an andere Stelle haben möchtest, gib Bescheid.

Selbst ist der Mann, habbich hinbekommen...

Zitat von Mira_Belle

an (1) den Code eingefügt, funktioniert das ganze Script nicht (mehr).

An (2) oder (3), funktioniert zwar grundsätzlich das Script,

aber der Text wird nicht geändert.

Hier auch.

Sieht aktuell bei mir noch so aus:

Ich hab dann mal per zusätzlichem .css versucht, den Text zu ersetzen, bekomme das aber auch nicht hin.

Vielleicht hat 2002Andreas ja einen Vorschlag/eine Idee

Zitat von Mira_Belle

an (1) den Code eingefügt, funktioniert das ganze Script nicht (mehr).

Logisch. Wie willst du auf etwas zugreifen, was an der Stelle noch überhaupt nicht definiert ist? Das steht dann aber eh in der Konsole, weil das ein Script-Fehler ist.

Zitat von Mira_Belle

An (2) oder (3), funktioniert zwar grundsätzlich das Script,

aber der Text wird nicht geändert.

Dann wird das ein Timing-Problem sein, weil das Script den Text vermutlich setzt, bevor Firefox das macht. Versuch es mit etwas Verzögerung:

(function() {
  if (location !== 'chrome://browser/content/browser.xhtml') {
    return;
  }

  const cs_AllTabs = document.getElementById('context_selectAllTabs');
  const cs_openTabs = document.getElementById('context_openTabInWindow');

  setTimeout(() => {
    cs_openTabs.setAttribute('label', 'Tab in neues Fenster verschieben');
  }, 1000);

  tabContextMenu.insertBefore(cs_openTabs, cs_AllTabs);
})();

Zitat von harff182

den Text zu ersetzen

#context_openTabInWindow label {
  display: none !important;
}

#context_openTabInWindow::after {
  content: "Tab in neues Fenster verschieben" !important;
  font-size: 13px !important;
}

2002Andreas:

Da ich meine alten Versuche noch im .css drin hatte, konnte ich meinen Fehler finden und lernen

Zitat von harff182

konnte ich meinen Fehler finden und lernen

Zitat von Mira_Belle

wie man das 2. Script a. optimieren und b. beide zusammen bringen kann,

(function() {
  if (location != 'chrome://browser/content/browser.xhtml')
    return;

    let cs_AllTabs = document.getElementById('context_selectAllTabs');
    let cs_openTab = document.getElementById('context_openTabInWindow');    

      setTimeout(() => {
    cs_openTab.setAttribute('label', 'Tab in neues Fenster verschieben');
  }, 1000);
    
    tabContextMenu.insertBefore(cs_openTab, cs_AllTabs);
})();

Zitat von Mira_Belle

aber bei mir funktioniert es so nicht!

Du hattest es mit meinem! Script getestet?

(Das Script von Sören aus Beitrag Nr.29 funktioniert hier nicht)

Funktioniert auch im Nightly einwandfrei.

Den Wert für Timeout auch mal verändert?

Zitat von Mira_Belle

Script a. optimieren und b. beide zusammen bringen

(function() {
  if (location != 'chrome://browser/content/browser.xhtml')
    return;
    let cs_AllTabs = document.getElementById('context_selectAllTabs');
    let cs_openTab = document.getElementById('context_openTabInWindow');
    tabContextMenu.insertBefore(cs_openTab, cs_AllTabs);
Components.utils.import("resource://gre/modules/Services.jsm");
let ss =  Components.classes["@mozilla.org/content/style-sheet-service;1"].getService(Components.interfaces.nsIStyleSheetService);
let custom_label = {
  init: function() {
    let uri = Services.io.newURI("data:text/css;charset=utf-8," + encodeURIComponent(`
     
      @namespace html url("http://www.w3.org/1999/xhtml");
     
      #context_openTabInWindow label {
        display: none !important;
      }
      #context_openTabInWindow::after {
        content: "Tab in neues Fenster verschieben" !important;
        font-size: 13px !important;
      }
    `), null, null);
    ss.loadAndRegisterSheet(uri, ss.AGENT_SHEET);
  }
};
custom_label.init();
})();

Zitat von Mira_Belle

Hatte den Wert setTimeout(() => { ... sogar auf 50000 erhöht, passierte aber nichts.

Zitat von 2002Andreas

(Das Script von Sören aus Beitrag Nr.29 funktioniert hier nicht)

Mein Fehler. Ich hatte mein Script mehrfach erfolgreich getestet und dann nach dem Testen die location-Prüfung auf eine strikte Prüfung (!== statt !=) geändert und das dann nicht mehr getestet. Ich hatte übersehen, dass location kein String ist. Wenn man strikt prüft (was man eigentlich immer tun sollte; ich verbiete in meinen Scripts per Linter nicht strikte Vergleiche), muss man location.href prüfen.

Hier also das korrigierte Script:

(function() {
  if (location.href !== 'chrome://browser/content/browser.xhtml') {
    return;
  }

  const cs_AllTabs = document.getElementById('context_selectAllTabs');
  const cs_openTabs = document.getElementById('context_openTabInWindow');

  setTimeout(() => {
    cs_openTabs.setAttribute('label', 'Tab in neues Fenster verschieben');
  }, 1000);

  tabContextMenu.insertBefore(cs_openTabs, cs_AllTabs);
})();

Zitat von Mira_Belle

Und eine letzte Frage!

Um einen Eintrag zu kopieren (klonen)

muss = document.getElementById( durch was ersetzt werden?

Meine Script-Erklärungen in Beitrag #16 waren wohl nutzlos. Dabei muss man sogar nur den Namen der Funktion Document.getElementById() lesen, der ja auch nur normales Englisch ist, um zu sehen, was an dieser Stelle passiert: Man gibt eine ID an und erhält das Element dafür zurück. Da wird weder etwas verschoben noch in sonst einer Weise geändert. Der Teil muss also überhaupt nicht ersetzt werden.

Wie ich bereits schrieb, findet das Verschieben als Teil der Node.insertBefore()-Methode statt, wenn ein existierendes statt ein neu erstelltes Element als erster Parameter übergeben wird. Verwende Node.cloneNode() zum Duplizieren des Original-Elements:

tabContextMenu.insertBefore(cs_openTabs.clondeNode(), cs_AllTabs);

Zu beachten ist, dass das Original und Duplikat nicht automatisch synchron gehalten werden. Damit:

cs_openTabs.setAttribute('label', 'Tab in neues Fenster verschieben');

… wird also nur der Original-Eintrag umbenannt. Die Komplettlösung sieht dementsprechend beispielsweise so aus:

(function() {
  if (location.href !== 'chrome://browser/content/browser.xhtml') {
    return;
  }

  const cs_AllTabs = document.getElementById('context_selectAllTabs');
  const cs_openTabsOriginal = document.getElementById('context_openTabInWindow');
  const cs_openTabsCopy = cs_openTabsOriginal.cloneNode();

  setTimeout(() => {
    cs_openTabsOriginal.setAttribute('label', 'Tab in neues Fenster verschieben');
    cs_openTabsCopy.setAttribute('label', 'Tab in neues Fenster verschieben');
  }, 1000);

  tabContextMenu.insertBefore(cs_openTabsCopy, cs_AllTabs);
})();