Skip to content

[FIX] TableModel: Define foreground color when providing background#6011

Merged
PrimozGodec merged 1 commit intobiolab:masterfrom
ales-erjavec:data-table-text-color
Jun 10, 2022
Merged

[FIX] TableModel: Define foreground color when providing background#6011
PrimozGodec merged 1 commit intobiolab:masterfrom
ales-erjavec:data-table-text-color

Conversation

@ales-erjavec
Copy link
Contributor

Issue

In 'Data Table' On dark styles the contrast of text on background for metas and class columns is poor.

Before

Screenshot_20220609_120352

After

Screenshot_20220609_120512

Description of changes

Define foreground color when providing background to ensure text is contrasted against the background when system text color is light.

Includes
  • Code changes
  • Tests
  • Documentation

... to ensure text is contrasted against the background when system text
color is light.
@codecov
Copy link

codecov bot commented Jun 9, 2022

Codecov Report

Merging #6011 (26cae39) into master (86555e6) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #6011      +/-   ##
==========================================
- Coverage   86.58%   86.58%   -0.01%     
==========================================
  Files         315      315              
  Lines       67281    67285       +4     
==========================================
+ Hits        58258    58261       +3     
- Misses       9023     9024       +1     

@PrimozGodec PrimozGodec merged commit 0879bee into biolab:master Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants