-
Notifications
You must be signed in to change notification settings - Fork 1
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
BC(web-react, web-twig): Remove isUnderlined from Link #DS-1509 #1693
BC(web-react, web-twig): Remove isUnderlined from Link #DS-1509 #1693
Conversation
✅ Deploy Preview for spirit-design-system-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for spirit-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change the PR title, it includes react too :)
.../web-twig/src/Resources/components/Breadcrumbs/__tests__/__fixtures__/breadcrumbsCustom.twig
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
f1e3577
to
ca1da98
Compare
- removing deprecation - solving #DS-1509
- removing deprecation - solving #DS-1509
ca1da98
to
e56bd3a
Compare
9b2178f
into
integration/BC-v3-design-tokens
Description
Additional context
Some changes were also needed in the Breadcrumbs component, which was using the Link component.
Issue reference
Remove
isUnderlined
prop from Link