Skip to content

fix(ColorScale): updated notations for colors that have hyphen in their names #3061

Merged
codecademydev merged 4 commits intomainfrom
kl-colortables-notation-fix
Apr 3, 2025
Merged

fix(ColorScale): updated notations for colors that have hyphen in their names #3061
codecademydev merged 4 commits intomainfrom
kl-colortables-notation-fix

Conversation

@LinKCoding
Copy link
Contributor

@LinKCoding LinKCoding commented Apr 3, 2025

Overview

Colors that have hyphens in name should use bracket notation.
Currently such colors are rendered like: theme.colors.for-example but should be theme.colors['for-example']

PR Checklist

  • I have run this code to verify it works

Testing Instructions

  1. Go to ColorMode, check that the colors that have hyphens in their name use bracket notation
  2. Go to the Theme pages, check that the colors in the ColorTable also use the appropriate notation
  3. ...
  4. Profit!

PR Links and Envs

N/A this PR affects Gamut only.

@nx-cloud
Copy link

nx-cloud bot commented Apr 3, 2025

View your CI Pipeline Execution ↗ for commit f8aa646.


☁️ Nx Cloud last updated this comment at 2025-04-03 16:16:33 UTC

@nx-cloud
Copy link

nx-cloud bot commented Apr 3, 2025

View your CI Pipeline Execution ↗ for commit f7e6dde.

Command Status Duration Result
nx run-many --target=build --all ✅ Succeeded 2s View ↗

☁️ Nx Cloud last updated this comment at 2025-04-03 15:23:36 UTC

@LinKCoding LinKCoding marked this pull request as ready for review April 3, 2025 15:46
@LinKCoding LinKCoding requested a review from a team as a code owner April 3, 2025 15:46
Copy link
Contributor

@aresnik11 aresnik11 left a comment

Choose a reason for hiding this comment

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

🔥

@codecademydev
Copy link
Collaborator

📬Published Alpha Packages:

@codecademy/styleguide@71.1.2-alpha.f8aa64.0

@codecademydev
Copy link
Collaborator

🚀 Styleguide deploy preview ready!

https://67eeb48d859f8b08f02fec09--gamut-preview.netlify.app

Deploy Logs

@LinKCoding LinKCoding added the Ship It :shipit: Automerge this PR when possible label Apr 3, 2025
@codecademydev codecademydev merged commit 6470234 into main Apr 3, 2025
20 of 21 checks passed
@codecademydev codecademydev removed the Ship It :shipit: Automerge this PR when possible label Apr 3, 2025
@codecademydev codecademydev deleted the kl-colortables-notation-fix branch April 3, 2025 16:28
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.

3 participants