On Fri, 27 Jan 2023 10:14:51 GMT, Damon Nguyen <[email protected]> wrote:
>> Fixes regression bug where Aqua L&F editable JComboBox with a TitledBorder >> was not visible. >> >> This fix adds a conditional that addresses cases where a border is used on >> an AquaComboBox. The patch that caused the issue set a static height, but >> this height caused issues with a TitledBorder, which modifies the height to >> account for the border. In this case, revert to the old method of >> calculating a comboBox's height. >> >> A point made in the previous PR was that the previous method had +1 to >> comboBox height. This could be re-added, but in the comment thread in [the >> previous PR](https://github.com/openjdk/jdk/pull/12087), leaving this out is >> better because the extra height causes the textField to be skewed South. >> >> <img width="63" alt="Screen Shot 2023-01-20 at 12 54 47 PM" >> src="https://user-images.githubusercontent.com/96267980/214998560-2340ac1d-5df3-47ba-94f7-56459211330e.png"> >> >> This also adds the test that checks an editable JComboBox across all L&F's >> except GTK. There seems to be an unrelated bug to this fix that causes GTK >> L&F comboBoxes to not behave as expected with these key presses. > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revision: > > Change return to continue Marked as reviewed by kizune (Reviewer). ------------- PR: https://git.openjdk.org/jdk20/pull/119
