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(web): Refactor Accordion styles to fully support theming via design tokens #DS-1074 #1162

Merged
merged 1 commit into from
Nov 23, 2023

Conversation

adamkudrna
Copy link
Contributor

@adamkudrna adamkudrna commented Nov 21, 2023

  • Header background is now always on the same (pseudo) element so nothing gets overlapped when colors change.
  • Stacking of header elements has been slightly refactored while retaining the intended behavior.
  • Component styles have been thoroughly documented so next time we don't have to spend a day on it.

https://jira.lmc.cz/browse/DS-1074


This PR replaces PR #1157.

@adamkudrna adamkudrna self-assigned this Nov 21, 2023
Copy link

netlify bot commented Nov 21, 2023

Deploy Preview for spirit-design-system-demo canceled.

Name Link
🔨 Latest commit 9106d00
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-demo/deploys/655f47077ee4770009c9e9c2

Copy link

netlify bot commented Nov 21, 2023

Deploy Preview for spirit-design-system-react ready!

Name Link
🔨 Latest commit 9106d00
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-react/deploys/655f4708d063ee00095738c1
😎 Deploy Preview https://deploy-preview-1162--spirit-design-system-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Nov 21, 2023

Deploy Preview for spirit-design-system-storybook ready!

Name Link
🔨 Latest commit 9106d00
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/655f4707d724020008d669dd
😎 Deploy Preview https://deploy-preview-1162--spirit-design-system-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the bug Something isn't working label Nov 21, 2023
@adamkudrna adamkudrna force-pushed the fix/accordion-theming branch 2 times, most recently from 72231ed to f137966 Compare November 21, 2023 15:31
Copy link

netlify bot commented Nov 21, 2023

Deploy Preview for spirit-design-system-validations canceled.

Name Link
🔨 Latest commit 9106d00
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-validations/deploys/655f47070f329e00086c9ea7

Copy link
Member

@crishpeen crishpeen left a comment

Choose a reason for hiding this comment

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

Nice one, thanks!

Copy link
Collaborator

@literat literat left a comment

Choose a reason for hiding this comment

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

I like the documentation as well. Helps a lot 👍

Copy link
Contributor

Coverage Status

coverage: 70.462% (-14.4%) from 84.868%
when pulling 008d5eb on fix/accordion-theming
into f700adf on main.

…sign tokens #DS-1074

* Header background is now always on the same element so nothing gets overlapped when colors change.
* Stacking of header elements has been slightly refactored while retaining the intended behavior.
* Component styles have been thoroughly documented so next time we don't have to spend a day on it.
@adamkudrna adamkudrna force-pushed the fix/accordion-theming branch from 008d5eb to 9106d00 Compare November 23, 2023 12:35
@adamkudrna adamkudrna merged commit ac0fec6 into main Nov 23, 2023
23 checks passed
@adamkudrna adamkudrna deleted the fix/accordion-theming branch November 23, 2023 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants