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: removing shadow-md token #963

Merged
merged 1 commit into from
Apr 30, 2024
Merged

Conversation

van-nguyen-ht
Copy link
Contributor

@van-nguyen-ht van-nguyen-ht commented Apr 18, 2024

Description:

following #573
shadow-md was created from the style importing from the old sketch file.
Since in the new design, it's visually sufficient for the map marker to consume the shadow token, this token is no longer valid and need to be deprecated at releasing

Definition of Reviewable:

PR notes: Irrelevant elements should be removed.

  • Documentation is created/updated
  • Migration Guide is created/updated
  • relevant tickets are linked

@van-nguyen-ht van-nguyen-ht self-assigned this Apr 18, 2024
@van-nguyen-ht van-nguyen-ht removed the request for review from mariohamann April 18, 2024 09:51
@van-nguyen-ht van-nguyen-ht added 🚧🛑🚧 waiting for DEV 🚧🛑🚧 Design is done but waiting for DEV to publish together 🔧 code needs changes in code labels Apr 18, 2024
@mariohamann mariohamann changed the title ci(token): Deprecating shadow-md token fix: Deprecating shadow-md token Apr 24, 2024
@mariohamann
Copy link
Contributor

This is a breaking change. Please ensure that when you finally commit.

@van-nguyen-ht
Copy link
Contributor Author

@mariohamann ma
oh it's only used for sd-map-marker in the past and now it's using shadow so I didn't think it's gonna be a big problem (?!)

@mariohamann
Copy link
Contributor

We don't know, if others used the shadow. As these tokens were public facing, it is a breaking change if it's removed now.

@mariohamann mariohamann changed the title fix: Deprecating shadow-md token fix: removing shadow-md token Apr 25, 2024
@van-nguyen-ht
Copy link
Contributor Author

@mariohamann it's alright I will add to release note :)

@karlbaumhauer karlbaumhauer merged commit c811e74 into main Apr 30, 2024
16 checks passed
@karlbaumhauer karlbaumhauer deleted the ci/deprecate-shadow-md-token branch April 30, 2024 12:48
karlbaumhauer pushed a commit that referenced this pull request Apr 30, 2024
# [@solid-design-system/tokens-v4.0.0](tokens/3.1.0...tokens/4.0.0) (2024-04-30)

### Bug Fixes

* removing shadow-md token ([#963](#963)) ([c811e74](c811e74)), closes [#573](#573)

### BREAKING CHANGES

* remove `shadow-md` token

following
van-nguyen-ht added a commit that referenced this pull request May 3, 2024
)

## Description:
link to ticket #963 
following
[#573](#573)
closes #573 
shadow-md was created from the style importing from the old sketch file.
Since in the new design, it's visually sufficient for the map marker to
consume the shadow token, this token is no longer valid and need to be
deprecated at releasing

## Definition of Reviewable:
*PR notes: Irrelevant elements should be removed.*
- [x] Documentation is created/updated
- [x] Migration Guide is created/updated
- [x] relevant tickets are linked
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧 code needs changes in code 🚧🛑🚧 waiting for DEV 🚧🛑🚧 Design is done but waiting for DEV to publish together
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants