RFR: 8300269: The selected item in an editable JComboBox with titled border is not visible in Aqua LAF [v3]

Phil Race prr at openjdk.org
Mon Jan 23 20:52:06 UTC 2023


On Fri, 20 Jan 2023 22:45:32 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:

>> The fixed height causes the titled border insets to reserve a majority of the height available for the component. This fixed height is meant for the default AquaComboBox usage, so the old default height is more applicable when a border, such as TitledBorder, is set on an Aqua Editable ComboBox.
>> 
>> A conditional to check for a border is added since this is the default for an AquaComboBox using AquaBorder in AquaComboBoxUI's rectangleForCurrentValue() step. A check for any border rather than an instance of a TitledBorder is used because this issue causes changes in other borders as well (I found it for LineBorder for example). 
>> 
>> The editable JComboBox is visible with a LineBorder, but the coordinates of the comboBox are aligned with the top of the arrowButton instead of the bottom (which was the case before the previous fix). So, the method now defaults to the behavior before the fix for any editable comboBox pre-fix if a different border is set.
>
> Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Change bounds to toolkit screen size

Based on what I see in the bug report showing the item is completely invisible
and that the fix looks safe and simple and local, I believe this should be considered a P2
regression in JDK 20 and fixed directly there ASAP.
So withdraw this PR and resubmit against JDK 20 following the RDP 2 protocols.

-------------

PR: https://git.openjdk.org/jdk/pull/12087



More information about the client-libs-dev mailing list