Bug 1579767: Reinstate the accessibility group role for the search one-offs container. r=dao a=lizzard
authorJames Teh <jteh@mozilla.com>
Mon, 09 Sep 2019 16:39:34 +0000
changeset 555058 a86e16a48e8c10e8dcada34e95c8ec8d7e2b7b8e
parent 555057 12c12e936df158a45fb317e6d4d9be29d59f9c7c
child 555059 938228e585ebd97f96f5134c09e53bb81224964c
push id2165
push userffxbld-merge
push dateMon, 14 Oct 2019 16:30:58 +0000
treeherdermozilla-release@0eae18af659f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdao, lizzard
bugs1579767
milestone70.0
Bug 1579767: Reinstate the accessibility group role for the search one-offs container. r=dao a=lizzard Differential Revision: https://phabricator.services.mozilla.com/D45151
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
@@ -16,17 +16,17 @@ class SearchOneOffs {
     this.container = container;
 
     this.container.appendChild(
       MozXULElement.parseXULToFragment(
         `
       <hbox class="search-panel-one-offs-header search-panel-header search-panel-current-input">
         <label class="searchbar-oneoffheader-search" value="&searchWithDesc.label;"/>
       </hbox>
-      <hbox class="search-panel-one-offs"/>
+      <hbox class="search-panel-one-offs" role="group"/>
       <vbox class="search-add-engines"/>
       <hbox class="search-one-offs-spacer"/>
       <button class="searchbar-engine-one-off-item search-setting-button-compact" tooltiptext="&changeSearchSettings.tooltip;"/>
       <button class="search-setting-button" label="&changeSearchSettings.button;"/>
       <menupopup class="search-one-offs-context-menu">
         <menuitem class="search-one-offs-context-open-in-new-tab" label="&searchInNewTab.label;" accesskey="&searchInNewTab.accesskey;"/>
         <menuitem class="search-one-offs-context-set-default" label="&searchSetAsDefault.label;" accesskey="&searchSetAsDefault.accesskey;"/>
       </menupopup>