morgan pushed to branch base-browser-128.1.0esr-14.0-1 at The Tor Project / Applications / Tor Browser

Commits: e867f2ac by Henry Wilkes at 2024-08-14T16:12:13+00:00 fixup! Bug 40925: Implemented the Security Level component

Bug 42705: Adopt the same markup as "Enhanced Tracking Protection" and "DNS over HTTPS" for the security level preferences.

We inherit all the rules for spacing and highlighting.

We also update the radio disabled opacity rule following bugzilla bug 1869233.

- - - - -

3 changed files:

- browser/components/securitylevel/content/securityLevel.js - browser/components/securitylevel/content/securityLevelPreferences.css - browser/components/securitylevel/content/securityLevelPreferences.inc.xhtml

Changes:

===================================== browser/components/securitylevel/content/securityLevel.js ===================================== @@ -195,7 +195,6 @@ var SecurityLevelPanel = {

_configUIFromPrefs() { if (!this._populated) { - console.warn("_configUIFromPrefs before XUL was populated."); return; }

@@ -353,10 +352,7 @@ var SecurityLevelPreferences = { // Have the container's selection CSS class match the selection state of the // radio elements. for (const { container, radio } of this._radioOptions) { - container.classList.toggle( - "securityLevel-radio-option-selected", - radio.selected - ); + container.classList.toggle("selected", radio.selected); } },

===================================== browser/components/securitylevel/content/securityLevelPreferences.css ===================================== @@ -15,44 +15,12 @@ font-weight: bold; }

-/* Overwrite indent rule from preferences.css */ -#securityLevel-radiogroup description.indent { - color: var(--in-content-page-color); -} - -#securityLevel-radiogroup radio { - font-weight: bold; -} - #securityLevel-radiogroup[disabled] { opacity: 0.5; }

/* Overwrite the rule in common-shared.css so we don't get 0.25 opacity overall * on the radio text. */ -#securityLevel-radiogroup[disabled] radio[disabled] { +#securityLevel-radiogroup[disabled] radio[disabled] .radio-label-box { opacity: 1.0; } - -.securityLevel-radio-option { - border: 1px solid var(--in-content-box-border-color); - border-radius: 4px; - margin: 3px 0; - padding: 9px; -} - -.securityLevel-radio-option.securityLevel-radio-option-selected { - background-color: var(--section-highlight-background-color); - border: 1px solid var(--in-content-accent-color); - -} - -.securityLevel-radio-option:not( - .securityLevel-radio-option-selected -) .securityLevel-descriptionList { - display: none; -} - -.securityLevel-descriptionList description { - display: list-item; -}

===================================== browser/components/securitylevel/content/securityLevelPreferences.inc.xhtml ===================================== @@ -45,78 +45,82 @@ </hbox> </hbox> <radiogroup id="securityLevel-radiogroup"> - <vbox class="securityLevel-radio-option"> + <vbox class="securityLevel-radio-option privacy-detailedoption info-box-container"> <radio value="standard" data-l10n-id="security-level-preferences-level-standard" aria-describedby="securityLevelSummary-standard" /> - <vbox id="securityLevelSummary-standard"> - <description - class="summary indent" - flex="1" - data-l10n-id="security-level-summary-standard" - /> + <vbox id="securityLevelSummary-standard" class="indent"> + <label data-l10n-id="security-level-summary-standard" /> </vbox> </vbox> - <vbox class="securityLevel-radio-option"> + <vbox class="securityLevel-radio-option privacy-detailedoption info-box-container"> <!-- NOTE: We point the accessible description to the wrapping vbox - rather than its first description element. This means that when the - - securityLevel-descriptionList is shown or hidden, its text content - - is included or excluded from the accessible description, - - respectively. --> + - privacy-extra-information is shown or hidden, its text content is + - included or excluded from the accessible description, respectively. + --> <radio value="safer" data-l10n-id="security-level-preferences-level-safer" aria-describedby="securityLevelSummary-safer" /> - <vbox id="securityLevelSummary-safer"> - <description - class="summary indent" - flex="1" - data-l10n-id="security-level-summary-safer" - /> - <vbox class="securityLevel-descriptionList indent"> - <description - class="indent" - data-l10n-id="security-level-preferences-bullet-https-only-javascript" - /> - <description - class="indent" - data-l10n-id="security-level-preferences-bullet-limit-font-and-symbols" - /> - <description - class="indent" - data-l10n-id="security-level-preferences-bullet-limit-media" - /> + <vbox id="securityLevelSummary-safer" class="indent"> + <label data-l10n-id="security-level-summary-safer" /> + <vbox class="privacy-extra-information"> + <vbox class="indent"> + <hbox class="extra-information-label"> + <label + class="content-blocking-label" + data-l10n-id="security-level-preferences-bullet-https-only-javascript" + /> + </hbox> + <hbox class="extra-information-label"> + <label + class="content-blocking-label" + data-l10n-id="security-level-preferences-bullet-limit-font-and-symbols" + /> + </hbox> + <hbox class="extra-information-label"> + <label + class="content-blocking-label" + data-l10n-id="security-level-preferences-bullet-limit-media" + /> + </hbox> + </vbox> </vbox> </vbox> </vbox> - <vbox class="securityLevel-radio-option"> + <vbox class="securityLevel-radio-option privacy-detailedoption info-box-container"> <radio value="safest" data-l10n-id="security-level-preferences-level-safest" aria-describedby="securityLevelSummary-safest" /> - <vbox id="securityLevelSummary-safest"> - <description - class="summary indent" - flex="1" - data-l10n-id="security-level-summary-safest" - /> - <vbox class="securityLevel-descriptionList indent"> - <description - class="indent" - data-l10n-id="security-level-preferences-bullet-disabled-javascript" - /> - <description - class="indent" - data-l10n-id="security-level-preferences-bullet-limit-font-and-symbols-and-images" - /> - <description - class="indent" - data-l10n-id="security-level-preferences-bullet-limit-media" - /> + <vbox id="securityLevelSummary-safest" class="indent"> + <label data-l10n-id="security-level-summary-safest" /> + <vbox class="privacy-extra-information"> + <vbox class="indent"> + <hbox class="extra-information-label"> + <label + class="content-blocking-label" + data-l10n-id="security-level-preferences-bullet-disabled-javascript" + /> + </hbox> + <hbox class="extra-information-label"> + <label + class="content-blocking-label" + data-l10n-id="security-level-preferences-bullet-limit-font-and-symbols-and-images" + /> + </hbox> + <hbox class="extra-information-label"> + <label + class="content-blocking-label" + data-l10n-id="security-level-preferences-bullet-limit-media" + /> + </hbox> + </vbox> </vbox> </vbox> </vbox>

View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/e867f2ac...