Bug 1470779: label History and Bookmarks sidebar containers for accessibility. r=mak

Without this, accessibility reports the chrome:// URL of the page element as the label, which is ugly for screen reader users in particular.

MozReview-Commit-ID: AIInSb79YcN

--HG--
extra : rebase_source : d39f2bc627eeb521122dbbcbdc4699af3942ca23
This commit is contained in:
James Teh 2018-06-27 10:53:03 +10:00
parent bdf8e4d393
commit 970e561de6
2 changed files with 8 additions and 2 deletions

View File

@ -12,13 +12,16 @@
%placesDTD;
<!ENTITY % editMenuDTD SYSTEM "chrome://global/locale/editMenuOverlay.dtd">
%editMenuDTD;
<!ENTITY % browserDTD SYSTEM "chrome://browser/locale/browser.dtd">
%browserDTD;
]>
<page id="bookmarksPanel"
xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
onload="init();"
onunload="PlacesUIUtils.setMouseoverURL('', window);">
onunload="PlacesUIUtils.setMouseoverURL('', window);"
aria-label="&bookmarksButton.label;">
<script type="application/javascript"
src="chrome://browser/content/places/bookmarksSidebar.js"/>

View File

@ -12,13 +12,16 @@
%placesDTD;
<!ENTITY % editMenuDTD SYSTEM "chrome://global/locale/editMenuOverlay.dtd">
%editMenuDTD;
<!ENTITY % browserDTD SYSTEM "chrome://browser/locale/browser.dtd">
%browserDTD;
]>
<page id="history-panel"
orient="vertical"
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
onload="HistorySidebarInit();"
onunload="PlacesUIUtils.setMouseoverURL('', window);">
onunload="PlacesUIUtils.setMouseoverURL('', window);"
aria-label="&historyButton.label;">
<script type="application/javascript"
src="chrome://browser/content/places/historySidebar.js"/>