RFR: 8248: [Accessibility] Low Contrast in High Contrast mode on Stacktrace View [v4]

Brice Dutheil bdutheil at openjdk.org
Mon Sep 16 08:37:15 UTC 2024


On Mon, 16 Sep 2024 08:16:45 GMT, Suchita Chaturvedi <schaturvedi at openjdk.org> wrote:

>> This PR is to fix the accessibility issue for High Contrast mode. Stracktrace View was not readable at all in dark mode.
>> 
>> Before the fix:
>> <img width="959" alt="image" src="https://github.com/user-attachments/assets/f12f1b7d-aa8a-4cce-8e0c-7731d5f9cc96">
>> 
>> After the fix:
>> <img width="959" alt="image" src="https://github.com/user-attachments/assets/8f7be98a-206b-4758-a1c2-8971d7ab1c55">
>
> Suchita Chaturvedi has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Merge branch '8248' of https://github.com/Suchitainf/jmc into 8248
>  - Resolve Spotbug error

application/org.openjdk.jmc.ui.common/src/main/java/org/openjdk/jmc/ui/common/util/ThemeUtils.java line 37:

> 35: 	private static final double BRIGHTNESS_THRESHOLD = 0.5;
> 36: 
> 37: 	private static void isDarkMode() {

**nitpick:** I'd rename `isDarkMode` that better describe that it updates the state, e.g. 

Suggestion:

	private static void updateCurrentThemeDarkModeStatus() {

-------------

PR Review Comment: https://git.openjdk.org/jmc/pull/580#discussion_r1760739614


More information about the jmc-dev mailing list