On Thu, 11 Aug 2022 15:47:40 GMT, Damon Nguyen <dngu...@openjdk.org> wrote:
>> When a JComboBox is editable, the button segment of the combo box is >> misaligned vertically and has a different height. This change fixes these >> issues and adds a manual test that checks the appearance of an editable and >> non-editable JComboBox. >> >> One of the discussions revolving this issue is the native macOS appearance >> of editable JComboBoxes. After looking through native macOS apps, the only >> one found is in System Preferences > Date & Time. The problem here is that >> the native equivalent found here uses a blue button with a single down arrow >> as the button's symbol. The current swing implementation uses a white button >> with an up & down arrow symbol for the button. A JRS widget button that has >> this blue button with a single downward arrow exists but does not support >> text fields. >> >> As such, I believe the best fix for this issue is to mainly fix the >> alignment and sizing issue. I looked through Apple's documentation for these >> UI elements but editable JComboBoxes aren't specifically listed anywhere. >> Similarly, there's barely any editable JComboBoxes used in native mac apps >> (only the date & time). So, I don't think it's a major issue if JComboBox >> does not exactly match the example found in Date & Time. > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revision: > > Removed newline. src/java.desktop/macosx/classes/com/apple/laf/AquaComboBoxUI.java line 460: > 458: System.out.println("rectForCurrentValue NEW: " + > comboBox.getHeight()); > 459: int width = comboBox.getWidth(); > 460: int height = 21; Is this a Aqua LAF constraint on max height of text-field? ------------- PR: https://git.openjdk.org/jdk/pull/9473