Closed Bug 1486951 Opened 2 years ago Closed 2 years ago
Change accesskey for DNS over HTTPS label
46 bytes, text/x-phabricator-request
|Details | Review|
From comment 18 on Bug 1482271 > Fwiw: for "Enable DNS over HTTPS", the b is available, and when ignoring the > XUL Accesskey Policy for strings with descenders (which is already done for > the one pixel wide i character), g could be used in "Use system proxy > settings" to free the U for URL. I would recommend to avoid the H because of > Help buttons already present or possibly getting introduced in all cases, > and the above appears to work. Adding "Enable" to this string was a late change and I didn't re-evaluate the available accesskeys. Using 'b' looks like a good suggestion.
Note that you shouldn't change the string ID to update accesskeys in English.
Is this something we needed to address in 63 before the merge?
(In reply to Mike Conley (:mconley) (:⚙️) from comment #2) > Is this something we needed to address in 63 before the merge? I'd say it's something to address before the final merge of 63 to beta, to avoid either having inconsistent values in 63 or 64, or having to request uplift.
[Tracking Requested - why for this release]: See comment #3
Here's a patch that just does the accesskey switch on the new 'Enable DNS over HTTPS" label. I could also follow the rest of :Tonnes' suggestion, but that involves changing accesskeys for existing content and from Comment #3 I'm thinking we should be more cautious about that?
Comment on attachment 9005455 [details] Bug 1486951 - Change accesskey for Enable DNS over HTTPS label. r=flod Francesco Lodolo [:flod] has approved the revision.
Attachment #9005455 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/5ee4ea4aee38 Change accesskey for Enable DNS over HTTPS label. r=flod
You need to log in before you can comment on or make changes to this bug.