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

[PLAY-1414] ViteJS Update #3520

Merged
merged 66 commits into from
Aug 2, 2024
Merged

[PLAY-1414] ViteJS Update #3520

merged 66 commits into from
Aug 2, 2024

Conversation

thestephenmarshall
Copy link
Contributor

@thestephenmarshall thestephenmarshall commented Jul 11, 2024

What does this PR do?

  • Removes Webpack(er) from Playbook
  • Adds Vite for src and website
  • Simplifies the Playbook build process
  • Removes left-over babel plugins
  1. https://runway.powerhrg.com/backlog_items/PLAY-1413
  2. https://runway.powerhrg.com/backlog_items/PLAY-1414

How to test? Steps to confirm the desired behavior:

  1. Go to the review link
  2. Visually inspect visible pages (and test pages)
  3. Open the PR up locally and run it according to README.md instructions
  4. Verify local env successfully runs without additional input

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.
  • TESTS I have added test coverage to my code.

@thestephenmarshall thestephenmarshall self-assigned this Jul 11, 2024
@thestephenmarshall thestephenmarshall added the milano 20 MAX - Deploy this PR to a review environment via Milano label Jul 11, 2024
@thestephenmarshall thestephenmarshall marked this pull request as ready for review July 12, 2024 20:55
@thestephenmarshall thestephenmarshall added this pull request to the merge queue Aug 2, 2024
Merged via the queue into master with commit d3b2870 Aug 2, 2024
5 checks passed
@thestephenmarshall thestephenmarshall deleted the PLAY-1414 branch August 2, 2024 19:01
github-merge-queue bot pushed a commit that referenced this pull request Sep 6, 2024
…ding SCSS Variables (e.g., $z_10) (#3608)

**What does this PR do?**

- ✅ Fix `@import playbook.scss` errors. Allow theming (e.g., set $z_10).
- ✅ Remove [PLAY-1414](#3520)
`calc()` functions that causes undefined variable errors.
- ✅ Simplify by using standard SCSS calculations instead of calc()
throughout playbook
- ✅ Copy and use intl-tel-input node_modules CSS file so it works with
SCSS.
- ✅ Update "how to theme" doc
- ✅ Prepare rails-react-example for PB 14.0.0 upgrade (add nullish
coalescing operator to babel config)

**How to test?** Steps to confirm the desired behavior:
1. Go to /kits page
2. Review and look for major style breaking changes

#### Checklist:
- [x] **LABELS** Add a label: `enhancement`, `bug`, `improvement`, `new
kit`, `deprecated`, or `breaking`. See [Changelog &
Labels](https://github.com/powerhome/playbook/wiki/Changelog-&-Labels)
for details.
- [x] **DEPLOY** I have added the `milano` label to show I'm ready for a
review.

---------

Co-authored-by: Carlos Lima <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
milano 20 MAX - Deploy this PR to a review environment via Milano
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants