-
Notifications
You must be signed in to change notification settings - Fork 159
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(toc): remove implicit top and bottom padding for TOC #12144
Conversation
✅ Deploy Preview for ibm-dotcom-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ibm-dotcom-web-components-react-wrap 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.
Nice work Matt! Looks good to me!!
### Related Ticket(s) [ADCMS-7171](https://jsw.ibm.com/browse/ADCMS-7171) ### Description Removes the top and bottom implicit padding on the TOC contents in the shadow root. ### Changelog **Changed** - Removes the internal top and bottom padding from TOC contents to leave to adopter as in v1 <!-- React and Web Component deploy previews are enabled by default. --> <!-- To enable additional available deploy previews, apply the following --> <!-- labels for the corresponding package: --> <!-- *** "test: e2e": Codesandbox examples and e2e integration tests --> <!-- *** "package: services": Services --> <!-- *** "package: utilities": Utilities --> <!-- *** "RTL": React / Web Components (RTL) --> <!-- *** "feature flag": React / Web Components (experimental) -->
Hey there! This issue/pull request was referenced in recently released v2.16.1. |
Related Ticket(s)
ADCMS-7171
ADCMS-7077
Description
Removes the top and bottom implicit padding on the TOC contents in the shadow root.
Changelog
Changed