<Swing Dev> [11] JDK-8194873: right ALT key hotkeys no longer work in Swing components
Prasanta Sadhukhan
prasanta.sadhukhan at oracle.com
Fri Jun 22 08:39:33 UTC 2018
ok. +1
Regards
Prasanta
On 6/22/2018 1:55 PM, Pankaj Bansal wrote:
>
> Hello Prasanta,
>
> <<437 @SuppressWarnings("deprecation")
> <<in BasicMenuItemUI
>
> While setting the modifier in accelerator, there are three possible
> cases with ALT and ALT_GRAPH modifiers we have to handle
>
> 1.When both ALT and ALT_GRAPH modifiers were set: We need to add the
> ALT only modifier keystroke which is used for left ALT key. Unsetting
> the ALT_GRAPH will do that as ALT is already set
>
> 2.When only ALT was set: We need to add the ALT+ALT_GRAPH modifier
> keystroke
>
> 3.When only ALT_GRAPH was set: We don’t need the ALT_GRAPH only
> modifier as this will never be the true (Windows gives true for both
> ALT and ALT_GRAPH when right ALT key is pressed). Add the ALT and
> ALT+ALT_GRAPH modifiers keystroke which are used for left ALT key and
> right ALT keys respectively.
>
> Now, to unset the ALT_GRAPH, I need to unset both ALT_GRAPH_MASK and
> ALT_GRAPH_DOWN_MASK as they are corresponding flags for old and new
> modifiers. So I need to access the ALT_GRAPH_MASK but it deprecated,
> so had to suppress warnings. This is used at lot of places for same
> reason.
>
> Please see KeyEvent-> setNewModifiers, setOldModifiers.
>
> Please let me know if this makes sense.
>
> <<The test still does not cleanup/dispose the frames on exit. Please
> have a look.
>
> Done.
>
> Webrev:
>
> http://cr.openjdk.java.net/~pbansal/8194873/webrev.05/
>
> *From:*Prasanta Sadhukhan
> *Sent:* Friday, June 22, 2018 10:56 AM
> *To:* Pankaj Bansal; Sergey Bylokhov
> *Cc:* swing-dev at openjdk.java.net
> *Subject:* Re: <Swing Dev> [11] JDK-8194873: right ALT key hotkeys no
> longer work in Swing components
>
> Can you explain
>
> 437 @SuppressWarnings("deprecation")
> in BasicMenuItemUI
>
> The test still does not cleanup/dispose the frames on exit. Please
> have a look.
>
> Regards
> Prasanta
>
> On 6/21/2018 5:55 PM, Pankaj Bansal wrote:
>
> Hi Prasanta/Sergey
>
> @Prasanta
>
> << The test passed for me in windows 10 even without your fix and the test frames and dialogs are kept visible even after the test finished. Please have a look.
>
> Sorry I think I was testing by printing log on console instead of throwing exceptions and I made webrev with that version. I have corrected it now. The test fails without fix and passes with fix on Windows 10.
>
> @Sergey
>
> << Is this bug applicable only to windows? can we make the test non-windows specific?
>
> Yes, I think this bug addresses the regression introduced when the fix was made forhttps://bugs.openjdk.java.net/browse/JDK-8041928. Both ALT and ALT_GRAPH keys work on Windows. The test added fails on linux and mac both with and without fix. I think the ALT_GRAPH key does not work on these platforms and it is not related to the regression being addressed here. We can file separate bug to investigate that. Please let me know what do you think regarding this.
>
> I have run possible affected jck tests on Windows and all the automated event tests in awt/swing jtreg tests on Windows, Linux and Mac. The fix is not producing any regression.
>
> webrev:
>
> http://cr.openjdk.java.net/~pbansal/8194873/webrev.04/
> <http://cr.openjdk.java.net/%7Epbansal/8194873/webrev.04/>
>
> Thanks,
>
> Pankj Bansal
>
> -----Original Message-----
>
> From: Prasanta Sadhukhan
>
> Sent: Thursday, June 21, 2018 11:58 AM
>
> To: Pankaj Bansal
>
> Cc:swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
>
> Subject: Re: <Swing Dev> [11] JDK-8194873: right ALT key hotkeys no longer work in Swing components
>
> Hi Pankaj,
>
> The test passed for me in windows 10 even without your fix and the test frames and dialogs are kept visible even after the test finished. Please have a look.
>
> Regards
>
> Prasanta
>
> On 6/20/2018 8:22 PM, Pankaj Bansal wrote:
>
> Hi Prasanta,
>
> I looked at documentation of KeyStroke.getKeyStroke and InputEvent.ALT_MASK should be used instead of ActionEvent.ALT_MASK. The corresponding InputEvent.ALT_GRAPH_MASK is already present. I used it by mistake last time.
>
> I have made changes for JMenuItem I have changed BasicJMenuItemUI for JMenuItem and updated the test case too. Other changes are same as webrev.02. Please have a look.
>
> Webrev:http://cr.openjdk.java.net/~pbansal/8194873/webrev.03/
> <http://cr.openjdk.java.net/%7Epbansal/8194873/webrev.03/>
>
> Regards,
>
> Pankaj Bansal
>
> -----Original Message-----
>
> From: Prasanta Sadhukhan
>
> Sent: Friday, April 20, 2018 5:04 PM
>
> To: Pankaj Bansal
>
> Cc:swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
>
> Subject: Re: <Swing Dev> [11] JDK-8194873: right ALT key hotkeys no
>
> longer work in Swing components
>
> Hi Pankaj,
>
> On 4/20/2018 3:48 PM, Pankaj Bansal wrote:
>
> Hi Prasanta,
>
> In case of JMenuItem, there are two ways to add key shortcuts
>
> 1.
>
> JMenuItem newMenuItem = new JMenuItem("New");
>
> newMenuItem.setMnemonic(KeyEvent.VK_N);
>
> In this case, the event is triggered if the N key is pressed. The modifiers are not used in this and pressing only N will work. So ALT or ALT_GRAPH is not required. These shortcut only work if the menu containing the given menu item is expanded.
>
> 2.
>
> JMenuItem newMenuItem = new JMenuItem("New");
>
> newMenuItem.setAccelerator(KeyStroke.getKeyStroke( KeyEvent.VK_N,
>
> ActionEvent.ALT_MASK)); In this case we are setting the accelerator key. In this case the key N will work with the modifier passed here. So the user is explicitly telling whether to use ALT, ALT_GRAPH or ALT+ALT+GRAPH. So I think we don’t need to change anything here. The menu containing the menu item does not have to be expanded in this case.
>
> But, as far I see in spec, ActionEvent does not have anyway to specify Right ALT mask or ALT_GRAPH_MASK so how the user can tell to use ALT_GRAPH.
>
> Regards
>
> Prasanta
>
> Please let me know what do you think about this.
>
> Regards,
>
> Pankaj Bansal
>
> -----Original Message-----
>
> From: Prasanta Sadhukhan
>
> Sent: Thursday, April 19, 2018 12:41 PM
>
> To: Pankaj Bansal; Andrej Golovnin
>
> Cc: Sergey Bylokhov;swing-dev at openjdk.java.net <mailto:swing-dev at openjdk.java.net>
>
> Subject: Re: <Swing Dev> [11] JDK-8194873: right ALT key hotkeys no
>
> longer work in Swing components
>
> Hi Pankaj,
>
> looks good. but it still does not test JMenuItem as I can see. Did you check if you have some menu items inside JMenu and set mnemonic, does Right Alt key works?
>
> Regards
>
> Prasanta
>
> On 4/10/2018 3:15 PM, Pankaj Bansal wrote:
>
> Hello Andrej,
>
> Thanks for the quick review. Yes, it does not sense to apply || on same value. It was a typo. Thanks for pointing it out.
>
> Webrev:
>
> http://cr.openjdk.java.net/~pbansal/8194873/webrev.02/
> <http://cr.openjdk.java.net/%7Epbansal/8194873/webrev.02/>
>
> Regards,
>
> Pankaj Bansal
>
> -----Original Message-----
>
> From: Andrej Golovnin [mailto:andrej.golovnin at gmail.com]
>
> Sent: Tuesday, April 10, 2018 2:18 PM
>
> To: Pankaj Bansal
>
> Cc: Prasanta Sadhukhan; Sergey Bylokhov;swing-dev at openjdk.java.net
> <mailto:swing-dev at openjdk.java.net>
>
> Subject: Re: <Swing Dev> [11] JDK-8194873: right ALT key hotkeys no
>
> longer work in Swing components
>
> Hi Pankaj,
>
> Webrev:
>
> http://cr.openjdk.java.net/~pbansal/8194873/webrev.01/
> <http://cr.openjdk.java.net/%7Epbansal/8194873/webrev.01/>
>
> src/java.desktop/windows/native/libawt/windows/awt_Component.cpp
>
> 3540 BOOL altIsDown = ((modifiers &
>
> java_awt_event_InputEvent_ALT_DOWN_MASK) ||
>
> 3541 (modifiers &
>
> java_awt_event_InputEvent_ALT_DOWN_MASK));
>
> Applying '||' on the same value does not make sense. I think the
>
> line
>
> 3541 should use 'java_awt_event_InputEvent_ALT_GRAPH_DOWN_MASK':
>
> 3541 (modifiers &
>
> java_awt_event_InputEvent_ALT_GRAPH_DOWN_MASK));
>
> Best regards,
>
> Andrej Golovnin
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20180622/dae51b69/attachment.html>
More information about the swing-dev
mailing list