Pressing Escape while a dialog is open closes the editor, should only close the dialog.
Description
Details
Related Objects
- Mentioned In
- rGOJU6f028abd0d6a: Kill the escape keydown event after handling a window close
rGVED4597c9a99dad: Stop propagation of events after handling trigger keys
rEVEDa18d1f1e347a: Check for stopped propagation before showing close dialog
rEVED9459e4e240a6: Check for stopped propagation before showing close dialog
Event Timeline
I can reproduce on Windows 7 x64 on every browser I tested: Opera 27, Firefox 34, IE 11.
Apparently also closing the editor in this way and then reopening it causes funny things like dialogs not opening at all anymore.
This is only reproducible if you make no changes. In that case Escape immediately closes the editor. But if you have made changes, it will launch a dialog asking you if want to keep your changes. But if you already have a dialog open (which right after pressing escape you do, even if it's in the process of closing, because it takes time for it to close), that will fail and it will look like nothing happened.
Change 179388 had a related patch set uploaded (by Catrope):
When Esc is pressed, don't close VE if a window is open
Change 179476 had a related patch set uploaded (by Esanders):
Kill the escape keydown event after handling a window close
Change 179478 had a related patch set uploaded (by Esanders):
Check for stopped propagation before show close dialog
Change 179479 had a related patch set uploaded (by Esanders):
Stop propagation of events after handling trigger keys
Change 179388 abandoned by Esanders:
When Esc is pressed, don't close VE if a window is open
Reason:
Replaced with these three commits:
https://gerrit.wikimedia.org/r/#/q/status:open topic:escape-propagation,n,z
Change 179476 merged by jenkins-bot:
Kill the escape keydown event after handling a window close
Change 179479 merged by jenkins-bot:
Stop propagation of events after handling trigger keys
Change 179478 merged by jenkins-bot:
Check for stopped propagation before showing close dialog
Change 180858 had a related patch set uploaded (by Jforrester):
Check for stopped propagation before showing close dialog
Change 180859 had a related patch set uploaded (by Jforrester):
Stop propagation of events after handling trigger keys
Change 180859 merged by jenkins-bot:
Stop propagation of events after handling trigger keys
Change 180858 merged by jenkins-bot:
Check for stopped propagation before showing close dialog