RFR: 4938801: The popup does not go when the component is removed [v6]
Alexey Ivanov
aivanov at openjdk.org
Tue Jul 29 12:03:59 UTC 2025
On Tue, 29 Jul 2025 11:56:49 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> Issue is seen that a popup doesn't get closed when the component that invokes it, gets removed from the parent container.
>> This is because the JPopupMenu does not listen to its invoker liefecycle thereby behaving as a standalone entity after creation.
>> Fix is made to make sure popup listens to its invoker lifecycle by registering its PropertyChangeListener to the invoker and listens to the ["ancestor" property name ], https://github.com/openjdk/jdk/blob/441dbde2c3c915ffd916e39a5b4a91df5620d7f3/src/java.desktop/share/classes/javax/swing/JComponent.java#L4853-L4858 which will become null when removed, wherein we should dispose of the popup
>
> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>
> Use named property listener, update test
Looks good to me.
May I be recorded as a contributor / co-author for the test?
-------------
Marked as reviewed by aivanov (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26407#pullrequestreview-3067165449
PR Comment: https://git.openjdk.org/jdk/pull/26407#issuecomment-3132156615
More information about the client-libs-dev
mailing list