From e88f71dd6d1d7634e3a90a7e53859ff6dc028e21 Mon Sep 17 00:00:00 2001 From: Anna Koskinen Date: Thu, 16 Oct 2014 13:37:48 +0300 Subject: Moved accessibility shortcut handling to server-side. [#14843] Also allowed multiple shortcuts for closing Window. Change-Id: I35280ad1553af10ae54bc001e5707357f206b0ee --- WebContent/release-notes.html | 2 + client/src/com/vaadin/client/ui/VWindow.java | 4 +- server/src/com/vaadin/event/ShortcutAction.java | 38 ++++ server/src/com/vaadin/ui/Window.java | 99 ++++++++- .../tests/components/window/CloseShortcut.java | 121 +++++++++++ .../tests/components/window/CloseShortcutTest.java | 222 +++++++++++++++++++++ 6 files changed, 472 insertions(+), 14 deletions(-) create mode 100644 uitest/src/com/vaadin/tests/components/window/CloseShortcut.java create mode 100644 uitest/src/com/vaadin/tests/components/window/CloseShortcutTest.java diff --git a/WebContent/release-notes.html b/WebContent/release-notes.html index 8e6385684f..6cedeb1e21 100644 --- a/WebContent/release-notes.html +++ b/WebContent/release-notes.html @@ -137,6 +137,8 @@
  • The semantics of empty and required for Field classes has been made more consistent. This mainly affects Checkbox which is now considered to be empty when it is not checked.
  • The previously inconsistent behavior in HTML vs plain text rendering of Calendar event captions has been made consistent.
  • Support for Opera 12 has been dropped. Newer versions based on the Blink rendering engine are still supported.
  • +
  • Window's accessibility shortcut was moved to server-side. Now setCloseShortcut overrides the default value, while addCloseShortcut can be used to add more than one shortcut key for closing the window. + The protected value closeShortcut in Window was removed.
  • Known issues