Skip to content

Conversation

@MaxFrank13
Copy link
Member

@MaxFrank13 MaxFrank13 commented May 20, 2025

Changes

  • Upgrades peer dependency on @openedx/paragon to ^23.0.0 (BREAKING CHANGE)
  • Changes @edx/brand npm alias to refer to @edx/elm-theme in dev dependencies.
  • Example app index.scss file has been updated to pull from the new location of the CSS files for openedx/paragon and edx/elm-theme

How to test locally

  • Pull down this branch
  • Make sure you are logged in on your local instance
  • Run npm ci and npm start
  • Go to localhost:8080
  • To confirm that design tokens are being used, you can inspect the CSS for any component and look for CSS variables
  • Another giveaway that elm-theme is being used correctly would be to look at the button component for "Sign Up" -- it should have completely rounded edges as shown in the screenshot

Screenshot

New header:

Screenshot 2025-05-20 at 5 50 08 PM

Old header:

Screenshot 2025-05-21 at 11 26 44 AM

@MaxFrank13 MaxFrank13 changed the title feat!: upgrade to @openedx/paragon and design tokens feat!: upgrade to @openedx/paragon v23 and design tokens May 20, 2025
@MaxFrank13 MaxFrank13 force-pushed the mfrank/upgrade-to-paragon-23 branch from 2672ead to b17913a Compare May 20, 2025 22:00
@MaxFrank13 MaxFrank13 changed the title feat!: upgrade to @openedx/paragon v23 and design tokens feat: upgrade to @openedx/paragon v23 and design tokens May 23, 2025
@MaxFrank13 MaxFrank13 force-pushed the mfrank/upgrade-to-paragon-23 branch from 3d17660 to 3ebaddd Compare May 23, 2025 19:09
@adamstankiewicz adamstankiewicz changed the title feat: upgrade to @openedx/paragon v23 and design tokens feat!: upgrade to @openedx/paragon v23 and design tokens Jun 3, 2025
@adamstankiewicz adamstankiewicz merged commit cc31bed into master Jun 3, 2025
4 checks passed
@adamstankiewicz adamstankiewicz deleted the mfrank/upgrade-to-paragon-23 branch June 3, 2025 12: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