Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update shell-session default color #1893

Merged
merged 6 commits into from
Jan 19, 2024

Conversation

jorytindall
Copy link
Contributor

@jorytindall jorytindall commented Jan 18, 2024

📌 Summary

This is a draft PR to capture some of the different examples referenced in these Figma mockups around syntax highlighting for shell-session examples and bash examples.

Can be trashed, this is meant to be more representative.

🔗 External links

Jira ticket: HDS-2949
Figma file: CodeBlock Color Updates


👀 Component checklist

  • Percy was checked for any visual regression
  • A11y tests have been run locally (yarn test:a11y --filter="COMPONENT-NAME")
  • If documenting a new component, an acceptance test that includes the a11yAudit has been added
  • A changelog entry was added via Changesets if needed (see templates here)

💬 Please consider using conventional comments when reviewing this PR.

Copy link

vercel bot commented Jan 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Jan 19, 2024 6:28pm
hds-website ✅ Ready (Inspect) Visit Preview Jan 19, 2024 6:28pm

Copy link
Contributor

@didoo didoo left a comment

Choose a reason for hiding this comment

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

👍

@@ -50,7 +50,7 @@

// Syntax highlighting tokens:
// general ----
--hds-code-block-color-token: var(--hds-code-block-color-green);
--hds-code-block-color-token: var(--hds-code-block-color-foreground-primary);
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@jorytindall jorytindall marked this pull request as ready for review January 19, 2024 20:11
@KristinLBradley
Copy link
Contributor

I commented within the Figma file and within Jory's Slack thread

@jorytindall jorytindall merged commit 784ee4b into main Jan 19, 2024
15 checks passed
@jorytindall jorytindall deleted the jt/update-code-block-default-color branch January 19, 2024 21:12
@hashibot-hds hashibot-hds mentioned this pull request Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants