Beiträge von BrokenHeart

Zitat von Horstmann

Reden wir hier nur über Tooltips, oder was anderes?

Nur über Tooltips ...

... und die Auswirkungen, die Änderungen am Code unter bestimmten Bedingungen haben können, wenn man dieses Tooltip-Problem behebt.

Ich habe allerdings festgestellt, dass sich das Verhalten bei "überlangen Menü-Popups" nur dann zeigt, wenn sich dort mindestens ein Unterverzeichnis befindet. Daher ist das, was Mitleser vorgeschlagen hat, nämlich das Löschen der Zeile mit dem Eintrag itemPopup.hidden = true;, die momentan wohl beste Lösung. Man sollte dann aber auch itemPopup.hidden = false; (ein paar Zeilen darunter) ebenfalls löschen.

Frage mich immer noch, wie die ganzen Manipulationen an den Lesezeichen-Menüs, Auswirkungen auf die Tab-Previews haben können? Habe es durch-debuggt und es wird zu keinem Zeitpunkt tab-preview-panel angesprochen . Alles Hexenwerk 👹.

Zitat von mkpcxxl

Bis jetzt hatte ich kein solches Popup. Was meinst du genau mit überlangen Popups ?

Ein Untermenü, welches mehr Einträge hat, als du auf deinem Bildschirm darstellen kannst, in dem also gescrollt werden muss (kleine Pfeile oben und unten), um sich alle Einträge anzuschauen.

Zitat von Mitleser

, versuche mal Zeile 298

auszukommentieren...

Hatte ich auch schon getestet. Dann gibt es allerdings Probleme mit der Anzeige von überlangen Popus, in denen gescrollt werden muss, die sich dann beim nächsten Aufruf die letzte Position nicht mehr merken.

Zitat von Mira_Belle

Also ich mache mir schon etwas arbeit und lasse nicht einfach eine KI ein Skrpt erstellen.

Alles klar. War nicht meine Absicht deine Arbeit in Frage zu stellen oder zu diskreditieren. Die meisten Leute hier im Forum wollen einfach eine funktionierende Lösung für ihr Problem und das ist ja genau das, was du abgeliefert hast. Von diesem Punkt aus betrachtet, ist deine Herangehensweise dann natürlich auch in Ordnung.

Hatte nur meine persönlichen Befindlichkeiten zu diesem Thema zum Besten geben wollen...

Zitat von Mira_Belle

Nein, bitte nicht antworten, brauchen nicht schon wieder eine Grundsatzdiskussion!

Ich ziehe immer KI zurate und wer das nicht gut findet, soll einfach "meine" Skripte und Korrekturen meiden. Punkt.

Eine Grundsatzdiskussion will ich auch nicht, zumal das Ergebnis hier wirklich gut ist (Einstellung wird sogar persistent in den Prefs gemacht!). Viele Sachen, die hier als "neues" JS-Skript oder CSS-Code erstellt wurden, hatten meist auch schon einige "Vordenker". Copy&Paste ist auch ok, solange man bei umfangreicheren Code den eigentlichen Urheber erwähnt, vor allem dann, wenn man nicht wirklich den gesamten Code, den man gepostet hat, auch wirklich verstanden hat.

Wie gesagt: Ich habe kein Problem damit, aber ich habe für mich persönlich festgestellt, dass bei mir der Spaß an der ganzen Bastelei etwas zurückgegangen ist. Mir fehlt da irgendwie das Tüfteln und "Fachsimpeln" über den Code. Und außerdem ist es z.T. wirklich sinnlos bei so kleinen Aufgaben, sich noch die Mühe zu machen, sich da reinzufuchsen, weil bei solchen Dingen die KI oft schnellere und auch bessere Lösungen bietet. Bin ehrlich gesagt froh, dass ich in einer Zeit beruflich damit zu tun hatte, wo es dieses "Hilfsmittel" noch nicht gab ...

Zitat von Mira_Belle

Das Problem könnte längst keines mehr sein, wenn das CSS bereitgestellt werden würde!

Vielleicht hat er eine ausgeprägte Form des "Code-Shamings" und glaubt, dass er dann für alle Zeiten deswegen stigmatisiert ist?

Zitat von Webmark

Jesus He Knows Me (Genesis)

2002Andreas Hast du die Änderung aus #111 in deinem Skript übernommen?

args = args.split(" ").map(a => a.replace("$1", openURL));  //Split bei Leerzeichen 

