Skip to content

Conversation

@zippyzow
Copy link
Contributor

@zippyzow zippyzow commented Jan 8, 2026

Introduced a new CubeIllustration React component and exported it from the illustrations package index for use in mono.

Overview

PR Checklist

Testing Instructions

Don't make me tap the sign.

  1. Go to storybook illustrations
  2. See new CubeIllustration
  3. Review mono and monolith PRs linked below which have integrated these gamut changes
  4. Finish and do a celebratory dance

PR Links and Envs

Repository PR Link
Monolith Monolith PR
Mono Mono PR

Introduced a new CubeIllustration React component and exported it from the illustrations package index for use in other parts of the application.
@nx-cloud
Copy link

nx-cloud bot commented Jan 8, 2026

View your CI Pipeline Execution ↗ for commit 91f22ca


☁️ Nx Cloud last updated this comment at 2026-01-08 21:38:00 UTC

@zippyzow zippyzow changed the title Add CubeIllustration component feat(Illustration): ✨ Add CubeIllustration component Jan 8, 2026
@codecademydev
Copy link
Collaborator

📬 Published Alpha Packages:

@codecademy/[email protected]
@codecademy/[email protected]
@codecademy/[email protected]

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

@zippyzow zippyzow marked this pull request as ready for review January 9, 2026 18:41
@zippyzow zippyzow requested a review from a team as a code owner January 9, 2026 18:41
@zippyzow zippyzow requested review from aresnik11 and sh0ji January 9, 2026 18:41
Copy link
Contributor

@LinKCoding LinKCoding left a comment

Choose a reason for hiding this comment

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

Aside from the width, this looks good :)
Not a blocker, but maybe check with the designer if the dimensions are ok

className={className}
fill="none"
height={height}
viewBox="0 0 48 48"
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a quick note — that the width/height on the figma is slightly different:

Image

@zippyzow zippyzow added the Ship It :shipit: Automerge this PR when possible label Jan 12, 2026
@codecademydev codecademydev merged commit 348f3d3 into main Jan 12, 2026
43 of 44 checks passed
@codecademydev codecademydev removed the Ship It :shipit: Automerge this PR when possible label Jan 12, 2026
@codecademydev codecademydev deleted the hr-live-276-illustration branch January 12, 2026 14:25
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