Inkompatibilitäten zwischen Addons und UserChromeJS-Skripten

Speravir

Ich setze das Folgende mal in ein eigenes Thema, so dass andere Leute eventuelle Beobachtungen ergänzen können. Ich habe das neue Thema in „Anpassungen“ erstellt, weil ich denke, dass es im Zweifelsfall leichter wäre, die Skripte anzupassen statt die Autoren zu einer Änderung zu bewegen.

Ich musste feststellen, dass einige userChrome.js-Skripte nicht (mehr) mit einigen Addons kompatibel sind. Sind die Addons deaktiviert, funktionieren die Skripte noch.

In meinem Fall sind es die Skripte animationtogglebutton_movable (Version v. 31.10.2014) und clearcachebuttonmovable (Version v. 25.10.2014) (möglicherweise wäre auch clearcache in derselben Version betroffen?) sowie die Addons Print Edit und Tab Scope. Während ich zur Not auf Print Edit verzichten könnte (aber andere vielleicht nicht …), ist Tab Scope für mich essentielle, täglich genutzte Funktionalität.

Ich weiß leider überhaupt nicht, wo ich da ansetzen soll, meine Javascript-Kenntnisse sind bestenfalls als rudimentär zu bezeichnen, und so wäre es schön, wenn jemand helfen könnte.

aborix

Teste zum Cache leeren dieses Skript:

Code

