Eintrag im (Tab)Kontextmenü verschieben

Zitat von Mira_Belle

nur bekomme ich es nicht hin.

Das geht auch nur per Script.

Sieht dann so aus:

(function() {

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

  let menuitem = document.createXULElement('menuitem');
  menuitem.id = 'context_closeLeftTabs';
  menuitem.setAttribute('label', 'Linke Tabs schließen');
  menuitem.setAttribute('oncommand',
    'for (let i = TabContextMenu.contextTab._tPos - 1; i >= 0; i--) \
       gBrowser.removeTab(gBrowser.tabs[i], {animate: true})');

  let tabContextMenu = document.getElementById('tabContextMenu');
  let refItem = document.getElementById('context_undoCloseTab');
  tabContextMenu.insertBefore(menuitem, refItem);
  tabContextMenu.insertBefore(document.getElementById('context_closeTabsToTheEnd'), refItem);
  tabContextMenu.insertBefore(document.getElementById('context_closeOtherTabs'), refItem);
  tabContextMenu.removeChild(document.getElementById('context_closeTabOptions'));

})();

Damit werden dann alle 3 Einträge verschoben.

Wenn du nur den einen Eintrag verschieben willst, dann musst du das im Script anpassen/auskommentieren.

Sieht dann so aus:

(function() {

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

  let menuitem = document.createXULElement('menuitem');
  //menuitem.id = 'context_closeLeftTabs';
  //menuitem.setAttribute('label', 'Linke Tabs schließen');
  menuitem.setAttribute('oncommand',
    'for (let i = TabContextMenu.contextTab._tPos - 1; i >= 0; i--) \
       gBrowser.removeTab(gBrowser.tabs[i], {animate: true})');

  //let tabContextMenu = document.getElementById('tabContextMenu');
  let refItem = document.getElementById('context_undoCloseTab');
  //tabContextMenu.insertBefore(menuitem, refItem);
  //tabContextMenu.insertBefore(document.getElementById('context_closeTabsToTheEnd'), refItem);
  tabContextMenu.insertBefore(document.getElementById('context_closeOtherTabs'), refItem);
 // tabContextMenu.removeChild(document.getElementById('context_closeTabOptions'));

})();

Zitat von Mira_Belle

ganz unten ein Seperator!

Hast du mal einen Screenshot dazu?

Hier funktioniert das mit dem Code auf deine Art einwandfrei.

Zitat von Mira_Belle

so sieht es hier aus.

Da habe ich gar keinen Separator

So sieht das aus, wenn alle ausgeblendet werden:

#tabContextMenu > menuseparator:nth-child(n+2) {
  display: none !important;
}

Zitat von Mira_Belle

Hab den Fehler gefunden!!

Zitat von Mira_Belle

erklärst mal das Skript?

Das Script stammt von aborix.

Es verschiebt die Einträge aus einem Popup nur in das andere.

Zitat von Mira_Belle

In welcher Zeile ist der Befehl zum Generieren

Ich bin auch kein Fachmann für Scripte

menuitem.setAttribute('oncommand',
    'for (let i = TabContextMenu.contextTab._tPos - 1; i >= 0; i--) \
       gBrowser.removeTab(gBrowser.tabs[i], {animate: true})');

Zitat von Mira_Belle

wo der neue Eintrag hin soll.

 let refItem = document.getElementById('context_undoCloseTab');

Zitat von Mira_Belle

dass der alte Eintrag gelöscht oder ausgeblendet werden soll.

tabContextMenu.insertBefore(document.getElementById('context_closeTabsToTheEnd'), refItem);

Ob das so wirklich passt, muss ein Fachmann bitte mal überprüfen.

Zitat von Mira_Belle

noch ein JavaSkript-Experte meldet.

Evtl. meldet sich ja aborix dazu

2002Andreas Du hast ein Element menuitem erstellt, von dem ich aber nicht sehe, dass es anschließend überhaupt genutzt wird. Das kann also auch auskommentiert oder entfernt werden.

---

Zur Erklärung der Zeilen:

let menuitem = document.createXULElement('menuitem');

Es wird ein neues menuitem-Element erstellt und einer lokalen Variablen zugewiesen, welche dazu passend den Namen „menuitem“ trägt.

menuitem.setAttribute('oncommand', 'for (let i = TabContextMenu.contextTab._tPos - 1; i >= 0; i--) \ gBrowser.removeTab(gBrowser.tabs[i], {animate: true})');

Dem neu erstellten Menüpunkt wird ein oncommand-Attribut zugewiesen. Das Script, welches hier als String hinterlegt worden ist, wird bei Klick auf den Menüpunkt ausgeführt.

Ausgehend davon, dass menuitem aber in der Folge gar nicht genutzt wird, braucht es beides auch nicht.

let tabContextMenu = document.getElementById('tabContextMenu');
let refItem = document.getElementById('context_undoCloseTab');

Ähnlich zur menuitem-Zeile. Bloß statt ein neues Element zu erstellen, wird ein bereits bestehendes Element mit der ID #tabContextMenu ausgelesen und der lokalen Variable tabContextMenu zugewiesen. Gleiches mit #context_undoCloseTab und der lokalen Variable mit dem Namen „refItem“.

Wichtig: Durch das Zuweisen zu einer lokalen Variable kann man in der Folge mit dem Element arbeiten. Nur durch das Zuweisen passiert gar nichts. Hier wird damit dann weiter gearbeitet:

  tabContextMenu.insertBefore(document.getElementById('context_closeOtherTabs'), refItem);

Die Methode Node.insertBefore() wird auf ein Eltern-Element angewendet, in dem Fall tabContextMenu. Der erste Parameter ist das Element, welches eingefügt werden soll. Statt ein neues Element zu erstellen, wird hier mittels document.getElementById('context_closeOtherTabs') das Element mit der ID #context_closeOtherTabs genommen (hätte natürlich genauso vorher einer lokalen Variable zugewiesen werden können). Der zweite Parameter ist das Referenz-Element, sprich das Element, vor dem das neue Element hinzugefügt werden soll.

Der „Trick“ hier ist, dass wenn kein neues Element eingefügt wird, sondern ein bereits bestehendes Element als erster Parameter angegeben wird, dieses verschoben und nicht etwa dupliziert wird.

Zitat von Sören Hentzschel

In deinem Script in Beitrag

Hallo Sören, vielen Dank für die Erklärung

Ich habe einfach etwas rumgebastelt in dem Script, was dann sogar durch Zufall funktioniert hat.

'nabend Mira_Belle ...

Wie sieht das denn bei Dir aus?

Ich erhalte mit deinem Code folgendes:

"Andere Tabs schließen" hatte ich vorher garnicht...