<Swing Dev> [14] RFR 8225505: ctrl-F1 does not show the tooltip of a menu item (JMenuItems)

Dmitry Markov dmitry.markov at oracle.com
Mon Aug 19 15:14:01 UTC 2019


Hi Prasanta,

Thank you for the review. I have added the regression test as you requested. Please find the new version here: http://cr.openjdk.java.net/~dmarkov/8225505/webrev.03/ <http://cr.openjdk.java.net/~dmarkov/8225505/webrev.03/>

Thanks,
Dmitry

> On 19 Aug 2019, at 10:12, Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com> wrote:
> 
> Fix looks ok to me but I think a testcase is required, even if manual.
> 
> Regards
> Prasanta
> On 16-Aug-19 2:23 PM, Dmitry Markov wrote:
>> I still need a second reviewer. Any volunteers?
>> 
>> Thank you in advance,
>> Dmitry 
>> 
>>> On 8 Aug 2019, at 08:37, Dmitry Markov <dmitry.markov at oracle.com <mailto:dmitry.markov at oracle.com>> wrote:
>>> 
>>> Thank you, Sergey!
>>> Looking for the second “+1”.
>>> 
>>> Dmitry
>>> 
>>>> On 8 Aug 2019, at 00:25, Sergey Bylokhov <sergey.bylokhov at oracle.com <mailto:sergey.bylokhov at oracle.com>> wrote:
>>>> 
>>>> Looks fine.
>>>> 
>>>> ----- dmitry.markov at oracle.com <mailto:dmitry.markov at oracle.com> wrote: 
>>>> > 
>>>> > Hi Sergey,
>>>> > 
>>>> > I looked into your suggestion again and found that it is possible to use MenuKeyListener inside ToolTipManager without new event generation. So I updated the fix based on your recommendation. Please find the new version here: http://cr.openjdk.java.net/~dmarkov/8225505/webrev.02/ <http://cr.openjdk.java.net/~dmarkov/8225505/webrev.02/>
>>>> > 
>>>> > Thanks,
>>>> > Dmitry
>>>> 
>>>> > On 2 Aug 2019, at 23:57, Sergey Bylokhov <sergey.bylokhov at oracle.com <mailto:sergey.bylokhov at oracle.com>> wrote:
>>>> 
>>>> > 
>>>> > Hi, Dmitry.
>>>> 
>>>> Yes, it is possible to get the same result using the approach you
>>>> mentioned. Unfortunately registration of MenuKeyListener inside
>>>> ToolTipManager is not enough. Also we need to generate new
>>>> MenuKeyEvent with proper source (component) to let the tooltip manager
>>>> know for which menu element tooltip text should be displayed/hidden
>>>> (similar thing I did in my proposal, see processToolTipKeyEvent()
>>>> method). 
>>>> I am sorry but I do not think we have to implement such solution since
>>>> generation of new events is still necessary plus implementation of
>>>> MenuKeyListener is required.
>>>> 
>>>> Are you sure that the new events a necessary? It will be be really good to implement the fix w/o such events. We cannot sent keyEvents since we never do it before, and it will be good not to sent new MenuKeyEvent. Are you sure that an existed MenuKeyEvent event does not have enough information?(It has the component, path and current MenuSelectionManager).
>>> 
>> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20190819/e6ea614f/attachment.html>


More information about the swing-dev mailing list