(function() {


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

	try {

		CustomizableUI.createWidget({

			id: 'clearcache-button',
			type: 'custom',
			defaultArea: CustomizableUI.AREA_NAVBAR,

			onBuild: function(aDocument) {
				var toolbaritem = aDocument.createElementNS('http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul', 'toolbarbutton');
				var attributes = {
					id: 'clearcache-button',
					class: 'toolbarbutton-1 chromeclass-toolbar-additional',
					removable: 'true',
					label: 'Cache leeren',
					tooltiptext: 'Cache leeren',
					style: 'list-style-image: url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8%2F9hAAAAGXRFWHRTb2Z0d2FyZQBBZG9iZSBJbWFnZVJlYWR5ccllPAAAAxhpVFh0WE1MOmNvbS5hZG9iZS54bXAAAAAAADw%2FeHBhY2tldCBiZWdpbj0i77u%2FIiBpZD0iVzVNME1wQ2VoaUh6cmVTek5UY3prYzlkIj8%2BIDx4OnhtcG1ldGEgeG1sbnM6eD0iYWRvYmU6bnM6bWV0YS8iIHg6eG1wdGs9IkFkb2JlIFhNUCBDb3JlIDUuMC1jMDYwIDYxLjEzNDM0MiwgMjAxMC8wMS8xMC0xODowNjo0MyAgICAgICAgIj4gPHJkZjpSREYgeG1sbnM6cmRmPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5LzAyLzIyLXJkZi1zeW50YXgtbnMjIj4gPHJkZjpEZXNjcmlwdGlvbiByZGY6YWJvdXQ9IiIgeG1sbnM6eG1wPSJodHRwOi8vbnMuYWRvYmUuY29tL3hhcC8xLjAvIiB4bWxuczp4bXBNTT0iaHR0cDovL25zLmFkb2JlLmNvbS94YXAvMS4wL21tLyIgeG1sbnM6c3RSZWY9Imh0dHA6Ly9ucy5hZG9iZS5jb20veGFwLzEuMC9zVHlwZS9SZXNvdXJjZVJlZiMiIHhtcDpDcmVhdG9yVG9vbD0iQWRvYmUgUGhvdG9zaG9wIENTNSIgeG1wTU06SW5zdGFuY2VJRD0ieG1wLmlpZDpBQzdFNERGRDNCNkYxMUUwQjRDRUVDOEM3Q0YyMDZEMSIgeG1wTU06RG9jdW1lbnRJRD0ieG1wLmRpZDpBQzdFNERGRTNCNkYxMUUwQjRDRUVDOEM3Q0YyMDZEMSI%2BIDx4bXBNTTpEZXJpdmVkRnJvbSBzdFJlZjppbnN0YW5jZUlEPSJ4bXAuaWlkOkFDN0U0REZCM0I2RjExRTBCNENFRUM4QzdDRjIwNkQxIiBzdFJlZjpkb2N1bWVudElEPSJ4bXAuZGlkOkFDN0U0REZDM0I2RjExRTBCNENFRUM4QzdDRjIwNkQxIi8%2BIDwvcmRmOkRlc2NyaXB0aW9uPiA8L3JkZjpSREY%2BIDwveDp4bXBtZXRhPiA8P3hwYWNrZXQgZW5kPSJyIj8%2BKcFfnQAAAuZJREFUeNqkkslv1EgUxr%2FyUm272267O510RCcThS1IhE1Bg9CAWgyZuc8px5FGHIAj4sAdLvwBuXDgDwAklguI5QBCiGWQWEYsEyAKUTrB9JLEbaddLhfVSbghhERJr0ol1fu9V9%2F7iBACP7O07jZ5dXL1IlIJ4wAtuiAXLsF%2BdAObjozB6S%2BDcANyQxDHuHX59l9DmwdO9i1%2BOK79cCmRggDbgjaf4K5WTT8vnFB8%2BvTHAEIMa1bxH56IAzR9Zo3%2FNgwEJos7HN8HEPmlNP1VNwqndcvzqJZFv2XLTkLEzXdj2Vz%2BsfKNtJLMtLpyIFZ2UJ47qxuqJ%2FiyhAUwCxVoVgFhnPBPi2K9A%2FmYENKNQ1Ax3Ak6V4ousyx39ExA9mfLogEmQqjyDV96iRV%2FKmVh5gEIWwdIdaT6RI60Fzp9Xw59f8vGubO%2BMdB%2F7VkTI41ebDY7SGf%2Fx8zb16jPzk7ZPHwlZKK2ppGQxcnf8uS6aj4pBi%2F%2BKPXqh5xyCNLzEvM1C63QwLv5aWwdMGBo1vWokSREVdYAaqrqgoiMoqqPSNweK%2FW0TylGHlpCsGtHBnwkBuNL2DNeAmMxf%2FOieD8TebLldA1AFHJUVv%2FMhdafj6erfT3MTjiBxIJJCBcUumkiYVInwfjyXLTCOzEIXXdiumpBbEhSMeSQBleVDvScC2QqIKp0JRIZEeJ2EzSv0sEtrWMfn89PJlzUv05hlgtedazCRYf17LS8NtTew1BUipS1pMDLYFEMKxtDcIHRg4N%2FFgcKe33fPr8mYk48qdBKYHI7O%2FWhaA46fkxaD2mSUjh210xLKPVpqNUasB0Nb98kHb%2FutdztR1%2BtAnRTn6mOVmsLtYWR9mJk3%2Flv5SZd%2Fnf7TDOHil13389r8e%2F7QKenaaS6v8xltFytNDJ%2Bt1iu1Lqzx7l75zCxewK6pkuVWcavN50o6rhhO3CiMDTSVZspiecYKzTrffLyuUU3b7dZkuCLAAMAxwpPxFIuoXQAAAAASUVORK5CYII%3D)',
					oncommand: 'Services.cache2.clear();'
				};
				for (var a in attributes)
					toolbaritem.setAttribute(a, attributes[a]);
				return toolbaritem;
			}

		});

	} catch(e) { };

})();

Alles anzeigen

Speravir

Zitat von aborix

Teste zum Cache leeren dieses Skript:

Ja, das funktioniert. Also, es ist ein Symbol zu finden – witzigerweise nicht im Einstellungsfenster, sondern gleich an der Stelle, wo ich es ursprünglich bereits hingesetzt hatte. Und … kurz mal testen … ja, der Cache wird auch gelöscht. DANKE! Wäre das Skript schon mal repariert. Kannst Du erkennen, ob Clearcache auch betroffen wäre?

