Page MenuHomePhabricator

Escape followed by Tab no longer removes focus from the editor in non-wikitext
Closed, ResolvedPublic

Description

The CodeMirror library has a built-in "escape hatch" where pressing Escape followed by Tab will move focus out of the editor (see docs). This seems to be broken on master, but was working in REL1_45.

The escape hatch is critical for accessibility. It is also documented in the keyboard shortcuts help dialog.

Event Timeline

MusikAnimal triaged this task as Medium priority.

I think it is caused by r1220866 where closePanel() always returns true.

Change #1247511 had a related patch set uploaded (by Bhsd; author: Bhsd):

[mediawiki/extensions/CodeMirror@master] CodeMirrorSearch: do not always intercept `Escape` key

https://gerrit.wikimedia.org/r/1247511

Lol, of course it was me who broke it! Thanks for the fix :)

Change #1247511 merged by jenkins-bot:

[mediawiki/extensions/CodeMirror@master] CodeMirrorSearch: do not always intercept `Escape` key

https://gerrit.wikimedia.org/r/1247511