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

feat(pie-button): DSW-000 Changing styles to use classmap #1587

Merged
merged 3 commits into from
Jul 24, 2024

Conversation

ashleynolan
Copy link
Contributor

@ashleynolan ashleynolan commented Jul 10, 2024

Describe your changes (can list changeset entries if preferable)

  • [Changed] - Styles now use classmap
  • [Fixed] - Fixed background colour when button is in xsmall or small size and responsive mode is true, where the bg-color wasn't updating to orange as required

Author Checklist (complete before requesting a review)

  • I have performed a self-review of my code
  • I have reviewed the PIE Storybook PR preview
  • I have reviewed visual test updates properly before approving
  • If changes will affect consumers of the package, I have created a changeset entry.

Reviewer checklists (complete before approving)

Reviewer 1 - @xander-marjoram

  • I have reviewed the PIE Storybook PR preview

Reviewer 2

  • I have reviewed the PIE Storybook PR preview

Copy link

changeset-bot bot commented Jul 10, 2024

🦋 Changeset detected

Latest commit: f93ef1a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 12 packages
Name Type
@justeattakeaway/pie-button Minor
pie-storybook Patch
@justeattakeaway/pie-cookie-banner Patch
@justeattakeaway/pie-modal Patch
@justeattakeaway/pie-webc Patch
wc-angular12 Patch
wc-next10 Patch
wc-next13 Patch
wc-nuxt2 Patch
wc-nuxt3 Patch
wc-vanilla Patch
wc-vue3 Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ashleynolan ashleynolan marked this pull request as ready for review July 17, 2024 15:48
@ashleynolan ashleynolan requested review from a team as code owners July 17, 2024 15:48
@ashleynolan ashleynolan merged commit d3d974c into main Jul 24, 2024
40 of 42 checks passed
@ashleynolan ashleynolan deleted the dsw-000-classpoc branch July 24, 2024 14:49
JoshuaNg2332 pushed a commit that referenced this pull request Jul 25, 2024
* refactor(pie-button): DSW-000 updated styles to use classmap

* format(pie-button): DSW-000 formatting updates for PR comments

* fix(pie-button): DSW-000 change loading attribute to class
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.

4 participants