Eine Lösung für den AnimationToggleButton zu finden ist wohl nicht so leicht?

aborix

Ich habe ClearCache bisher nicht mit den betreffenden Erweiterungen getestet und ohne Test kann ich nichts sagen. Hast du es mal getestet?

Teste für den Animations-Button dieses Skript:

Code

(function() {


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


	try {


		CustomizableUI.createWidget({


			id: 'animation-button',
			type: 'custom',
			defaultArea: CustomizableUI.AREA_NAVBAR,


			onBuild: function(aDocument) {


				var button = aDocument.createElementNS('http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul', 'toolbarbutton');
				var attributes = {
					id: 'animation-button',
					class: 'toolbarbutton-1 chromeclass-toolbar-additional',
					removable: 'true',
					label: 'Animation',
					tooltiptext: 'Linksklick: GIF-Animationen einschalten\nMittelklick: Animation einmal abspielen\nRechtsklick: Ausschalten',
					oncontextmenu: 'return false'
				};
				for (var a in attributes)
					button.setAttribute(a, attributes[a]);
				var animmode = Services.prefs.getCharPref('image.animation_mode');
				button.setAttribute('anim', animmode);
				button.IsOnce = (animmode == 'once');


				function onClick() {


					var button = document.getElementById('animation-button');
					function setPref(value) {
						Services.prefs.setCharPref('image.animation_mode', value);
					};
					function getPref() {
						return Services.prefs.getCharPref('image.animation_mode');
					};
					function setIsOnce(value) {
						for (var win of Application.windows)
							win._window.document.getElementById('animation-button').IsOnce = value;
					};


					switch (event.button) {


						case 0:
							var animmode = getPref();
							setPref('normal');
							if (button.IsOnce) {
								BrowserReloadSkipCache();
								setIsOnce(false);
							} else {
								if (animmode == 'normal')
									BrowserReloadSkipCache();
								else
									BrowserReload();
							};
							break;


						case 1:
							setPref('once');
							BrowserReloadSkipCache();
							setIsOnce(true);
							break;


						case 2:
							setPref('none');
							BrowserStop();
							break;
					};


					for (var win of Application.windows)
						win._window.document.getElementById('animation-button').setAttribute('anim', getPref());
				};


				button.setAttribute('onclick', '(' + onClick.toString() + ')();');
				return button;
			}
		});


	} catch(e) { };


	var css =
		'#animation-button[anim="normal"] {list-style-image: url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAABbElEQVR42sWTP6vBURjHfy9DdiarDAZZhBhs/iS/SWGQMlBkIKKUuyMSCwklgxTyFmwWZq/BR8+x3Fu6Nwz31Klvp/N8nuf7PR1Ne2EZjcYvm81GMpmk3W7z62WDwaBLQblcJpfLoes6x+ORzWaDy+Uin88/B1gsFiqVCtPplFAoxGq1Yr1e43Q6CQQC+Hy+n4VWqxW73U6r1eJ6vZJKpZCuUuhwOPjT13w+p1qt4vF4yGazmEwmXslFi8fjZDIZvF6v8ildZdREIoHb7SYYDCotDUTLhJJBLBbD7/ejpdNp+v0+hUKBy+VCs9lUGZxOJxqNBrVaTWmZsl6vcz6fKZVKdDoder0emtlsVonOZjP2+z3L5ZLJZMJut1N6PB6r88ViofThcEBsb7dbFbKyMRqNuN1uL2+xrwBi4R2AZPYRIBwOPwASxkeAbrf7z4DBYPAWIBKJPADFYlG98XA4fLolZHnq72fy2aLRKHfeRoLqwjwI3AAAAABJRU5ErkJggg==)} ' +
		'#animation-button[anim="once"] {list-style-image: url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAABaElEQVR42p2TvYrCUBSE72OIvVa2ksIipBENWKRTQ0gqQVOIYKEgpIhYCIL7AAmx0EYRLcRCAip5BTsbrX0GR04s3PwsbDwwzQ3fnDsTLmMJJp1O/3Ach1arBWawt/6aVCqlEWCaJnq9HjRNw+Vygeu6HzjOIJfLYTgcYr1eo1arYb/f43A4QBAESJIUhEn5fB6FQgHT6RSPxwO6roO2EsjzfHBDGKbZbrcYjUYol8vodrvIZDLxueJgmkajgU6nA1EU/Zy0la7abDZRKpVQrVYj1y4Wi1BVFZVKBazdbmM2m2EwGOB+v2MymfgdXK9XjMfjCHy73WAYBizLguM4YNlsFv1+H5vNBufzGbvdDqvVCqfTKQIvl0t4ngeKfTwe/ZL9GIvFAs/nM6AwHP5Oovi+AUVICpOos4jBf2FSvV5/G1AZSeGAgW3bieGAwTfwx+BLmCTLctBgPp/HikqmX/37jB6boih4AcQgRNTa5HaFAAAAAElFTkSuQmCC)} ' +
		'#animation-button[anim="none"] {list-style-image: url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAABbUlEQVR42p2TP6uCYBjF/RY32lqaWqOhQVyigoa2/iA6CdUgQUNB0FA0BEH3AxQ56FKIDdEQgopfoa2l5j6Dp/u8F+J69cK1B87i6+887zkixyWYdDr9WSgU0O12EXAc058vp1IpmYDpdIrhcAhZlnG5XGDb9guONcjlcpjNZjBNE81mE6fTCefzGYIgoF6vh2BmkM/nUSwWsVqt8Hg80Ov1QFsJ5Hk+tCEC0xwOB8znc1QqFQwGA2QymY+4SLEwjaIo6Pf7qFarLCdtpat2Oh2Uy2U0Go3ItUulEiRJQq1WA6eqKjRNw3g8xv1+x3K5ZB1cr1csFosIfLvdMJlMsF6vsd1uwWWzWYxGI1iWBc/zcDwesd/v4bpuBN7tdvB9HxTbcRxWMothGAaCIAjrFxw5/xLFZwYUISlMos6iBv+ESa1W69uAykgKhww2m01iOGTwDvwyeBcmtdvtsIGu67GikulT/3xGP5soingC1EBhpA0JGaMAAAAASUVORK5CYII=)}';
	var stylesheet = document.createProcessingInstruction('xml-stylesheet', 'type="text/css" href="data:text/css;utf-8,' + encodeURIComponent(css) + '"');
	document.insertBefore(stylesheet, document.documentElement);


})();

