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