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

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Tue Aug 20 08:01:26 UTC 2019


+1

Regards
Prasanta
On 19-Aug-19 8:44 PM, Dmitry Markov wrote:
> 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/
>
> Thanks,
> Dmitry
>
>> On 19 Aug 2019, at 10:12, Prasanta Sadhukhan 
>> <prasanta.sadhukhan at oracle.com 
>> <mailto: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/
>>>>> >
>>>>> > 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/20190820/10935491/attachment.html>


More information about the swing-dev mailing list