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

Allow passing of tag into widget constructor #150

Merged
merged 1 commit into from
Jan 14, 2021

Conversation

telamonian
Copy link
Member

This fix is related to jupyterlab/jupyterlab#9491

This PR adds tag to the IOptions object that can be passed into the widget constructor:

  • if node is unset and:
    • tag is unset: the widget's node will be a new div element (same as before)
    • tag is set: the widget's node will be a new element of tag tag
  • if node is set, tag will always be ignored, since no new element construction will occur

Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@blink1073 blink1073 merged commit 9c5f31c into jupyterlab:master Jan 14, 2021
@isabela-pf isabela-pf added this to Done in Accessibility Jan 28, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants