Skip to content

fix: restore undefined custom properties [part 5] #3475

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

Merged
merged 13 commits into from
Jan 6, 2025

Conversation

5t3ph
Copy link
Contributor

@5t3ph 5t3ph commented Jan 3, 2025

Description

CSS-1087

Restore missing custom properties from the foundations spectrum two theme files that were flagged by the linter. This fixes linter errors for the following components:

  • Sidenav
  • Rating
  • Slider
  • Switch
  • Datepicker

How and where has this been tested?

Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.

Validation steps

@jawinn :

  • yarn linter sidenav no longer shows errors or warnings
  • yarn linter rating no longer shows errors or warnings
  • yarn linter slider no longer shows errors or warnings
  • yarn linter switch no longer shows errors or warnings
  • yarn linter datepicker no longer shows errors or warnings

To-do list

  • ✨ This pull request is ready to merge. ✨

dependabot bot and others added 5 commits January 2, 2025 10:57
Bumps [diff2html](https://github.com/rtfpessoa/diff2html) from 3.4.48 to 3.4.51.
- [Release notes](https://github.com/rtfpessoa/diff2html/releases)
- [Commits](rtfpessoa/diff2html@3.4.48...3.4.51)

---
updated-dependencies:
- dependency-name: diff2html
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore(deps-dev): bump lint-staged from 15.2.11 to 15.3.0

Bumps [lint-staged](https://github.com/lint-staged/lint-staged) from 15.2.11 to 15.3.0.
- [Release notes](https://github.com/lint-staged/lint-staged/releases)
- [Changelog](https://github.com/lint-staged/lint-staged/blob/master/CHANGELOG.md)
- [Commits](lint-staged/lint-staged@v15.2.11...v15.3.0)

---
updated-dependencies:
- dependency-name: lint-staged
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* build: update config tooling to resolve lint-staged correctly

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: [ Cassondra ] <castastrophe@users.noreply.github.com>
@5t3ph 5t3ph added the S2 Spectrum 2 label Jan 3, 2025
Copy link

changeset-bot bot commented Jan 3, 2025

🦋 Changeset detected

Latest commit: a8ffabc

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

This PR includes changesets to release 5 packages
Name Type
@spectrum-css/datepicker Patch
@spectrum-css/rating Patch
@spectrum-css/sidenav Patch
@spectrum-css/slider Patch
@spectrum-css/switch 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

Copy link
Contributor

github-actions bot commented Jan 3, 2025

🚀 Deployed on https://pr-3475--spectrum-css.netlify.app

Copy link
Contributor

github-actions bot commented Jan 3, 2025

File metrics

Summary

Total size: 1.73 MB*
Total change (Δ): 🔴 ⬆ 2.30 KB (0.13%)

Table reports on changes to a package's main file. Other changes can be found in the collapsed Details section below.

Package Size Δ
datepicker 12.94 KB 🔴 ⬆ 0.08 KB
rating 8.57 KB 🔴 ⬆ 0.42 KB
sidenav 14.46 KB 🔴 ⬆ 0.51 KB
slider 29.38 KB 🔴 ⬆ 1.38 KB
switch 24.46 KB 🟢 ⬇ 0.06 KB

Details

datepicker

Filename Head Compared to base
index.css 12.94 KB 🔴 ⬆ 0.08 KB (0.58%)

rating

Filename Head Compared to base
index.css 8.57 KB 🔴 ⬆ 0.42 KB (5.01%)

sidenav

Filename Head Compared to base
index.css 14.46 KB 🔴 ⬆ 0.51 KB (3.55%)

slider

Filename Head Compared to base
index.css 29.38 KB 🔴 ⬆ 1.38 KB (4.95%)

switch

Filename Head Compared to base
index.css 24.46 KB 🟢 ⬇ 0.06 KB (-0.25%)
* Size determined by adding together the size of the main file for all packages in the library.
* Results are not gzipped or minified.
* An ASCII character in UTF-8 is 8 bits or 1 byte.

@jawinn jawinn self-requested a review January 3, 2025 19:03
Bumps [gh-pages](https://github.com/tschaub/gh-pages) from 6.2.0 to 6.3.0.
- [Release notes](https://github.com/tschaub/gh-pages/releases)
- [Changelog](https://github.com/tschaub/gh-pages/blob/main/changelog.md)
- [Commits](tschaub/gh-pages@v6.2.0...v6.3.0)

---
updated-dependencies:
- dependency-name: gh-pages
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@rise-erpelding rise-erpelding self-requested a review January 6, 2025 14:34
Copy link
Collaborator

@rise-erpelding rise-erpelding left a comment

Choose a reason for hiding this comment

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

Looks great!

Noting that Datepicker still looks a little off, but it appears to be because of other components that weren't fixed in this PR such as popover, textfield, and action button.

dependabot bot and others added 6 commits January 6, 2025 09:40
Bumps [chromatic](https://github.com/chromaui/chromatic-cli) from 11.20.2 to 11.22.0.
- [Release notes](https://github.com/chromaui/chromatic-cli/releases)
- [Changelog](https://github.com/chromaui/chromatic-cli/blob/main/CHANGELOG.md)
- [Commits](chromaui/chromatic-cli@v11.20.2...v11.22.0)

---
updated-dependencies:
- dependency-name: chromatic
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…into seckles/css-1087-undef-props-part-5
@5t3ph 5t3ph merged commit 52e773e into spectrum-two Jan 6, 2025
12 checks passed
@5t3ph 5t3ph deleted the seckles/css-1087-undef-props-part-5 branch January 6, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants