Suchleisten-Script arbeitet nicht mehr im Nightly

Boersenfeger

Mein Suchleisten-Script arbeitet nicht mehr im Nightly. Es wird kein neuer Tab geöffnet und die Suche wird auch nicht im alten Tab angestossen. Woran liegts?

JavaScript

/* Suchfeld automatisch leeren und in neuem Tab oeffnen */


(function() {
    var searchbar = document.getElementById("searchbar");
    searchbar.doSearch = function(aData, aWhere, aEngine, aParams, aOneOff) {
      let textBox = this._textbox;


      if (aData && !PrivateBrowsingUtils.isWindowPrivate(window) && this.FormHistory.enabled) {
        this.FormHistory.update({
            op: "bump",
            fieldname: textBox.getAttribute("autocompletesearchparam"),
            value: aData,
        }, {
            handleError(aError) {
              Cu.reportError("Saving search to form history failed: " + aError.message);
            },
        });
      }


      let engine = aEngine || this.currentEngine;
      let submission = engine.getSubmission(aData, null, "searchbar");
      let telemetrySearchDetails = this.telemetrySearchDetails;
      this.telemetrySearchDetails = null;
      if (telemetrySearchDetails && telemetrySearchDetails.index == -1) {
        telemetrySearchDetails = null;
      }


      const details = {
        isOneOff: aOneOff,
        isSuggestion: (!aOneOff && telemetrySearchDetails),
        selection: telemetrySearchDetails,
      };
      BrowserSearch.recordSearchInTelemetry(engine, "searchbar", details);


      let params = {
        postData: submission.postData,
      };
      if (aParams) {
        for (let key in aParams) {
          params[key] = aParams[key];
        }
      }
      openTrustedLinkIn(submission.uri.spec, "tab", params);


      this.value = '';
          
      this.currentEngine = this.engines ? this.engines[0] : this._engines[0];
    };


}());


setTimeout(function() {
   if (!window.BrowserSearch)
      return;
   var searchbar = BrowserSearch.searchBar;
   if (!searchbar)
      return;
   var textbox = searchbar.textbox;
   var tId;
   textbox.addEventListener('input', function() {
      clearTimeout(tId);
      tId = setTimeout(function() {
         textbox.value = '';
         document.getElementById('PopupSearchAutoComplete').hidePopup();
      }, 5000);
   });
}, 0);

Alles anzeigen

Vielen Dank für Ideen!

Boersenfeger

Upps, danke fürs Abtrennen!

2002Andreas

Zitat von Boersenfeger

Vielen Dank für Ideen!

Teste bitte mal nur mit diesem Teil aus dem Script.

JavaScript

setTimeout(function() {
   if (!window.BrowserSearch)
      return;
   var searchbar = BrowserSearch.searchBar;
   if (!searchbar)
      return;
   var textbox = searchbar.textbox;
   var tId;
   textbox.addEventListener('input', function() {
      clearTimeout(tId);
      tId = setTimeout(function() {
         textbox.value = '';
         document.getElementById('PopupSearchAutoComplete').hidePopup();
      }, 5000);
   });
}, 0);

Alles anzeigen

geldhuegel

Ich kann zwar nicht zur Lösung beitragen, aber mitteilen, dass ich das von Börsenfeger eingestellte Script auch benütze: es funktioniert bei mir wie gewohnt.

BrokenHeart

Zitat von geldhuegel

Ich kann zwar nicht zur Lösung beitragen, aber mitteilen, dass ich das von Börsenfeger eingestellte Script auch benütze: es funktioniert bei mir wie gewohnt.

Ja, noch!

Boersenfeger schreibt ja von Problemen in der Nightly. In ca. 2 Monaten wird dann wohl auch die 'Standard'-Release betroffen sein.

Boersenfeger

Neben einem Timing-Problem ist ein Grund für das Versagen des Scripts, dass die Funktion recordSearchInTelemetry(engine, "searchbar", details); in 'BrowserSearch' (Zeile 37) in der Nightly nicht mehr existiert.

Ich befürchte, die Suche muss jetzt wohl ganz anders ausgeführt werden..., aber die Hoffnung ruht wohl auf den Fähigkeiten der bekannten Skript-Profis!

Edit: Habe jetzt einfach Zeile 37 auskommentiert ( keine Ahnung, ob das jetzt eine gute Idee ist? ). Aber jetzt läuft das Script wieder:

Bitte das hier mal GUT testen

Edit2: ...oder das Skript aus #3 von Andreas nehmen, was wohl die bessere Lösung darstellt.

Code

/* Suchfeld automatisch leeren und in neuem Tab oeffnen */

(function() {
    var searchbar = document.getElementById("searchbar");
    
    searchbar.doSearch = function(aData, aWhere, aEngine, aParams, aOneOff) {
    let textBox = this._textbox;


      if (aData && !PrivateBrowsingUtils.isWindowPrivate(window) && this.FormHistory.enabled) {
        this.FormHistory.update({
            op: "bump",
            fieldname: textBox.getAttribute("autocompletesearchparam"),
            value: aData,
        }, {
            handleError(aError) {
              Cu.reportError("Saving search to form history failed: " + aError.message);
            },
        });
      }


      let engine = aEngine || this.currentEngine;
      let submission = engine.getSubmission(aData, null, "searchbar");
      let telemetrySearchDetails = this.telemetrySearchDetails;
      this.telemetrySearchDetails = null;
      if (telemetrySearchDetails && telemetrySearchDetails.index == -1) {
        telemetrySearchDetails = null;
      }


      const details = {
        isOneOff: aOneOff,
        isSuggestion: (!aOneOff && telemetrySearchDetails),
        selection: telemetrySearchDetails,
      };
     // BrowserSearch.recordSearchInTelemetry(engine, "searchbar", details);


      let params = {
        postData: submission.postData,
      };
      if (aParams) {
        for (let key in aParams) {
          params[key] = aParams[key];
        }
      }
      openTrustedLinkIn(submission.uri.spec, "tab", params);


      this.value = '';
          
      this.currentEngine = this.engines ? this.engines[0] : this._engines[0];
    };

}());


setTimeout(function() {
   if (!window.BrowserSearch)
      return;
   var searchbar = BrowserSearch.searchBar;
   if (!searchbar)
      return;
   var textbox = searchbar.textbox;
   var tId;
   textbox.addEventListener('input', function() {
      clearTimeout(tId);
      tId = setTimeout(function() {
         textbox.value = '';
         document.getElementById('PopupSearchAutoComplete').hidePopup();
      }, 5000);
   });
}, 0);

Alles anzeigen

2002Andreas

Zitat von BrokenHeart

Bitte das hier mal GUT testen:

Ich bin mir nicht sicher ob ich etwas falsch verstanden habe

Das Script aus Beitrag Nr. 3 leert die Suche nach 5 Sekunden, oder wenn ich auf Enter/Pfeil klicke, und sich ein neuer Tab öffnet.

Getestet im aktuellen Nightly.

Was genau bewirken diese Zeilen aus dem Script?

JavaScript

(function() {
    var searchbar = document.getElementById("searchbar");
    searchbar.doSearch = function(aData, aWhere, aEngine, aParams, aOneOff) {
      let textBox = this._textbox;


      if (aData && !PrivateBrowsingUtils.isWindowPrivate(window) && this.FormHistory.enabled) {
        this.FormHistory.update({
            op: "bump",
            fieldname: textBox.getAttribute("autocompletesearchparam"),
            value: aData,
        }, {
            handleError(aError) {
              Cu.reportError("Saving search to form history failed: " + aError.message);
            },
        });
      }


      let engine = aEngine || this.currentEngine;
      let submission = engine.getSubmission(aData, null, "searchbar");
      let telemetrySearchDetails = this.telemetrySearchDetails;
      this.telemetrySearchDetails = null;
      if (telemetrySearchDetails && telemetrySearchDetails.index == -1) {
        telemetrySearchDetails = null;
      }


      const details = {
        isOneOff: aOneOff,
        isSuggestion: (!aOneOff && telemetrySearchDetails),
        selection: telemetrySearchDetails,
      };
      BrowserSearch.recordSearchInTelemetry(engine, "searchbar", details);


      let params = {
        postData: submission.postData,
      };
      if (aParams) {
        for (let key in aParams) {
          params[key] = aParams[key];
        }
      }
      openTrustedLinkIn(submission.uri.spec, "tab", params);


      this.value = '';
          
      this.currentEngine = this.engines ? this.engines[0] : this._engines[0];
    };


}());

Alles anzeigen

BrokenHeart

2002Andreas :

Wenn ich dein Skript benutze, wird die Suche immer im gleichen Tab und nicht in einem neuen Tab ausgeführt. Wenn das bei dir anders ist, dann ist das schon merkwürdig und verstehe ich auch nicht.

Meines Erachtens öffnet der obere Teil des Scripts unter anderem die Suche in einem neuen Tab:

openTrustedLinkIn(submission.uri.spec, "tab", params);

Aber was da genau im Detail passiert ist mir jetzt auch nicht 100% klar

2002Andreas

Zitat von BrokenHeart

nicht in einem neuen Tab ausgeführt.

Dafür nutze ich diese config Einstellung.

browser.search.openintab = true

Naja, evtl. kann uns ja der Fachmann aborix genaueres dazu sagen

BrokenHeart

Zitat von 2002Andreas

Dafür nutze ich diese config Einstellung.

browser.search.openintab = true

Du hast recht! Mit dieser Einstellung wird jetzt das Suchergebnis in einem neuen Tab dargestellt. Nach einem zugegebenermaßen kurzen Test sehe ich mit deinem Script aus #3 auch keine Probleme mehr und damit ist das wohl die beste Lösung.

Zitat von 2002Andreas

Naja, evtl. kann uns ja der Fachmann aborix genaueres dazu sagen

Ja, das sollte er, ich will nicht dumm sterben...

2002Andreas

Zitat von BrokenHeart

ich will nicht dumm sterben

Sehr gute Einstellung

Bin dann offline..pass auf dich auf..bis morgen...

BrokenHeart

Zitat von 2002Andreas

Bin dann offline..pass auf dich auf..bis morgen...

Wünsche dir eine gute Nacht...

Boersenfeger

Zitat von 2002Andreas

Teste bitte mal nur mit diesem Teil aus dem Script.

JavaScript

setTimeout(function() {
   if (!window.BrowserSearch)
      return;
   var searchbar = BrowserSearch.searchBar;
   if (!searchbar)
      return;
   var textbox = searchbar.textbox;
   var tId;
   textbox.addEventListener('input', function() {
      clearTimeout(tId);
      tId = setTimeout(function() {
         textbox.value = '';
         document.getElementById('PopupSearchAutoComplete').hidePopup();
      }, 5000);
   });
}, 0);

Alles anzeigen

Vielen Dank, 2002Andreas! Dieser Teil ist völlig ausreichend und die Config-Einstellung ist hier auch so.

Danke auch an die anderen Beteiligten für eure Ideen!

Kleine kosmetische Nachfrage:

In der Suchleiste ist nun noch das Wort Suchen präsent. Gleiches gilt auch für die Suchleiste in der Lesezeichen-Sidebar, hier heisst es Lesezeichen durchsuchen.. kann man beides oder auch nur eins davon mit CSS ausblenden?

2002Andreas

Zitat von Boersenfeger

kann man beides oder auch nur eins davon mit CSS ausblenden?

Teste bitte, das wäre für alle solche Einträge pauschal:

CSS

::placeholder {
    color: transparent !important;
}

Boersenfeger

Perfekt, vielen Dank!

2002Andreas

Freut mich wenn ich dir damit helfen konnte, war gern geschehen

Boersenfeger

Leider funktioniert es wieder nicht in Nightly

JavaScript

/* Suchfeld automatisch leeren */

setTimeout(function() {
   if (!window.BrowserSearch)
      return;
   var searchbar = BrowserSearch.searchBar;
   if (!searchbar)
      return;
   var textbox = searchbar.textbox;
   var tId;
   textbox.addEventListener('input', function() {
      clearTimeout(tId);
      tId = setTimeout(function() {
         textbox.value = '';
         document.getElementById('PopupSearchAutoComplete').hidePopup();
      }, 5000);
   });
}, 0);

Alles anzeigen

2002Andreas

Zitat von Boersenfeger

Leider funktioniert es wieder nicht in Nightly

Kein Problem hier damit.

Boersenfeger

Hat hier nur "geklemmt". Nach Neustart gings wieder... Danke fürs Prüfen