Page MenuHomePhabricator

While using the mobile site with a keyboard, pressing escape does not close the visual editor
Closed, ResolvedPublic

Description

This could be someone using the mobile site on desktop, or someone using a mobile device with a keyboard attached.

Event Timeline

Change 805810 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/VisualEditor@master] Use surface's cancel command to close the editor

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

Change 805810 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Use surface's cancel command to close the editor

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

matmarex edited projects, added Editing QA; removed Patch-For-Review.
matmarex moved this task from Inbox to QA on the Editing-team (Kanban Board) board.
EAkinloose added subscribers: matmarex, EAkinloose.

This works fine. However @matmarex , I am curious about the expected result for pressing the esc key while in the source editor. Nothing happens currently.

I think that's expected, the mobile source editor never supported this and we didn't change it here, but I guess maybe we should change it as well?

I'm not sure if we want to work on this. The change for visual mode was motivated by some inconsistencies in the behavior of the Escape key on desktop, and it was easier to apply the same bahvior to mobile than to try to disable it, but there isn't really such motivation for the source mode.

I think that's expected, the mobile source editor never supported this and we didn't change it here, but I guess maybe we should change it as well?

I'm not sure if we want to work on this. The change for visual mode was motivated by some inconsistencies in the behavior of the Escape key on desktop, and it was easier to apply the same bahvior to mobile than to try to disable it, but there isn't really such motivation for the source mode.

It's fine to leave it as is for now. I just wanted to confirm. Thanks!

Esanders renamed this task from While using the mobile site with a keyboard, pressing escape does not close the editor to While using the mobile site with a keyboard, pressing escape does not close the visual editor.Jun 17 2022, 4:01 PM
ppelberg claimed this task.