Thank you, Sergey! Looking for the second “+1”. Dmitry
> On 8 Aug 2019, at 00:25, Sergey Bylokhov <[email protected]> wrote: > > Looks fine. > > ----- [email protected] <mailto:[email protected]> 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 <[email protected] > > <mailto:[email protected]>> 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).