Alles anzeigen

Speravir

Zitat von aborix

Ich habe ClearCache bisher nicht mit den betreffenden Erweiterungen getestet und ohne Test kann ich nichts sagen. Hast du es mal getestet?

Nein, hatte ich nicht. Ich hatte gehofft/gedacht, dass Du das evtl. relativ schnell erkennen kannst, wenn Du dir die anderen Skripte ansiehst.

Zitat

Teste für den Animations-Button dieses Skript:

Nachdem ich erst herausfinden musste, dass zwischendurch das Problem auf dem Stuhl vor dem Bildschirm saß (deswegen stundenlanges vergebliches Testen der vermeintlichen Erweiterung, die nun schuld sein musste, und erst heute eine Antwort), funktioniert das Skript nun auch! Cooool!!!11 Nochmal allerherzlichsten Dank!

aborix

Gern geschehen!

Die Buttons in AnimationToggleButtonMovable und ClearCacheButtonMovable werden auf gleiche Art erzeugt, in ClearCache ist es etwas anders. Darum konnte ich das Funktionieren schwer einschätzen.

Ich habe nun die Skripte mit aktivem Print Edit und Tab Scope getestet. Bei mir funktionieren allerdings alle drei.

Die beiden neuen Skripte sind wieder anders aufgebaut, Details stehen in
MDN - CustomizableUI.jsm.

aborix

Mithrandir und Endor haben hier bzw. hier berichtet, dass bei ihnen auch der Wetterfuchsbutton mit TabScope nicht verträglich ist.
Auf Anregung von Endor habe ich auch hier den Code-Teil zur Erzeugung des Buttons durch einen, der CustomizableUI.jsm verwendet, ersetzt.

Download:
http://www.xup.in/dl,76914779/Wetterfuchsbutton.uc.js/

Mithrandir

Hallo aborix,

magst du clearcachebuttonmovable aus Beitrag #2 noch so anpassen, dass auch das Menü unter "Extras" noch erscheint? Und dann fände ich es super, wenn du clearcachebuttonmovable sowie animationtogglebutton_movable wieder irgendwo hochlädst, sodass ich nicht mit den Leerzeichen am Anfang der Zeilen zu kämpfen habe.

Die anderen Anpassungen lade ich nun hoch (wetterfuchsbutton und saveuserchrome).

aborix

Ja, mache ich gerne.

http://www.xup.in/dl,75225265/AnimationToggleButton.uc.js/

http://www.xup.in/dl,65884740/ClearCacheButton.uc.js/

Der Cache ist seit Fx 32 anders aufgebaut, das Skript funktioniert dennoch auch in Fx 31esr.

Ich finde, das "Movable" in den Namen könnten wir auch weglassen.
Wir haben sonst noch mehrere bewegliche Buttons, die das auch nicht im Namen haben.
Wenn im Readme steht, dass der Button frei positionierbar ist, sollte das reichen.

Mithrandir

Danke, ich hoffe, dass ich jetzt alle erwischt habe, die Steffen auf GitHub vermisste.

Umbenennen ist ein wenig schwieirg, bzw. ich weiß nicht, wie ich die Historie dann von Datei A auf Datei B übertragen kann (ich bin eben kein ausgefuchster Git-Benutzer).

Sören Hentzschel

Wenn du Git zum Umbenennen verwendest, dann geht auch keine Historie verloren, sprich in der Kommandozeile git mv statt mv, beziehungsweise wenn du ein grafisches Tool für Git benutzt, dann dort einfach umbenennen.

Mithrandir

Danke, das (git mv Ordneralt Ordnerneu und dann im Ordnerneu noch git mv DateiAlt DateiNeu) habe ich jetzt probiert und anscheinend doch falsch gemacht. Ist aber auch egal, müssen wir hier nicht weiter erläutern.

Speravir

Zitat von Mithrandir

Danke, ich hoffe, dass ich jetzt alle erwischt habe, die Steffen auf GitHub vermisste.

THX. Im Speziellen und im Allgemeinen!
(Ich hatte Mithrandir per PN gebeten, evtl. doch die geänderten Versionen hochzuladen.)

aborix

Mal nur aus Neugier gefragt:

Funktioniert bei jedem, der es nutzt oder testet, AutoClickMod.uc.js zusammen mit Tab Scope?

Bir mir ja, aber das muss ja nichts heißen.

Endor

Bestätigt. Funktioniert hier auch einwandfrei.
Mfg.
Endor

Speravir

InspectElementModY ist nicht kompatibel mit InspectorWidget.

Das ist insofern schade, weil in der jüngsten Version von InspectorWidget die Möglichkeit eingeführt wurde, per Tastenkombination+Mausklick auch dort den DOM-Inspector zu öffnen, wo kein (weiteres) Popupmenü geöffnet werden kann. Auf das Addon könnte allerdings verzichtet werden, wenn man diese Möglichkeit in das Skript Contexpector einbauen könnte. Der Sinn wäre dann, dass man mit InspectElementModY, das ja nur per Tastenkombi gestartet wird, Firebug oder den internen Inspector öffnet, mit Contexpector den DOM-Inspector. Derzeit ist es sinnvoll, beide Varianten nutzen zu können, sie ergänzen sich.

Oder man baut, wenn möglich, in InspectElementModY die Möglichkeit für zwei verschieden Tastenkombinationen ein …