Ich glaube, man sollte das bestehende OpenWith-Skript aber so umschreiben, dass man die Parameter in einzelnen Strings übergeben kann. Dann hat man das ganze Problem mit dem "Aufdröseln" mittels split() nicht mehr...

Zitat von lenny2

Das Kontextmenü funktioniert also, aber ein Klick darauf erstellt ein neues Profil in %AppData%.

Das dürfte an dem Leerzeichen bei "User Data" liegen. split(" ") (in Zeile 124 bei #111) trennt ja einfach alle Übergabeargumente, wenn er ein Leerzeichen findet. Ob es reicht \"User\ Data\" zu schreiben, weiß ich nicht. Vielleicht müsste man den split-Parameter anpassen?! Ein Komma zwischen den Übergabeargumenten würde sich anbieten, also so split(","). Dann müssten aber alle bisherigen Aufrufe entsprechend auch angepasst werden...

Zitat von lenny2

"Ungoogled Chromium": "D:\\Ungoogled Chromium Portable\\chrome.exe<>$1 --user-data-dir="User Data"

Zitat von lenny2

, wird das Skript nicht mehr im Kontextmenü angezeigt.

Wahrscheinlich weil das Skript schon abgestürzt ist, nachdem dieser syntaktisch falsche String eingefügt wurde. Du hast 3(!) Anführungszeichen. Das schließende Anführungszeichen hast du vergessen.

Außerdem müssen die beiden Anführungszeichen um "User Data" 'escaped' werden, also sieht der korrekte String dann so aus:

"Ungoogled Chromium":    "D:\\Ungoogled Chromium Portable\\chrome.exe<>$1 --user-data-dir=\"User Data\""

Zitat von 2002Andreas

Egal wie ich das hier eintrage:

Ich kann es nicht genau erkennen, aber es sieht in deinem Screenshot so aus, als wenn sich keine Leerzeichen zwischen den Argumenten befinden. Diese sind aber zwingend notwendig.

Wie gesagt, ob es so beim Firefox klappt kann ich nicht sagen, aber das Skript läuft jetzt einwandfrei. Ich habe noch etliche Parameter übergeben und alle werden im Chrome-Browser richtig interpretiert und übernommen. Wenn Parameter beim Firefox nicht laufen, dann müsste man speziell für den Firefox Extra-Anpassungen einbauen, was ich jetzt aber auch nicht vorhabe. Aber du sagst ja, dass du für deinen Aufruf schon eine Alternativlösung hast...

Edit: Ok, Sören war schneller

Mira_Belle Danke für deine Antwort.

Ich habe schon verstanden, dass du diese "Anpassung" gar nicht (mehr) brauchst. Bloß, wenn jemand sagt, dass etwas bei ihm/ihr nicht funktioniert - wie auch noch in deinem letzten Posting - dann hake ich natürlich nach, da es hier einwandfrei funktioniert.

Ich hoffe mal, du hast wirklich das Skript aus #111 benutzt bzw. die Zeile in deinem ursprünglichen(!) Skript geändert und nicht den "Workaround" aus #108. Der funktionierte nämlich wirklich nicht, deswegen hatte ich den Code ja auch zwischenzeitlich gelöscht.

Ich kann hier mit meinen Firefox-Installationen nicht vernünftig testen, weil ich schon so viele Sachen (u.a. profile.ini, install.ini) angepasst habe, dass das Ergebnis nicht aussagekräftig wäre und ich auch schon mein Profil damit zerschossen hatte. Deshalb teste ich mit Chrome und da funktioniert die kleine Anpassung. Und wenn die Parameterübergabe in Chrome funktioniert, sollte es natürlich auch mit einer Firefox-Installation klappen.

Zitat von 2002Andreas

Lässt es sich einstellen/eintragen, dass man ein bestimmtes Profil damit dann aufruft?

Prinzipiell schon. Ich konnte es nur bedingt testen, aber die einzelnen Aufrufparameter werden zumindest korrekt als Array extrahiert und an die Process-Funktion weitergegeben. Nur bin ich mir nicht sicher, ob der Aufruf mit der Seite als erstem Parameter in diesem Fall so erlaubt ist...

Weiter bin ich nicht gekommen, weil sich dann, wegen meiner merkwürdigen Firefox-Installationen, kein zusätzlicher Firefox bei mir öffnen lässt. Werde irgendwann mal drei nicht-portable Standard-Installationen erstellen, damit ich eben solche Sachen in Zukunft auch testen kann.

Du musst aber beim Übergabestring die inneren Anführungszeichen um den Profilnamen 'escapen' -> \", also so:

// ...
"Firefox Clean":    "C:\Program Files\Mozilla Firefox\firefox.exe<>$1 -P \"Clean\" ",
//...

Zitat von Mira_Belle

Oweh!

Jetzt bin aber erst recht raus!

?

Ok, jetzt weiß ich immer noch nicht, was bei dir nicht funktioniert, aber egal. Ja, warten wir ab, ob es Lenny hilft, oder nicht...

Zitat von Mira_Belle

Aber leider auch nichts.

Was funktioniert denn nicht ?

Wie gesagt, beim Firefox ist entscheidend, welche Einstellungen für das Öffnen von Links gesetzt sind (siehe oben).

Außerdem gibt es auch noch diese Flags in 'about:config':

user_pref("browser.link.open_newwindow",3); 
user_pref("browser.link.open_newwindow.restriction",0); 
user_pref("browser.link.open_newwindow.override.external",3); 

Das kann das Verhalten ebenfalls beeinflussen...

Habe es jetzt nur mit meinem unverbastelten Google Chrome getestet, da bei meinen portablen Feuerfüchschen zu viele Einstellungen geändert wurden.

Bitte nochmal testen (geändert hat sich nur Zeile 124):

// ==UserScript==
// @name			OpenWith
// @description		Fügt dem Kontextmenü ein Menü hinzu, zum Öffnen der aktuelle Seite und eines ausgewählten Links in einem anderen Browser.
// @description		und eines ausgewählten Links in einem anderen Browser.
// @version			1.2.1a  63+
// @author			y2k
// @include			main
// @charset			UTF-8
// @namespace		http://tabunfirefox.web.fc2.com/
// @note			Anwendungssymbol anzeigen
// @note			als .uc.js Script umgeschrieben
// ==/UserScript==
(function() {
"use strict";

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

/*
	Vor Verwendung, Pfad auf eigene Umgebung ändern(\ wird durch \\ ersetzt)
	Zum Übergeben von Argumenten, wie folgt vorgehen:
	C:\\Program Files\\Internet Explorer\\iexplore.exe<>$1 Argument Argument
	※ $1 wird in URL umgewandelt
*/
const BrowserPath = {
	"Google Chrome":	"C:\\Programme\\Google\\Chrome\\Application\\chrome.exe<>$1 -new-window",
	"Microsoft Edge":	"C:\\Program Files (x86)\\Microsoft\\Edge\\Application\\msedge.exe"
};

const FlatMenu = false;

const OpenWith = {
	start: function() {
		const cm = document.getElementById("contentAreaContextMenu");
		cm.addEventListener("popupshowing", function(e) {
			if (e.target == this) { OpenWith.onpopup(e); }
		}, false);
	},

	createMenu: function() {
		if (this.pageItem || this.linkItem) {
			return;
		}

		const contextMenu = document.getElementById("contentAreaContextMenu");
		const pageMenu = this.$C("menu", { id: "context-open-with-page", label: "Seite öffnen mit:" });
		contextMenu.insertBefore(pageMenu, contextMenu.querySelector(":scope > #context-bookmarkpage, :scope > #context-savepage"));

		const linkMenu = this.$C("menu", { id: "context-open-with-link", label: "Link öffnen mit:" });
		contextMenu.insertBefore(linkMenu, contextMenu.querySelector(":scope > #context-sep-open"));

		this.pageItem = this.createMenuItem(pageMenu, "openPage", FlatMenu? "Seite öffnen mit $1 ":"  $1");
		this.linkItem = this.createMenuItem(linkMenu, "openLink", FlatMenu? "Link öffnen mit $1 ":"  $1");
	},
	
	createMenuItem: function(menu, method, format) {
		const frag = document.createDocumentFragment();
		
		let menuitem = [];
		for (let i of Object.keys(BrowserPath)) {
			const item = this.$C("menuitem", {
				label:	format.replace("$1", i),
				class:	"menuitem-iconic",
				image:	"moz-icon:file:///" + BrowserPath[i].split("<>")[0] + "?size=16",
				value:	JSON.stringify([ method, i ]),
			});
			item.addEventListener("command", this, false);

			frag.appendChild(item);
			menuitem[menuitem.length] = item;
		}
		
		if (!FlatMenu) {
			const menupopup = this.$C("menupopup");
			menupopup.appendChild(frag);
			menu.appendChild(menupopup);
			menuitem = [ menu ];
		}
		else {
			const parent = menu.parentNode;
			parent.insertBefore(frag, menu);
			parent.removeChild(menu);
		}
		
		return menuitem;
	},

	$C: function(tag, attrs) {
		const elem = document.createXULElement(tag);
		if (attrs) {
			for (let key of Object.keys(attrs))
				elem.setAttribute(key, attrs[key]);
		}
		return elem;
	},

	onpopup: function(e) {
		this.createMenu();
		
		const isHtml = /^(https?|file):/.test(gBrowser.currentURI.spec);
		const pageItemHidden = !isHtml || gContextMenu.onLink || gContextMenu.onTextInput;
		const linkItemHidden = !isHtml || !gContextMenu.onLink || gContextMenu.onTextInput;
		
		const pageItem = this.pageItem;
		for (let i = 0, l = pageItem.length; i < l; i++) {
			pageItem[i].hidden = pageItemHidden;
		}
		const linkItem = this.linkItem;
		for (let i = 0, l = linkItem.length; i < l; i++) {
			linkItem[i].hidden = linkItemHidden;
		}
	},

	handleEvent: function(event) {
		if (event.type === "command") {
			const [ method, key ] = JSON.parse(event.originalTarget.getAttribute("value"));
			const url = method === "openPage"? gBrowser.currentURI.spec: gContextMenu.linkURL;
			this.launch(BrowserPath[key], url);
		}
	},
	
	launch: function(browserPath, openURL) {
		let [ path, args ] = browserPath.split("<>");
		if (args) {
			args = args.split(" ").map(a => a.replace("$1", openURL));  //Split bei Leerzeichen 
		} else {
			args = [ openURL ];
		}
				
		const file = Cc['@mozilla.org/file/local;1'].createInstance(Ci.nsIFile);
		file.initWithPath(path);
		
		const process = Cc['@mozilla.org/process/util;1'].createInstance(Ci.nsIProcess);
		process.init(file);
		process.run(false, args, args.length, {});
	},
};

OpenWith.start();

})();

Die Fehlermeldungen in der Konsole rührten von einem Problem mit den Leerzeichen in meinem Pfad. Dann muss die Angabe des Aufrufs im Skript nämlich so aussehen: "D:\\Programme (Portable)\\Firefox Portable\\FireFox Release\\FirefoxLoader.exe"+"<>$1 -new-window".

Aber die Übergabe von Argumenten (zusätzlich zur URL) scheint in diesem Skript überhaupt nicht richtig zu funktionieren. In Chrome exakt das gleiche Ergebnis. Dort wird -new-window ebenso ignoriert. In der CMD funktioniert bei beiden Browsern der identische Aufruf einwandfrei.

Ja, der Workaround funktioniert auch nicht...

Neben der grundsätzlichen Anpassung des Skripts, sehe ich beim Firefox die einzige Möglichkeit in der Änderung der Voreinstellung(?) in den Einstellungen und zwar diese hier:

Wenn man den Pfad, wie in der Beschreibung im Skript dargestellt, korrekt aufruft, dann wird der Aufruf der Webseite auch richtig erzeugt und funktioniert auch über die Eingabeaufforderung.

Allerdings werden beim eigentlichen Aufruf im Skript diese Fehler erzeugt:

19:23:20.686 NS_ERROR_ILLEGAL_VALUE: Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsICommandLine.handleFlagWithParam]
    bch_handle resource:///modules/BrowserContentHandler.sys.mjs:474
