Skip to content

fix: improve color contrast of editor find in dark mode#2248

Merged
dsmmcken merged 1 commit intomainfrom
dmckenzie_fix_highlight_color
Oct 4, 2024
Merged

fix: improve color contrast of editor find in dark mode#2248
dsmmcken merged 1 commit intomainfrom
dmckenzie_fix_highlight_color

Conversation

@dsmmcken
Copy link
Copy Markdown
Contributor

@dsmmcken dsmmcken commented Oct 4, 2024

User reported editor find contrast in dark mode was bad. Changed color for semantic editor highlight in dark mode to an orange similar to what monokai uses for find in vs code. Confirmed existing color in light mode was fine.

Before:
image

After:
image

@dsmmcken dsmmcken requested a review from bmingles October 4, 2024 14:36
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.42%. Comparing base (92b133b) to head (80a7590).

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #2248    +/-   ##
========================================
  Coverage   46.42%   46.42%            
========================================
  Files         698      698            
  Lines       38761    38761            
  Branches     9801     9680   -121     
========================================
  Hits        17993    17993            
- Misses      20714    20757    +43     
+ Partials       54       11    -43     
Flag Coverage Δ
unit 46.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Contributor

@bmingles bmingles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dsmmcken dsmmcken merged commit f8dd133 into main Oct 4, 2024
@dsmmcken dsmmcken deleted the dmckenzie_fix_highlight_color branch October 4, 2024 15:21
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants