Skip to content

Commit

Permalink
Bug 1487304: Add more labels/descriptions in Privacy preferences for …
Browse files Browse the repository at this point in the history
…accessibility. r=johannh,MarcoZ

This allows screen reader users to determine the context of various controls when tabbing through them.
Specific changes:
1. Describe the "Cookies and Site Data" group using the disk space indicator; e.g. "Your stored cookies, site data and cache are currently using 315 MB of disk space."
2. Associate the description for the "Content Blocking" group; i.e. "Block third-party content, like ads or code..."
3. Correct association of the label for the "Do Not Track" setting; i.e. 'Send websites a “Do Not Track” signal that you don’t want to be tracked'
4. Associate the label for the "Permissions" group.
5. Make each permission a labelled group so the user knows what the various "Settings…" buttons are for.
6. Associate the label for the "Data Collection and Use" group.

Differential Revision: https://phabricator.services.mozilla.com/D4638

--HG--
extra : moz-landing-system : lando
  • Loading branch information
jcsteh committed Aug 31, 2018
1 parent c224a4c commit f856e68
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions browser/components/preferences/in-content/privacy.xul
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@
</groupbox>

<!-- Site Data -->
<groupbox id="siteDataGroup" data-category="panePrivacy" hidden="true">
<groupbox id="siteDataGroup" data-category="panePrivacy" hidden="true" aria-describedby="totalSiteDataSize">
<caption><label data-l10n-id="sitedata-header"/></caption>

<hbox data-subcategory="sitedata" align="baseline">
Expand Down Expand Up @@ -279,7 +279,7 @@
</groupbox>

<!-- Tracking / Content Blocking -->
<groupbox id="trackingGroup" data-category="panePrivacy" hidden="true">
<groupbox id="trackingGroup" data-category="panePrivacy" hidden="true" aria-describedby="contentBlockingDescription">
<caption>
<label id="trackingProtectionHeader" hidden="true" data-l10n-id="tracking-header"/>
<label id="contentBlockingHeader" data-l10n-id="content-blocking-header"/>
Expand Down Expand Up @@ -481,7 +481,7 @@
</vbox>
</hbox>
<vbox id="doNotTrackLearnMoreBox">
<label><label class="tail-with-learn-more" data-l10n-id="do-not-track-description"></label><label
<label><label class="tail-with-learn-more" data-l10n-id="do-not-track-description" id="doNotTrackDesc"></label><label
class="learnMore text-link" href="https://www.mozilla.org/dnt"
data-l10n-id="do-not-track-learn-more"></label></label>
<radiogroup id="doNotTrackRadioGroup" aria-labelledby="doNotTrackDesc" preference="privacy.donottrackheader.enabled">
Expand All @@ -500,16 +500,16 @@
</hbox>

<!-- Permissions -->
<groupbox id="permissionsGroup" data-category="panePrivacy" hidden="true" data-subcategory="permissions">
<caption class="search-header" hidden="true"><label data-l10n-id="permissions-header"/></caption>
<groupbox id="permissionsGroup" data-category="panePrivacy" hidden="true" data-subcategory="permissions" aria-labelledby="permissionsGroupLabel">
<caption class="search-header" hidden="true"><label data-l10n-id="permissions-header" id="permissionsGroupLabel"/></caption>

<grid>
<columns>
<column flex="1"/>
<column/>
</columns>
<rows>
<row id="locationSettingsRow" align="center">
<row id="locationSettingsRow" align="center" role="group" aria-labelledby="locationPermissionsLabel">
<description flex="1">
<image class="geo-icon permission-icon" />
<separator orient="vertical" class="thin"/>
Expand All @@ -532,7 +532,7 @@
</hbox>
</row>

<row id="cameraSettingsRow" align="center">
<row id="cameraSettingsRow" align="center" role="group" aria-labelledby="cameraPermissionsLabel">
<description flex="1">
<image class="camera-icon permission-icon" />
<separator orient="vertical" class="thin"/>
Expand All @@ -555,7 +555,7 @@
</hbox>
</row>

<row id="microphoneSettingsRow" align="center">
<row id="microphoneSettingsRow" align="center" role="group" aria-labelledby="microphonePermissionsLabel">
<description flex="1">
<image class="microphone-icon permission-icon" />
<separator orient="vertical" class="thin"/>
Expand All @@ -578,7 +578,7 @@
</hbox>
</row>

<row id="notificationSettingsRow" align="center">
<row id="notificationSettingsRow" align="center" role="group" aria-labelledby="notificationPermissionsLabel">
<description flex="1">
<image class="desktop-notification-icon permission-icon" />
<separator orient="vertical" class="thin"/>
Expand Down Expand Up @@ -730,8 +730,8 @@
<label class="header-name" flex="1" data-l10n-id="collection-header"/>
</hbox>

<groupbox id="dataCollectionGroup" data-category="panePrivacy" hidden="true">
<caption class="search-header" hidden="true"><label data-l10n-id="collection-header"/></caption>
<groupbox id="dataCollectionGroup" data-category="panePrivacy" hidden="true" aria-labelledby="dataCollectionGroupLabel">
<caption class="search-header" hidden="true"><label data-l10n-id="collection-header" id="dataCollectionGroupLabel"/></caption>

<description>
<label class="tail-with-learn-more" data-l10n-id="collection-description"/>
Expand Down

0 comments on commit f856e68

Please sign in to comment.