RFR: 7900303: auto updating components need accessbility control [v2]
Alexandre Iline
shurailine at openjdk.org
Fri Aug 30 18:33:26 UTC 2024
On Fri, 30 Aug 2024 09:06:42 GMT, Dmitry Bessonov <dbessono at openjdk.org> wrote:
>> The auto-updating component that can be improved in JTHarness UI in terms of accessibility is test run progress bar.
>>
>> Such progress bars are visible on the main desktop and in test run monitor window.
>>
>> We should provide a mechanism for the user to hide test run progress bars.
>>
>> ### Default state
>>
>> 
>>
>> 
>>
>> Visually unchanged comparing to what we have now, without the patch.
>>
>> ### Started with `-Djavatest.desktop.testrunprogressmonitor.hidden=true`
>>
>> 
>>
>> 
>
> Dmitry Bessonov has updated the pull request incrementally with one additional commit since the last revision:
>
> CRN second year update
LGTM
src/com/sun/javatest/exec/ProgressMonitor.java line 382:
> 380:
> 381: boolean showTestRunProgressMonitor =
> 382: !Boolean.valueOf(System.getProperty("javatest.desktop.testrunprogressmonitor.hidden"));
AFAICS in the JavaTest code the property names sometimes are stored in string constants and sometimes are hardcoded in the sources. I can not tell which way is preferred, so just making a comment that optionally a string constant can be used for the property name.
-------------
PR Comment: https://git.openjdk.org/jtharness/pull/82#issuecomment-2322118738
PR Review Comment: https://git.openjdk.org/jtharness/pull/82#discussion_r1739236474
More information about the jtharness-dev
mailing list