Bug 1515939 - Fix styling of the submenu used to add new search engines. r=dao, a=RyanVM
authorPaolo Amadini <paolo.mozmail@amadzone.org>
Fri, 21 Dec 2018 14:36:34 +0000
changeset 509200 e453e25d62cb66588c5370f8a95965220c241582
parent 509199 1d747c8ca942c7594f682259a3a59f3995ea6d1d
child 509201 39073b9505b81c8edc41f469cd51b8b3375d18fe
push id1905
push userffxbld-merge
push dateMon, 21 Jan 2019 12:33:13 +0000
treeherdermozilla-release@c2fca1944d8c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdao, RyanVM
bugs1515939
milestone65.0
Bug 1515939 - Fix styling of the submenu used to add new search engines. r=dao, a=RyanVM Differential Revision: https://phabricator.services.mozilla.com/D15198
browser/components/search/content/search-one-offs.js
--- a/browser/components/search/content/search-one-offs.js
+++ b/browser/components/search/content/search-one-offs.js
@@ -553,18 +553,18 @@ class SearchOneOffs {
 
     let engines = gBrowser.selectedBrowser.engines;
     let tooManyEngines = engines.length > this._addEngineMenuThreshold;
 
     if (tooManyEngines) {
       // Make the top-level menu button.
       let button = document.createXULElement("toolbarbutton");
       list.appendChild(button);
-      button.classList.add("addengine-item", "badged-button");
-      button.setAttribute("class", "addengine-menu-button");
+      button.classList.add("addengine-menu-button", "addengine-item",
+                           "badged-button");
       button.setAttribute("type", "menu");
       button.setAttribute("label",
         this.bundle.GetStringFromName("cmd_addFoundEngineMenu"));
       button.setAttribute("crop", "end");
       button.setAttribute("pack", "start");
 
       // Set the menu button's image to the image of the first engine.  The
       // offered engines may have differing images, so there's no perfect