-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Update migration guide about the Toolbar selector #15501
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
Conversation
|
Thanks for making a pull request to jupyterlab! |
krassowski
left a comment
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.
Thank you, leaving one grammar suggestion, a note and a question.
| - The icon in the ``ToolbarButtonComponent`` is a direct child of the ``jp-button`` component. | ||
|
|
||
| The icon was previously encapsulated in a span with the class ``.jp-ToolbarButtonComponent-icon``. | ||
| Accessing that icon to change its properties require now something like ``jp-button > svg``. |
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.
What should we do about #15511?
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
krassowski
left a comment
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.
Thank you @brichet!
This PR adds some documentation in the migration guide about the
ToolbarandToolbarButtonComponentselectors.References
#15021 includes web components, which change the DOM of the
ToolbarandToolbarButtonComponent.Code changes
None
User-facing changes
None
Backwards-incompatible changes
None