STR: 1. Open Firefox Options (e.g. Tools menu -> Options). 2. Select the Privacy page. 3. Tab to the "Firefox will:" drop-down in the History group. 4. Press tab again. Expected: Focus should land on the "clear your recent history" link. Actual: Focus skips this link. Instead, it moves out of the History group into the Location Bar group. This also applies to the "manage your Do Not Track settings" and "remove individual cookies" links. There is no way to focus these links (class: inline-link) at all using the keyboard.

These links do have href="#", which normally makes an <a> tag focusable. However, for some reason, this doesn't seem to work here. Is this a weird XUL/HTML interaction? Setting tabindex="0" does fix this. Fixing the href="#" failure is perhaps the more "correct" solution, but it's also probably much harder.

Gijs, any idea on this why that a with an href doesn't get focus by default?

(In reply to Marco Zehe (:MarcoZ) from comment #2) > Gijs, any idea on this why that a with an href doesn't get focus by default? Nope. It seems Services.focus.elementIsFocusable(content.document.getElementById("historyRememberCookies"), false); returns false. It shouldn't. I don't know why it does, and a quick look at code isn't helpful. Neil probably does? Another workaround besides tabindex="1" would be to use a XUL <label> instead of an <html:a> -- this is what the other links that *are* focusable do (e.g. 'learn more' after do not track). Are they reasonably accessible otherwise?

(In reply to :Gijs Kruitbosch from comment #3) > Another workaround besides tabindex="1" Maybe a typo, but note i was suggesting tabindex="0", not "1", as "1" will give it precedence over other stuff without a tabindex, which I don't think what we want. > would be to use a XUL <label> > instead of an <html:a> -- this is what the other links that *are* focusable > do (e.g. 'learn more' after do not track). Are they reasonably accessible > otherwise? Yes. I assumed there was some specific reason label wasn't being used here...

(In reply to :Gijs Kruitbosch from comment #3) > Another workaround besides tabindex="1" would be to use a XUL <label> > instead of an <html:a> -- this is what the other links that *are* focusable > do (e.g. 'learn more' after do not track). Are they reasonably accessible > otherwise? They are perfectly accessible when marked up as a link, yes.

Debugging shows that the file has the base url 'about:preferences#privacy'. A link of '#' in there would become 'about:preferences#privacy#' which when trying to construct a url for this throws NS_ERROR_MALFORMED_URI, so it is link with an invalid uri and thus not focusable.

(In reply to James Teh [:Jamie] from comment #1) > Setting tabindex="0" does fix this. Err... no. It fixes the focusability, but you can't press enter to activate it. Sorry.

This changes the href="#" to href="javascript:void(0)", which is a valid URL and thus works as expected. I don't know whether this is an acceptable solution, however.

Comment on attachment 8763149 [details] [diff] [review] Patch Jared, I'd be fine with this, but we could also switch to <label> if you think that's better, or try and write more complicated JS to fix the hrefs to make sense, or something. Though tbh, I also don't think "#" should just be appended to the document base URI... just "#" as a link should work irrespective of the hash that's in use on the page. :-\

(In reply to :Gijs Kruitbosch from comment #9) > Comment on attachment 8763149 [details] [diff] [review] > Patch > > Jared, I'd be fine with this, but we could also switch to <label> if you > think that's better, or try and write more complicated JS to fix the hrefs > to make sense, or something. > > Though tbh, I also don't think "#" should just be appended to the document > base URI... just "#" as a link should work irrespective of the hash that's > in use on the page. :-\ So basically this is because the about protocol handler doesn't care about baseURI and just ignores it, so relative URIs don't work. Changing to an absolute "about:preferences#privacy" (or any other hash or lack thereof) would also work. Changing the about protocol handler to make this work would be somewhat useful for things like about:support, I guess? But also scary. :-\

Comment on attachment 8763149 [details] [diff] [review] Patch Review of attachment 8763149 [details] [diff] [review]: ----------------------------------------------------------------- I would prefer that we use a xul:label with class="text-link" here. The javascript:void(0) isn't seen often in our codebase, and it appears odd in the status bar.

Review commit: https://reviewboard.mozilla.org/r/60296/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/60296/

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, https://reviewboard.mozilla.org/r/60296/#review57472

Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/d724f337b4f0 fix focusing of text links in about:preferences, r=jaws

Backed out on fx-team: remote: https://hg.mozilla.org/integration/fx-team/rev/4aeee4260fcd for causing bug 1282794.

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, Review request updated; see interdiff: https://reviewboard.mozilla.org/r/60296/diff/1-2/

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, Can we get rid of inline-link? It only appears to be used by browser/base/content/sync/genericChange.xul and browser/base/content/sync/setup.xul at this point.

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, Review request updated; see interdiff: https://reviewboard.mozilla.org/r/60296/diff/2-3/

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, The inline-link thingy in toolkit/mozapps/update/content/updates.xul is entirely separate. Let's not touch this here.

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, Review request updated; see interdiff: https://reviewboard.mozilla.org/r/60296/diff/3-4/

Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/fx-team/rev/d69c5dd616ad fix focusing of text links in about:preferences, r=dao

I have reproduced this bug with Nightly 50.0a1 (2016-06-16), windows 8.1 64bit! The bug's fix is verified on Latest Nightly 50.0a1. Build ID : 20160713030216 User Agent : Mozilla/5.0 (Windows NT 6.3; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0 [bugday-20160713]

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, Approval Request Comment [Feature/regressing bug #]: n/a [User impact if declined]: some links in the preferences have poor contrast and aren't keyboard accessible [Describe test coverage new/current, TreeHerder]: minimal, these are mostly styling changes [Risks and why]: low risk, baked for a week and has been verified [String/UUID change made/needed]: nope

Comment on attachment 8764368 [details] Bug 1280472 - fix focusing of text links in about:preferences, This patch fixes some non-accessible links via keyboard in a11y and is verified. Take it in 49 aurora.