Skip to content

Fix accessibility issue for css variable highlight - #2844

Merged
Jake Bailey (jakebailey) merged 1 commit into
microsoft:v2from
jakebailey:fix-css-color
May 31, 2023
Merged

Fix accessibility issue for css variable highlight#2844
Jake Bailey (jakebailey) merged 1 commit into
microsoft:v2from
jakebailey:fix-css-color

Conversation

@jakebailey

Copy link
Copy Markdown
Member

For #2840

We wasted a bunch of time testing this before I stumbled on the problem that there's some cache somewhere that duplicates this theme, such that changing it didn't actually work. Cleaning the repo and reinstalling/rebuilding/etc actually shows that changing this does work.

This color upstream is https://github.com/shikijs/shiki/blob/main/packages/shiki/themes/light-plus.json#L294 (the same as what the accessibility team/tool say to use).

@jakebaileyJake Bailey (jakebailey) added the deploy-preview Enables automatic deployments to preview environments on a PR label May 31, 2023
@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-glacier-09aa73710-2844.centralus.azurestaticapps.net

@jakebailey

Copy link
Copy Markdown
MemberAuthor

Works:

image

@jakebailey
Jake Bailey (jakebailey) merged commit b7b80fe into microsoft:v2May 31, 2023
@jakebailey
Jake Bailey (jakebailey) deleted the fix-css-color branch May 31, 2023 18:47
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy-previewEnables automatic deployments to preview environments on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jakebailey@navya9singh