Skip to content

resolve bg color for each color mode in the dom renderer#3849

Merged
meganrogge merged 3 commits intoxtermjs:masterfrom
meganrogge:bgColor-mode
Jun 2, 2022
Merged

resolve bg color for each color mode in the dom renderer#3849
meganrogge merged 3 commits intoxtermjs:masterfrom
meganrogge:bgColor-mode

Conversation

@meganrogge
Copy link
Copy Markdown
Member

fix #3846

Co-authored-by: Daniel Imms [email protected]

Co-authored-by: Daniel Imms <[email protected]>
@meganrogge meganrogge self-assigned this Jun 2, 2022
@meganrogge meganrogge added this to the 4.19.0 milestone Jun 2, 2022
@meganrogge meganrogge requested a review from Tyriar June 2, 2022 17:16
@meganrogge meganrogge enabled auto-merge (squash) June 2, 2022 17:35
@Tyriar
Copy link
Copy Markdown
Member

Tyriar commented Jun 2, 2022

Just need to flip the classes/styles around in the tests since we do bg first now

@meganrogge meganrogge merged commit bc9c9dd into xtermjs:master Jun 2, 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.

Minimum contrast ratio isn't working for inverted text in DOM renderer

2 participants