Skip to content

Code tools shouldn’t muck with background color#1375

Merged
jjallaire merged 1 commit into
mainfrom
bugfix/codetools-bg
Jul 13, 2022
Merged

Code tools shouldn’t muck with background color#1375
jjallaire merged 1 commit into
mainfrom
bugfix/codetools-bg

Conversation

@dragonstyle
Copy link
Copy Markdown
Collaborator

@dragonstyle dragonstyle commented Jul 13, 2022

Since the code is being highlighted, it is possible that it will only be functional against a specifically colored background (especially dark or light). We should respect the background color if we’re going to highlight the code.

Additionally, since the code div is contained within a popup, remove any border radius so it fills the container edge to edge.

PR opened for discussion.

Screen Shot 2022-07-13 at 9 06 16 AM

Addresses #1370

Since the code is being highlighted, it is possible that it will only be functional against a specifically colored background (especially dark or light). We should respect the background color if we’re going to highlight the code.

Additionally, since the code div is contained within a popup, remove any border radius so it fills the container edge to edge.
@dragonstyle dragonstyle requested a review from jjallaire July 13, 2022 13:07
@jjallaire jjallaire merged commit 0744c02 into main Jul 13, 2022
@cderv cderv linked an issue Jul 13, 2022 that may be closed by this pull request
7 tasks
@cscheid cscheid deleted the bugfix/codetools-bg branch September 2, 2022 15:04
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.

Highlighting code-tools source code not legible with highlight-style: a11y-dark

2 participants