Problem mit Code für die Lesezeichen

camel-joe

Hallo,

vor einiger Zeit habe ich von Endor diesen Code erhalten:

Code

/* Alle Lesezeichen anzeigen */


[
{
"label" : "Alle Lesezeichen anzeigen",
"accesskey": "Strg+Umschalt+B",
"oncommand": "window.open('chrome://browser/content/places/places.xul', 'Browser:ShowAllBookmarks', 'chrome,resizable=yes');"
},
]
.forEach(function(attrs) {
var menuitem = document.createElement("menuitem");
for (var key in attrs)
menuitem.setAttribute(key, attrs[key]);
document.getElementById("menu_ToolsPopup").insertBefore(menuitem, document.getElementById("menu_preferences"));
});

Alles anzeigen

Hier der Beitrag dazu: https://www.camp-firefox.de/forum/viewtopi…t=98785#p893766

Nun habe ich aber das Problem, dass dadurch die Leiste mit Importieren und Sichern nicht angezeigt wird.

Endor

Hallo camel-joe
Ja habe ich eben auch gesehen. Da fehlt oben die ganze Leiste.
Mal sehen.
Mfg.
Endor

aborix

Entschuldigung für die Einmischung, aber ich konnte nicht widerstehen:

Code

(function () {
   if (location != 'chrome://browser/content/browser.xul') return;
	var Lesezeichen_verwalten = document.getElementById('bookmarksShowAll');
	Lesezeichen_verwalten.parentElement.removeChild(Lesezeichen_verwalten.nextSibling);
	var Einstellungen = document.getElementById('menu_preferences');
	Einstellungen.parentElement.insertBefore(Lesezeichen_verwalten, Einstellungen);   
}) ();

Endor

Das ist keine Einmischung, sonder Rettung in höchster Not.
Vielen Dank aborix!!! :klasse:
Mfg.
Endor

camel-joe

Hallo zusammen,

kann ich davon ausgegen, dass der Code von aborix den Code von Endor ersetzen soll? Oder soll er hinzugefügt werden?

aborix

Ersetzen, nicht hinzufügen.

Wenn der Eintrag wo anders hin soll oder eine Trennlinie hinzukommen soll, sag Bescheid.
Ein anderer Text ist natürlich auch möglich.

camel-joe

Alles klar, es funktioniert prima. Bisher passt alles. Vielen Dank. :klasse:
Eine Alternative für den Text wäre "Alle Lesezeichen anzeigen" (wie beim Original).

aborix

Gern geschehen.

mit neuem Text:

Code

(function () {
   if (location != 'chrome://browser/content/browser.xul') return;
	var Lesezeichen_verwalten = document.getElementById('bookmarksShowAll');
	Lesezeichen_verwalten.parentElement.removeChild(Lesezeichen_verwalten.nextSibling);
	var Einstellungen = document.getElementById('menu_preferences');
	Einstellungen.parentElement.insertBefore(Lesezeichen_verwalten, Einstellungen);
	Lesezeichen_verwalten.setAttribute('label', 'Alle Lesezeichen anzeigen');
}) ();

Mittlerweile weiß ich auch, was bei Endors Code fehlt:
bei den Eigenschaften des Fensters muß "toolbar" dabei sein:

Code

"oncommand": "window.open('chrome://browser/content/places/places.xul', 'Browser:ShowAllBookmarks', 'chrome,resizable=yes,toolbar');"

Für "oncommand" ist folgendes evtl. besser, denn damit wird das Bibliotheksfenster auf jeden Fall vollständig angezeigt und man braucht keine zusätzlichen Angaben:

Code

"oncommand": "PlacesCommandHook.showPlacesOrganizer('AllBookmarks')"

Oder auch folgendes, hier jetzt mit "command" anstatt "oncommand":

Code

"command": "Browser:ShowAllBookmarks"