DownloadStatusModoki.uc.js umbasteln

BrokenHeart :

Ich hab mir im oberen Bereich mit diesem Code schon eine Leiste eingerichtet:

//  Statusleiste für Suchen, Buttons etc....
//  https://www.camp-firefox.de/forum/viewtopic.php?f=16&t=122105&hilit=statusbar&start=15#p1052231
//  und folgende
//  ab FF68 nicht .XUL sondern .xhtml und weiteres
//  https://www.camp-firefox.de/forum/thema/128155-anpassungen-der-css-codes-ab-firefox-68-bzw-69/?postID=1116235#post1116235
//  https://www.camp-firefox.de/forum/thema/128155-anpassungen-der-css-codes-ab-firefox-68-bzw-69/?postID=1116240#post1116240
//


(function() {

  if (location != AppConstants.BROWSER_CHROME_URL)
  return;
    
  var tb = document.createXULElement('toolbar');
  tb.id = 'fp-statusbar-2';
  tb.setAttribute('customizable', true);
  tb.setAttribute('mode', 'icons');

  var parentE = document.getElementById('PersonalToolbar').parentNode;
  var childE = document.getElementById('PersonalToolbar');
  parentE.insertBefore( tb , childE );

  CustomizableUI.registerArea( 'fp-statusbar-2' , { legacy: true } );

  CustomizableUI.registerArea('new-toolbar', {legacy: true});
  
  CustomizableUI.registerToolbarNode(tb);

})();

Kommt die Deinen Codes in die Quere?

Ich seh unten zwar was eingerichtet, kann aber bei nem Download nix erkennen.

Und warum hast du den Code zweimal angegeben?

Eingabefehler :blushing:

Ich lösch den alten teil mal raus....

Zitat von harff182

BrokenHeart :

Ich hab mir im oberen Bereich mit diesem Code schon eine Leiste eingerichtet:

Kommt die Deinen Codes in die Quere?

Ich seh unten zwar was eingerichtet, kann aber bei nem Download nix erkennen.

Ich denke mal, dass du beim Kopieren hier in das Forum zweimal die gleiche Funktion in das Code-Fenster eingestellt hast, ansonsten wäre das sonst ein etwas merkwürdiger Statusleisten-Code.

Ersetzte bitte, wenn auch nur testweise, deine Statusleiste, durch die für die Statusbar angepasste Version von Aborix. Benenne die Datei mit deinem alten Code um und schreibe den oberen Code in eine neue Datei in deinem Chrome-Verzeichnis. Wenn es dann funktioniert dann weiß man das es daran liegt.

So wie ich das sehe ist in deinem Statusleistenskript auch keine Style-Information vorhanden, dann würde ich sowieso die von Aborix nehmen.

Edit: Sehe gerade, dass das ja gar keine Statusleiste, sondern ein Extraleiste sein soll: Dann stimmt wahrscheinlich der Abstand nicht mehr und die margin-top muss angepasst werden. Hast du jetzt beide Leisten? Mach doch bitte mal einen Screenshot...

Zitat von BrokenHeart

Sehe gerade, dass das ja gar keine Statusleiste, sondern ein Extraleiste sein soll

Das ist eine Extraleiste. Derzeit wird wohl alles, was irgendwie eine zusätzliche Symbolleiste ist, als Statusleiste bezeichnet. Es gibt noch ein Skript, das wirklich die alte Statusleiste (mit URL-Status-Anzeige) wieder zur Verfügung stellt. Das liegt aber auch am Skript oben, da wird ja auch von statusbar gesprochen.

Zitat von BrokenHeart

So wie ich das sehe ist in deinem Statusleistenskript auch keine Style-Information vorhanden

Bei mir hat sie die Akzentfarbe von Win 10, die ist bei mir Dunkelblau.

Zitat von BrokenHeart

Edit: Sehe gerade, dass das ja gar keine Statusleiste, sondern ein Extraleiste sein soll: Dann stimmt wahrscheinlich der Abstand nicht mehr und die margin-top muss angepasst werden. Hast du jetzt beide Leisten? Mach doch bitte mal einen Screenshot...

Hmpf... momentan seh ich weder meine Leiste, noch das, was mit Deinen Scripts und dem Css aufgetaucht war.

Ich versuchs notfalls mal mitm Reboot, kann dann 'n Moment dauern...

milupo : Ja, ich weiß was du meinst. Aber eigentlich ist es egal wie man die zusätzlich Leiste nennt. 'Statusbar' nenne ich persönlich jede Leiste, auf der mir bei einem Browser mindestens eine URL-Vorschau und Verbindungsinformationen angezeigt werden.

harff182 : Also Reboot halte ich für übertrieben. Lass dir doch Zeit...eilt doch nicht.

Ach ja, ganz wichtig: Börsenfeger wollte das ganze für die aktuelle Nightly. Also habe ich auch nur damit getestet!

In dem DownloadStatusModoki-Skript steht auch // @compatibility Firefox 73+.

Also, wenn du keine Nightly/Beta benutzt wird es sehr wahrscheinlich auch nicht funktionieren.

Dann musst du noch 2 1/2 Wochen warten, dann kommt die FF73 raus.

Zitat von BrokenHeart

'Statusbar' nenne ich persönlich jede Leiste, auf der mir bei einem Browser mindestens eine URL-Vorschau und Verbindungsinformationen angezeigt werden.

Das ist ja auch richtig. Aber mittlerweile nennen viele „stinknormale“ durcch ein Skript erzeugte Symbolleisten so.

Übrigens mit der Akzentfarbe und statusbar meinte ich die obere zusätzliche Leiste, auf die sich harff182 bezieht. Die untere Leiste hat ja wenigstens vbox.style.backgroundColor als Gestaltungsmöglichkeit.

Habe deine Extraleiste über der Personal-Toolbar(!) jetzt auch bei mir in der Nightly getestet. Und die Downloadbar wird ohne Fehler angezeigt und funktioniert. An deiner Extraleiste liegt es also nicht. Hätte mich auch sehr gewundert, an dieser Position. Dann ist der Grund, wie oben schon geschrieben, dass du FF72 benutzt.

Alles klar, Vorfreude ist die schönste Freude und meinereiner kann warten

Ich meld mich dann Mitte Februar wieder...

Guuts nächtle...

harff182 : Das ist die richtige Einstellung.

Ebenfalls gute Nacht...

harff182 Stelle mal ein Bildschirmfoto ein. Es gibt auch Leisten, die werden nur angezeigt, wenn sie Inhalt haben, die Lesezeichenleiste zum Beispiel.

seis drum....

Zitat von harff182

seis drum....

Das ist aber nur die Downloadleiste. Dort müsste dein Download-Status erscheinen, wenn du etwas herunterlädst. Gedacht ist alles so: Boersenfeger wollte, dass die Downloadleiste und untere Statusleiste/Symbolleiste, die du vielleicht ursprünglich gar nicht hast, auf einer Leiste erscheint. Außerdem soll alles rechts in der Leiste angezeigt werden. Du brauchst dazu die zwei Skripte und die CSS-Datei aus Beitrag #40 von BrokenHeart.

Übrigens, ich habe ja noch die "echte Statusleiste". Dachte ich. Diese Leiste ist jetzt bis auf ein einziges Symbol leer, das ich zum Test dorthin geschoben habe. Sie hat die Reihenfolge mit der unteren Symbolleiste (gelb) getauscht, die jetzt darunter ist und vorher über der blauen Leiste war. Die URL-Anzeige ist jetzt ebenfalls in der gelben Leiste und nicht mehr in der blauen Leiste. Außerdem wirkt die Textfarbe aus dem Skript, das die blaue Leiste erzeugt, auf den Text in der gelben Leiste.

Das Skript für die blaue Leiste heißt RevertAddonBarStatusBar.uc.js.

Anscheinend benutzt du jetzt zwei Skripte für die Statusleiste?! Eigentlich ist das ganze nur für eine Statusbar gedacht

Du solltest vielleicht dann mal schauen, ob die gleichen IDs verwendet werden. Ansonsten kann ich nur was dazu sagen, wenn du den Code für 'RevertAddonBarStatusBar.uc.js' hier mal reinstellst...

Zitat von BrokenHeart

wenn du den Code für 'RevertAddonBarStatusBar.uc.js' hier mal reinstellst...

Hier ist er:

 //Original code is from 
//https://github.com/Endor8/userChrome.js/blob/master/Firefox-57/RevertAddonBarStatusBar.uc.js
//All credit goes to Endor8

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

    let toolbar = document.createXULElement("toolbar");
    toolbar.id = "bottom-toolbar";
    toolbar.setAttribute("customizable", "true");
    toolbar.setAttribute("mode", "icons");
    toolbar.setAttribute("context", "toolbar-context-menu");

    let vbox = document.createXULElement("vbox");
    vbox.id = "bottom-toolbar-vbox";
    let browserBottombox = document.getElementById("browser-bottombox");
    browserBottombox.parentNode.insertBefore(vbox, browserBottombox);
    vbox.appendChild(toolbar);
    CustomizableUI.registerArea("bottom-toolbar", {legacy: true});
     CustomizableUI.registerToolbarNode(toolbar);

//STYLING SECTION
//You can put the css tweaks here inside your userChrome.css
//then remove everthing leftof inside the section
    
    var css = `
:root {
--bottom-toolbar-text-color: blue;
--bottom-toolbar-background-color: #0000DE;
--bottom-toolbar-height: 24px; /* The value should be between 22px - 34px if you ever want to change it */
}

#bottom-toolbar-vbox {
background-color: var(--bottom-toolbar-background-color);
}

#bottom-toolbar toolbarbutton {
-moz-appearance: toolbarbutton !important;
--toolbarbutton-inner-padding: 2px !important; /* Try increase this if you ever change the toolbar height 2px - 10px */
margin: 0 2px !important; /* The space between each toolbarbutton */
}

/* If you can harly see the icon due to its color,
try this */
/* #bottom-toolbar toolbarbutton {
filter: invert(65%) !important;
}
*/
/*---*/

#bottom-toolbar {
-moz-appearance: toolbar !important;
height: var(--bottom-toolbar-height);
/* Makes the buttons auto stick to the right side */
direction: rtl;
/*--*/
}

.browserContainer > #statuspanel {
background-color: transparent !important;
left: 5px !important;
bottom: 2px !important; /* Change this if you find the text too high or too low */
border: none !important;
max-width: 50% !important; /* This is the max length of the status text on the toolbar */
transition: none !important;
}

.browserContainer > #statuspanel > #statuspanel-inner > #statuspanel-label {
background-color: transparent !important;
margin-left: 0 !important;
border: none !important;
padding: 0 !important;
color: var(--bottom-toolbar-text-color) !important;
}

#statuspanel[inactive] {
opacity: 1 !important;
}

#statuspanel[inactive] #statuspanel-inner::before {
content: "Dokónčeny" !important;
background-color:transparent !important;
color: var(--bottom-toolbar-text-color) !important;
}

#statuspanel[inactive] #statuspanel-label {
opacity: 0 !important;
}

#main-window[inFullscreen="true"] #statuspanel {
display:none !important;
}`;
    var sss = Cc["@mozilla.org/content/style-sheet-service;1"].getService(Ci.nsIStyleSheetService);
    var uri = makeURI("data:text/css;charset=UTF=8," + encodeURIComponent(css));
    sss.loadAndRegisterSheet(uri, sss.AGENT_SHEET);

//END OF STYLING SECTION

})();

Das müsstest du auch schon von mir haben, wir hatten uns, glaube ich, vergangenes Jahr mal darüber ausgetauscht.

Außerdem kann ich bestätigen, dass die Downloadleiste im Gegensatz zu Fx 74 in Fx 72 leer bleibt.

Zitat

Das müsstest du auch schon von mir haben, wir hatten uns, glaube ich, vergangenes Jahr mal darüber ausgetauscht.

Ja stimmt, hatte ich sogar noch in einer Test-Version installiert..

Die IDs überschneiden sich bei beiden Skripten nicht und bei mir funktioniert es auch wie es soll. Das einzige, was durch die zweite Leiste nicht mehr ganz passt, ist die Position des 'Statuspanels'( also die Schrift links für die URL-Anzeige). Die muss dann eben wieder weiter nach oben. Das hatte ich aber schon im ersten Posting geschrieben...

Aber du hast mich auf die Idee gebracht, wirklich ALLE Style-Einstellungen in das Statusleisten-Skript zu schreiben. So dass wirklich nichts mehr außer diesen drei Dateien benötigt wird. Damit es mehr 'Out-Of-The-Box' funktioniert...

Aber wie gesagt, zwei Statusleisten führen ja den Sinn der eigentlichen Aktion ad absurdum, nämlich anstatt 2 Leisten, alles in einer Leiste zu vereinen.

So jetzt geht's in Bett.

Gute Nacht!

Zitat von BrokenHeart

Aber wie gesagt, zwei Statusleisten führen ja den Sinn der eigentlichen Aktion ad absurdum, nämlich anstatt 2 Leisten, alles in einer Leiste zu vereinen.

Das ist ja auch mein Spezialfall, ich hatte ja auch nicht die Absicht, Downloadleiste und Symbolleiste/Statusleiste zusammenzulegen. das war ja Boersenfegers Wunsch.

Ich habe den Fehler für die leere Downloadleiste gefunden. Ganz einfach: In Zeile 92 des Skriptes DownloadsStatusModoki.uc.jsmuss in Fx 72 noch die Endung .xul in (chrome://browser/content/downloads/contentAreaDownloadsView.xul) stehen. Erst ab Fx 73 gilt .xhtml.

harff182 Ersetze bitte in Zeile 92 des Skriptes DownloadsStatusModoki.uc.js xhtml durch xul.

Zitat von milupo

Ich habe den Fehler für die leere Downloadleiste gefunden. Ganz einfach: In Zeile 92 des Skriptes DownloadsStatusModoki.uc.jsmuss in Fx 72 noch die Endung .xul in (chrome://browser/content/downloads/contentAreaDownloadsView.xul) stehen. Erst ab Fx 73 gilt .xhtml.

Super, vielen Dank

Ich hatte es bis jetzt überhaupt nur in der Nightly Version getestet. Dann streiche ich die Ergänzung in #40 wieder raus und verweise auf deinen Hinweis.