Looks fine. ----- [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/ > > Thanks, > Dmitry > On 2 Aug 2019, at 23:57, Sergey Bylokhov < [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).