BrowserContentHandler.sys.mjs:483:15
    bch_handle resource:///modules/BrowserContentHandler.sys.mjs:483
	
19:23:20.687 Warning: unrecognized command line flag -new-window BrowserContentHandler.sys.mjs:1574:17
    dch_handle resource:///modules/BrowserContentHandler.sys.mjs:1574	

Edit: Code gelöscht. Siehe unten...

Zitat von lenny2

Wenn es darum geht, 1$ hinzuzufügen, hilft das auch nicht.

Die Frage war ja, warum der Link nicht übergeben wird. Wenn zusätzliche Argumente übergeben werden, muss bei diesem Skript zwingend die "$1" als erster Parameter übergeben werden, sonst funktioniert die URL-Ersetzung der aktuellen Seite eben nicht.

Dass dann immer noch nicht das von dir gewünschte Ergebnis (Seite in neuem Fenster öffnen) kommt, ist wieder eine andere Geschichte. Wenn es in der Windows-Eingabeaufforderung funktioniert, dann sollte es eigentlich auch über dieses Skript funktionieren. Müsste man sich halt mal den fertigen Aufrufstring in der Firefox-Konsole ausgeben oder einen Debugger bemühen.

Mira_Belle Du musst dich schon an die exakte Syntax halten, so wie es in der Beschreibung angegeben wurde. aslo <>$1 direkt nach dem Pfad und dann die Argumente. Wenn es dann nicht funktioniert -> siehe was ich Lenny oben geschrieben habe.

Zitat von Mira_Belle

Warum wird der Link nicht mit übergeben?

Steht doch direkt in der Beschreibung :

Zitat

Code

C:\\Program Files\\Internet Explorer\\iexplore.exe<>$1 Argument Argument
	※ $1 wird in URL umgewandelt

Zitat von Sören Hentzschel

Das Öffnen, Wechseln sowie Schließen von Tabs funktioniert bei mir alles ohne irgendwelche Probleme, sogar einhändig.

Bedingt durch die geringere Größe auf dem Smartphone werden keine Tabs auf der Firefox-UI dargestellt, sondern nur ein Tab-Button mit der Anzahl der offenen Tabs. In der Dropdown-Liste werden dann gerade mal 6½ Tabs gleichzeitig ohne Scrollen angezeigt. Da ich auf dem Desktop normalerweise viel mehr gleichzeitig offene Tabs habe und diese auch mit Namen auf der Oberfläche dargestellt bekomme, ist das ein ganz anderes Arbeiten als auf dem Smartphone. Am Desktop kann ich mit der Maus z.B. sehr einfach mehrere Tabs selektieren und diese verschieben, schließen, duplizieren oder als Lesezeichen per drag&drop anlegen. Das alles geht beim mobilen Firefox nicht (oder nur sehr umständlich) und viele Sachen ergeben auf dem Smartphone auch nur wenig Sinn. Und auch das Wischen über die Adressleiste zum Tabwechsel ist ein Blindflug und ist nur sinnvoll bei sehr wenigen Tabs.

Wenn du mit den vorhandenen Funktionen zufrieden bist, dann ist das doch toll. Ich brauche jedenfalls am Smartphone keine Tabs in der aktuellen Form.

Zitat von Sören Hentzschel

Wieso das? Dafür gibt es doch den Startseiten-Button?

Und was soll das bringen? Solange die Chronik nicht wieder genau am Startpunkt des Aufrufs steht oder ich händisch den Tab geschlossen habe, wird beim nächsten Aufruf der selben URL-Verknüpfung wieder ein neuer Tab erzeugt. Also genau das, was ich bei einer Verknüpfung nicht haben wollte. Und auch egal ob ich den Firefox Home-Button oder den Android Startseiten-Button nutze.

Darum geht es doch bei der aktuellen Version: Es gibt keinen "selektierten" Tab, den ich für einen Aufruf über eine Verknüpfung wiederverwenden könnte und da sich die Verknüpfungen alle außerhalb eines Tabs befinden, wird eben jedes mal ein neuer Tab erzeugt. Da auch kein Tab wieder geschlossen wird, wenn ich Firefox normal "beende" (Zurück-Button), muss ich alle Tabs wieder händisch schließen.

Zitat von Sören Hentzschel

Scherzfrage? Der übliche Weg, den die allermeisten Nutzer gehen, um Websites in einem Browser aufzurufen, ist die Adressle

Ich würde mir bei dir NIEMALS einen "Scherz" erlauben. Meine Frage war absolut ernst gemeint! Die gleiche Frage hatte ich auch AngelOfDarkness in einem anderen Thread gestellt und ich hatte eine umfassende Antwort darauf bekommen. Daher auch meine Frage, ob du alle regelmäßig besuchten Tabs immer offen hältst, die du mir leider nicht beantwortest hast. Da du immer alles über die Adressleiste öffnest, wohl eher nicht...

Dass die "allermeisten" Nutzer auf Smartphones ihre Seiten immer über die Adressleiste aufrufen, kann ich aus meiner Erfahrung nicht bestätigen, da ich aber keine Zahlen dazu habe, schreibe ich mal nichts dazu.