errael commented on PR #4739:
URL: https://github.com/apache/netbeans/pull/4739#issuecomment-1271064220

   > manual pass
   
   Using `findDialogParent`, which is needed anyway since not all the classes 
in question meet the criteria,  there's good odds that the `Component` in 
question is the focusOwner so things should work out. If it's not the focus 
owner, I guess it would still be safe to use it as the owner; the dialog in 
question is placed over that component; but I think focus owner would be 
better. Hmm, might need to check isVisible()?
   
   I can probably be talked into it; but I'm uncomfortable making manual 
changes without testing and I'm not sure I'll be able to navigate to all the 
dialogs.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to