Page MenuHomePhabricator

Add access key for global search field
Closed, WontfixPublic

Description

Since / shorcut has been removed in D15976, then it would be great to add access key for global search field. Suggesting "f", which is most typically used. It will also improve the accessibility.

Event Timeline

epriestley claimed this task.
epriestley added a subscriber: epriestley.

See discussion in D15984.

I don't see anything relevant there actually, could you please elaborate?

This feature has a tiny impact and a huge support cost.

See D1902, T989, T11263, D15984, T4103 , D15976, https://secure.phabricator.com/w/changelog/2016.22/, T2527, T11231, T8286, and here for how much time I've already spent discussing it.

Those are all about preferences and JavaScript handling...

What I'm trying to request is only to add accesskey attribute, nothing else.

From current

<input type="text" name="query" id="UQ0_71" autocomplete="off" /> <!-- page source -->
<input type="text" autocomplete="off" id="UQ0_71" name="query" data-autoid="autoid_10" class=" jx-typeahead-placeholder"> <!-- Firebug DOM copy -->

to desired

<input type="text" name="query" id="UQ0_71" autocomplete="off" accesskey="f" /> <!-- page source -->
<input type="text" autocomplete="off" id="UQ0_71" name="query" data-autoid="autoid_10" class=" jx-typeahead-placeholder" accesskey="f" /> <!-- Firebug DOM copy -->

That shouldn't be that difficult?

Can you substantiate "f" being "typically used"? Here's what I see in use in the wild from a quick survey of popular sites and software:

SoftwareAccess Key
Mediawikif
TwitterNone
GoogleNone
FacebookNone
USPSNone
Whitehouse.govNone
YahooNone
BaiduNone
AmazonNone
QQNone
BingNone
AppleNone
MicrosoftNone
RedditNone
LinkedInNone
MozillaNone

Here's some code review and bug tracking software:

SoftwareAccess Key
GitHubNone
BitbucketNone
TracNone
BugzillaNone
TrelloNone

Here are some standards and accessibility-focused sites:

SiteAccess Key
W3C (w3.org)None
WebAIMNone
JAWS Screen ReaderNone
ADA.govNone
"UK Government Recommendations (2002)"4
"UK Government Recommendations (2002)" Host SiteNone
"Standard Access Keys 2014"4
"Standard Access Keys 2014" Parent Site (a Mediawiki)f

Good point, thanks for research. I was looking in some notes from research done several years ago, thus obviously no longer accurate.

However, it is still quite comfortable to navigate to the search input:

Github: data-hotkey="s" tabindex="1"
W3C: tabindex="3"
Google main page: autofocused
Yahoo: autofocus=""

If you don't want to make Phabricator more user friendly for users using keyboard by adding access key which won't hurt anything, how about tabindex="1", which is totally harmless? (autofocus isn't good in this particular case)

TBH, I seriously do not understand what's bad on it. If it would be breaking something or adding other maintenance work (such as preferences or so), I would understand, but adding single harmless attribute? Sure I can miss somethhing and I would be happy if you could explain why accesskey or tabindex would be bad for Phabricator. Thank you.

In the future, we might reasonably want to put tabindex="1" on the first input on form pages, like the "Edit Task" page, so that you could press tab once to quickly begin editing. If we put tabindex="1" on global search, the cost of making this change in the future will be much higher.

...which takes us back to accesskey...