aborix

Zitat von Speravir

[...] per Tastenkombination+Mausklick auch dort den DOM-Inspector zu öffnen, wo kein (weiteres) Popupmenü geöffnet werden kann.

Für Einträge von Popupmenüs geht das mit dem Skript Popup Inspect.

Wenn ich am Ende des Skripts noch diese Zeile hinzufüge:

Code

document.addEventListener("click", EventListener, true);

funktioniert es anscheinend überall, insbes. auch in der Adress- und Suchleiste und beim Vor- und Zurück-Button.

Die Tastenkombination können wir auch ändern.

Speravir

Zitat von aborix

Für Einträge von Popupmenüs geht das mit dem Skript Popup Inspect.
Wenn ich am Ende des Skripts noch diese Zeile hinzufüge:
[…]

Aah, stimmt. ~~Dann teste ich das mal ein wenig.~~

Funktioniert bestens! Jetzt müsste nur noch eine andere Tastenkombination gefunden werden, um es gemeinsam mit InspectElementModY einsetzen zu können. Vielleicht dieselbe wie bei InspectorWidget? Das wäre dann STRG+SHIFT+<linker Mausklick>. Oder nur SHIFT, so dass PopupInspect mit linker Maustaste und InspectElementModY mit rechter Maustaste?

aborix

Eine Tastenkombination mit Linksklick ist insofern schlecht, als die beim jeweiligen Element übliche Funktion des Linksklicks sich nicht immer verhindern lässt. Das ist z.B. bei den Einträgen im Datei-Menü so.
Auch InspectorWidget hat diese Schwachstelle.

Da das Skript jetzt nicht mehr zu unterscheiden braucht, ob man in einem Popupmenü oder anderswo klickt, konnte ich es vereinfachen.
Die Tastenkombination ist jetzt mal Strg+Rechtsklick.

Code

(function() {
	addEventListener('click', function(event) {
		if (event.button == 2 && event.ctrlKey) {
			event.preventDefault();
			event.stopPropagation();
			openDialog('chrome://inspector/content/', '_blank', 'chrome, all, dialog=no', event.originalTarget);
		};
	}, true);
})();

Nachtrag:
Ich sehe gerade, in der Skriptsammlung gibt es bereits ein Skript mit genau der gleichen Funktion:
Inspect Element

Speravir

Zitat von aborix

Eine Tastenkombination mit Linksklick ist insofern schlecht, als die beim jeweiligen Element übliche Funktion des Linksklicks sich nicht immer verhindern lässt. Das ist z.B. bei den Einträgen im Datei-Menü so.
Auch InspectorWidget hat diese Schwachstelle.

Ach so.

Zitat

Da das Skript jetzt nicht mehr zu unterscheiden braucht, ob man in einem Popupmenü oder anderswo klickt, konnte ich es vereinfachen.

~~(Bin noch nicht zum Testen gekommen.)~~

Siehe Update am Ende.

Zitat

Nachtrag:
Ich sehe gerade, in der Skriptsammlung gibt es bereits ein Skript mit genau der gleichen Funktion:
Inspect Element

Boah, da habe ich den Wald vor lauter Bäumen (sprich Skripten mit sehr ähnlicher Funktion) nicht mehr gesehen *). InspectElementModY ist ja, wie der Name sagt, eine modifizierte, weiterentwickelte Version von InspectElement. Es müsste also in einem vom beiden Skripten die Tastenkombi geändert werden.

*) Oder hier eher: die einzelnen Bäume vor lauter Wald.

Apropos: Hast Du zufällig eine Idee zu meinem Punkt 1 in Re: userChrome.js Scripte für den Fuchs (Diskussion), Beitrag Nr. 33?

Update:
Ich habe den Code in eine neue Datei hineinkopiert, diese ClickInspect genannt, da ja nix mehr mit Popup ist und …

funktioniert ebenfalls prima!

Edit: Tippfehler korrigiert.