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

fix: fix the issues caused by component rename #778

Merged
merged 2 commits into from
Aug 14, 2020

Conversation

jianliao
Copy link
Contributor

@jianliao jianliao commented Jul 30, 2020

Description

Recently component rename PRs: #751, #540 and Dialog change cause some issues. This PR is to fully apply the rename and fix the dialog examples.

Change summary:

  1. Fix spectrum-css documentation site
  2. Fix VRT
  3. Fix security vulnerability
  4. Change Divider version from 1.0.0-beta1 to 1.0.0-beta.1 for version consistency
  5. Modify diff.sh to include the renamed components (@lawdavis I am not sure if I do it right)
  6. Change a few class names for SearchWithin component (I am not sure if it is necessary. If not, I can revert them instantly)

How and where has this been tested?

  • How this was tested:
  • Browser(s) and OS(s) this was tested with:

Screenshots

To-do list

  • If my change impacts other components, I have tested to make sure they don't break.
  • If my change impacts documentation, I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • This pull request is ready to merge.

Sorry, something went wrong.

@adobe-spectrum-bot
Copy link
Collaborator

VRT successfully! 🎊

View the VRT result

@adobe-spectrum-bot
Copy link
Collaborator

Spectrum CSS documentation build successfully! 🎉

View the documentation

@adobe-spectrum-bot
Copy link
Collaborator

VRT successfully! 🎊

View the VRT result

@GarthDB GarthDB merged commit e1d180c into main Aug 14, 2020
@GarthDB GarthDB deleted the fix-component-rename-issue branch August 14, 2020 21:56
lazd pushed a commit that referenced this pull request Sep 22, 2020
* fix: fix the issues caused by component rename

* fix: fix package security vulnerabilities

Co-authored-by: Jian Liao <[email protected]>
GarthDB added a commit that referenced this pull request Sep 22, 2020
* feat: tags contributd by rsp@v3

* fix: move border-radius setting to index.css

* fix: add back the margin-inline-end for the tag has icon or avatar

* fix: fix truncation example for tag and fix clear button size

* fix: remove the margin-left for clearButton

* fix: adjust the color to match design for standard tag item

* fix: implements the background for clearButton

* fix: fix some mismatch between impl and design xd

* fix: mix tag with or without icon/avatar

* fix: use dna variables for border-shadow and border-size

* fix: update dna variable for focus-ring and border-size

* Performance for focus-ring
transition box-shadow is hard, opacity is easy

* build(deps-dev): bump prismjs from 1.17.1 to 1.21.0

Bumps [prismjs](https://github.com/PrismJS/prism) from 1.17.1 to 1.21.0.
- [Release notes](https://github.com/PrismJS/prism/releases)
- [Changelog](https://github.com/PrismJS/prism/blob/master/CHANGELOG.md)
- [Commits](PrismJS/prism@v1.17.1...v1.21.0)

Signed-off-by: dependabot[bot] <[email protected]>

* fix: fix the issues caused by component rename (#778)

* fix: fix the issues caused by component rename

* fix: fix package security vulnerabilities

Co-authored-by: Jian Liao <[email protected]>

* test: update VRT instruction documentation (#772)

* test: update VRT instruction documentation

* docs: fixed typo

Co-authored-by: Jian Liao <[email protected]>
Co-authored-by: Garth Braithwaite <[email protected]>

* fix: high contrast mode for checkboxes. (#791)

partly resolves #786

This will not work in IE11 or with our legacy build. It must support CSS vars.

Co-authored-by: Milan Brkic <[email protected]>
Co-authored-by: Jian Liao <[email protected]>
Co-authored-by: Rob Snow <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jian Liao <[email protected]>
Co-authored-by: Aaron <[email protected]>
Co-authored-by: James Nurthen <[email protected]>
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.

None yet

3 participants