Skip to content

Conversation

@brichet
Copy link
Contributor

@brichet brichet commented Dec 7, 2023

This PR adds some documentation in the migration guide about the Toolbar and ToolbarButtonComponent selectors.

References

#15021 includes web components, which change the DOM of the Toolbar and ToolbarButtonComponent.

Code changes

None

User-facing changes

None

Backwards-incompatible changes

None

@jupyterlab-probot
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@krassowski krassowski added this to the 4.1.0 milestone Dec 8, 2023
Copy link
Member

@krassowski krassowski left a 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``.
Copy link
Member

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?

brichet and others added 3 commits December 11, 2023 09:06
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

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

Thank you @brichet!

@krassowski krassowski changed the title Update doc in migration guide about the Toolbarselector Update doc in migration guide about the Toolbar selector Dec 12, 2023
@krassowski krassowski changed the title Update doc in migration guide about the Toolbar selector Update migration guide about the Toolbar selector Dec 12, 2023
@krassowski krassowski merged commit e8d547d into jupyterlab:main Dec 12, 2023
@brichet brichet deleted the doc_toolbar_migration branch December 13, 2023 11:37
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants