RFR: 8283214: [macos] Screen magnifier does not show the magnified text for JcomboBox [v6]
Abhishek Kumar
abhiscxk at openjdk.org
Tue Jul 25 12:03:54 UTC 2023
On Mon, 24 Jul 2023 19:11:00 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Abhishek Kumar has updated the pull request incrementally with one additional commit since the last revision:
>>
>> bugid removed in comments
>
> src/java.desktop/macosx/classes/com/apple/laf/AquaComboBoxButton.java line 34:
>
>> 32:
>> 33: import apple.laf.JRSUIState;
>> 34: import apple.laf.JRSUIConstants.*;
>
> I think we should expand all wild card imports.
Updated.
> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicComboBoxUI.java line 1131:
>
>> 1129: switch ( i ) {
>> 1130: case 0:
>> 1131: if ( popup instanceof Accessible accessiblePopup) {
>
> Suggestion:
>
> if (popup instanceof Accessible accessiblePopup) {
This code segment is reverted back and handled in `AquaComboBoxUI` now.
> test/jdk/javax/accessibility/JComboBox/TestJComboBoxScreenMagnifier.java line 50:
>
>> 48: "1) Enable Screen magnifier on the Mac \n\n" +
>> 49: "System Preference -> Accessibility -> Zoom -> " +
>> 50: "Select ( Enable Hover Text) \n\n" +
>
> Suggestion:
>
> "Select "Enable Hover Text"\n\n" +
Updated.
> test/jdk/javax/accessibility/JComboBox/TestJComboBoxScreenMagnifier.java line 58:
>
>> 56: InvocationTargetException {
>> 57: PassFailJFrame passFailJFrame = new PassFailJFrame(
>> 58: "JMenu Screen Magnifier Test Instructions", INSTRUCTIONS, 5, 12, 40);
>
> Suggestion:
>
> "JComboBox Screen Magnifier Test Instructions", INSTRUCTIONS, 5, 12, 40);
Updated.
> test/jdk/javax/accessibility/JComboBox/TestJComboBoxScreenMagnifier.java line 63:
>
>> 61: passFailJFrame.awaitAndCheck();
>> 62: }
>> 63: private static void createAndShowUI() {
>
> Suggestion:
>
> }
>
> private static void createAndShowUI() {
>
>
> Please add a blank line between methods.
Updated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14497#discussion_r1273426943
PR Review Comment: https://git.openjdk.org/jdk/pull/14497#discussion_r1273426473
PR Review Comment: https://git.openjdk.org/jdk/pull/14497#discussion_r1273426634
PR Review Comment: https://git.openjdk.org/jdk/pull/14497#discussion_r1273426830
PR Review Comment: https://git.openjdk.org/jdk/pull/14497#discussion_r1273426730
More information about the client-libs-dev
mailing list