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

Support other design systems #442

Merged
merged 3 commits into from
Jul 18, 2023
Merged

Support other design systems #442

merged 3 commits into from
Jul 18, 2023

Conversation

tvararu
Copy link
Collaborator

@tvararu tvararu commented Jul 17, 2023

This is #441 but re-raised from a branch on this repo.

This is mostly identical to the work done in x-govuk/govuk-form-builder#127.

Commits kept separate for each individual component in case it makes it easier to review, but should be squashed up before merge.

Fixes #440.

TODO:

  • Fix links
  • Squash superfluous commits

@netlify
Copy link

netlify bot commented Jul 17, 2023

Deploy Preview for govuk-components ready!

Name Link
🔨 Latest commit 8d9651d
🔍 Latest deploy log https://app.netlify.com/sites/govuk-components/deploys/64b657d87588430008dcb526
😎 Deploy Preview https://deploy-preview-442--govuk-components.netlify.app/introduction/configuration
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tvararu tvararu force-pushed the config-brand branch 2 times, most recently from 6fd2a38 to 376f38b Compare July 17, 2023 12:36
@tvararu tvararu marked this pull request as ready for review July 17, 2023 12:36
@tvararu
Copy link
Collaborator Author

tvararu commented Jul 17, 2023

This seems to be working now for our app. PR is ready for review.

@tvararu tvararu requested a review from peteryates July 17, 2023 12:36
tvararu added 3 commits July 18, 2023 10:12
This allows users to change the `govuk` prefix for classes, to reuse the
components with other design systems.
This makes them whitelabelable and allows the use of other design
systems.
Update all the component tests to use the new example.
@tvararu
Copy link
Collaborator Author

tvararu commented Jul 18, 2023

Comments addressed and commits squashed into one.

Copy link
Member

@peteryates peteryates left a comment

Choose a reason for hiding this comment

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

Magnificent ✅

Great work, thank you @tvararu ❤️

@peteryates
Copy link
Member

It would be nice as a follow up to include a section in the guide showing off an NHS styled component. We can use the rendered_partial argument so we can render it in an iframe and not have to load two sets of CSS.

@tvararu tvararu added this pull request to the merge queue Jul 18, 2023
Merged via the queue into main with commit 7118e7b Jul 18, 2023
@tvararu tvararu deleted the config-brand branch July 18, 2023 15:54
@tvararu tvararu restored the config-brand branch July 19, 2023 10:35
@tvararu tvararu deleted the config-brand branch September 19, 2023 08:22
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.

Support other design systems
3 participants