-
Notifications
You must be signed in to change notification settings - Fork 4
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: ✨ merge next
into main
#1785
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: This PR is meant to cleanup the docs changelog which contains test entries. It also sets the package to its correct version. ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [ ] Documentation is created/updated - [ ] Migration Guide is created/updated - [ ] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [ ] Stories (features, a11y) are created/updated - [ ] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: <!-- *PR notes: Please describe the changes in this PR.* --> ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [ ] Documentation is created/updated - [ ] Migration Guide is created/updated - [ ] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [ ] Stories (features, a11y) are created/updated - [ ] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1502 This PR addresses: - Reordering of DOM elements so headline is read first on `sd-teaser` ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [x] Documentation is created/updated - [x] relevant tickets are linked --------- Co-authored-by: Mario Hamann <mario.hamann@virtual-identity.com>
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: This PR is meant to corrrectly publish the styles package into npm. ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [ ] Documentation is created/updated - [ ] Migration Guide is created/updated - [ ] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [ ] Stories (features, a11y) are created/updated - [ ] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1497 This PR addresses: - Implement `aria-orientation` on sd-divider ## Definition of Reviewable: - [x] E2E tests (features, a11y, bug fixes) are created/updated - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1496 #1500 This PR addresses: - Adapt sd-accordion HTML using `details` and `summary` elements - Adapt expandable sd-quickfact HTML using `details` and `summary` elements ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1709 ## Definition of Reviewable: - [x] Documentation is created/updated - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1477 ## Definition of Reviewable: - [x] E2E tests (features, a11y, bug fixes) are created/updated - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1505 This PR addresses: - sd-hidden-links behind header due to z-index issue on screenshot test. - added debug attribute to show sd-hidden-links on screenshot test. ## Definition of Reviewable: - [x] Stories (features, a11y) are created/updated - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: This PR addresses the missing translation of the input placeholder in the sd-select and sd-combobox components and closes [#998](#998) ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [ ] Documentation is created/updated - [ ] Migration Guide is created/updated - [x] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [ ] Stories (features, a11y) are created/updated - [ ] relevant tickets are linked --------- Co-authored-by: Mario Hamann <mario.hamann@virtual-identity.com>
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: This PR addresses a failing test in the combobox placeholder text. ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [ ] Documentation is created/updated - [ ] Migration Guide is created/updated - [x] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [ ] Stories (features, a11y) are created/updated - [ ] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: The PR contains breaking changes. The sd-badge variant attribute options have been renamed to remove any semantic meaning from them. Closes [#1593 ](#1593) ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [ ] Documentation is created/updated - [x] Migration Guide is created/updated - [x] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [ ] Stories (features, a11y) are created/updated - [ ] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: <!-- *PR notes: Please describe the changes in this PR.* --> Closes #1665 ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [x] Documentation is created/updated - [x] Migration Guide is created/updated - [x] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [x] Stories (features, a11y) are created/updated - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1518 This PR addresses: - Fixed issue with gradients on flipcard (noticed it when creating the template) - Implement flipcard template according to design ## Definition of Reviewable: - [x] Stories (features, a11y) are created/updated - [x] relevant tickets are linked --------- Co-authored-by: Mario Hamann <mario.hamann@virtual-identity.com>
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1318 This PR addresses: - sd-select default scrollbar appearing by default, even when unecessary ## Definition of Reviewable: - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1476 and #1778 This PR addresses: - Adjust sd-expandable spacings according to figma - Reorder of sd-expandable dom order. (button first so it is targetable by keyboard first) - Improvement of the sd-expandable template labels - Added accessibility hints to sd-expandable ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [x] Documentation is created/updated - [ ] Migration Guide is created/updated - [ ] E2E tests (features, a11y, bug fixes) are created/updated <!-- *If this PR includes a bug fix, an E2E test is necessary to verify the change. If the fix is purely visual, ensuring it is captured within our chromatic screenshot tests is sufficient.* --> - [x] Stories (features, a11y) are created/updated - [x] relevant tickets are linked --------- Co-authored-by: Mario Hamann <mario.hamann@virtual-identity.com>
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: <!-- *PR notes: Please describe the changes in this PR.* --> Closes #1511 This PR addresses: - Fix quote template a11y - Add accessibility hint on quote template - Add "profile" placeholder image ## Definition of Reviewable: - [x] Stories (features, a11y) are created/updated - [x] relevant tickets are linked
<!-- ## Title: Please consider adding the [skip chromatic] flag to the PR title in case you dont need chromatic testing your changes. --> ## Description: Closes #1504 This PR addresses: - [x] Default story headline changed to `h2`. - [x] Rephrase sd-headline stories description to explain why semantics and styles are disconnected - [x] Add a template to show real life examples from disconnected headlines ## Definition of Reviewable: <!-- *PR notes: Irrelevant elements should be removed.* --> - [x] Documentation is created/updated - [x] Stories (features, a11y) are created/updated - [x] relevant tickets are linked
🚀 Storybook has been deployed for branch |
🦋 Changeset detectedLatest commit: 97eba6a The changes in this PR will be included in the next version bump. 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 |
smfonseca
approved these changes
Jan 16, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.