TableTool icon indicating copy to clipboard operation
TableTool copied to clipboard

Support For Mojave Dark Mode

Open chrisgrande opened this issue 7 years ago • 4 comments

Changed table view to use system colors to provide support for Mojave Dark Mode. dark_mode

chrisgrande avatar Dec 07 '18 19:12 chrisgrande

I'm not very happy with the color choices. Colors like unemphasizedSelectedTextBackgroundColor have a very specific function, and should not be used for unrelated things. Either use a custom color (use asset catalog to create custom color with light / dark mode), or use a color that is suitable from a semantic point of view (is there a system grid color?)

jakob avatar Dec 17 '18 13:12 jakob

It's a good point. I had tested gridColor but it was very dark and wasn't being correctly color mixed with the Desktop picture - this now seems to be the case with unemphasizedSelectedTextBackgroundColor which does't get mixed in Dark Mode. Changes in 10.14.2 perhaps?

I have switched to gridColor (it seems to be working correctly now) for the grid lines and kept the unused table sections as alternatingContentBackgroundColor as it does the right thing in Dark Mode and adapts to the Desktop Picture.

chrisgrande avatar Dec 18 '18 22:12 chrisgrande

Any plans to merge it?

tyson90 avatar Jul 09 '20 12:07 tyson90

I'd love to have dark mode on my Table Tool app. Any plans on updating the app to support dark mode?

javinladish avatar Oct 08 '20 20:10 javinladish