On Tue, 29 Mar 2022 07:22:55 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> @prsadhuk I have pushed latest changes related to AquaLookAndFeel.java file 
>> only. The CSystemColors.m is not checked-in because of 
>> [selectedContentBackgroundColor](https://developer.apple.com/documentation/appkit/nscolor/2998830-selectedcontentbackgroundcolor)
>>  type property build issues. 
>> 
>> 
>>> In my testing in osx10.15 with your fix, I see focus ring is following 
>>> highlight color and not accent color...did you modify anything in your fix? 
>>> <img alt="Screenshot 2022-03-28 at 11 52 09 AM" width="688" 
>>> src="https://user-images.githubusercontent.com/43534309/160338556-117c8e9b-4e78-4368-8040-dfca1d82e1ab.png";>
>> 
>> For this scenario, wanted to verify if the **selectionBackground** 
>> explicitly set to green in AquaLookAndFeel?
>
> With latest repo and your fix, I put a debug log in deriveContrastFocusRing() 
> and it seems to produce this output (for scenario above)
> 
> selectedBackgroundColor 
> com.apple.laf.AquaImageFactory$SystemColorProxy[r=54,g=128,b=34]
> 
> I see, even if I change the highlight Color to blue, it does not call 
> deriveContrastFocusRing() again and the focus ring remains green although 
> background Color of JTable cell changes to blue [ I think 
> deriveContrastFocusRing needs to be called every time accent color is changed 
> in System->Preferences]
> 
> <img width="537" alt="Screenshot 2022-03-29 at 12 49 59 PM" 
> src="https://user-images.githubusercontent.com/43534309/160556234-cf1732f3-57bd-4530-82e6-e2080e9e1551.png";>
> .

@prsadhuk Pushed new changes related to -

- On-the-fly focus ring changes
- Since **selectionBackground** was following highlight color changes in mac 
10.15 and accent color changes in mac 11.6, I have updated 
**deriveContrastFocusRing**() to use original **focusRingColor** (which follows 
accent color changes in both versions) to obtain the lighter version.

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

PR: https://git.openjdk.java.net/jdk/pull/7768

Reply via email to