RFR: 8283214: [macos] Screen magnifier does not show the magnified text for JcomboBox [v3]
Alexey Ivanov
aivanov at openjdk.org
Mon Jul 3 11:18:58 UTC 2023
On Mon, 3 Jul 2023 11:11:47 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>>> What if `JComboBox` uses a non-standard renderer which takes a property from an object? Let's assume, `JComboBox` is populated with `Rectangle` objects, a custom renderer is set which displays the value of the `bottom` field in the combo box. Your code, if I understand it correctly, will default to _full rectangle_ rather than what's really displayed.
>>>
>> You were right. At least in this case the current fix returns _full_rectangle_ dimension in a string form in stead of the text displayed in JComboBox. I had a doubt here if a custom renderer is used to populate the JComboBox, isn't it required to `override` the `toString` method to return useful information from user end?
>>
>> If not then I think it is required to look for alternate fix, probably to get the displayed text from JComboBox and set it as an accessibleName.
>>
>> >How does JTable handle this kind of situation? Or JList?
>>
>> I case of JTable and JList, the accessible context is of type JLabel which returns the accessibleName in string format.
>
>> > What if `JComboBox` uses a non-standard renderer which takes a property from an object? Let's assume, `JComboBox` is populated with `Rectangle` objects, a custom renderer is set which displays the value of the `bottom` field in the combo box. Your code, if I understand it correctly, will default to _full rectangle_ rather than what's really displayed.
>>
>> You were right. At least in this case the current fix returns _full_rectangle_ dimension in a string form instead of the text displayed in JComboBox.
>
> Thank you for confirming it.
>
>> I had a doubt here if a custom renderer is used to populate the JComboBox, isn't it required to `override` the `toString` method to return useful information from user end?
>
> No, it's not required to override the `toString` method if the programmer doesn't use it.
>
>> If not then I think it is required to look for alternate fix, probably to get the displayed text from JComboBox and set it as an accessibleName.
>
> Could be…
>
> How would you get the text? It should be provided by a renderer…
>
>> > How does JTable handle this kind of situation? Or JList?
>>
>> I case of JTable and JList, the accessible context is of type JLabel which returns the accessibleName in string format.
>
> This doesn't really answer the question. Do `JTable` and `JList` correctly display the magnified text when a custom renderer is used? In the case with rectangle and a custom renderer? Do they still display text from `toString`?
>
> If neither `JTable` nor `JList` handle this situation, it's okay to go with `toString` as you implemented.
How does `JTable` handle the situation where an image is displayed? You can use SwingSet2 for testing: it displays colored text and images.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14497#discussion_r1250725240
More information about the client-libs-dev
mailing list