Skip to content

Reintroduce selection container's set_title/get_title methods #3471

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

Closed
jasongrout opened this issue May 25, 2022 · 2 comments · Fixed by #3477
Closed

Reintroduce selection container's set_title/get_title methods #3471

jasongrout opened this issue May 25, 2022 · 2 comments · Fixed by #3477
Milestone

Comments

@jasongrout
Copy link
Member

A conversation with @michaelp-db about stability of APIs inspired me to reconsider the set_title/get_title method deletion done in 91ba29d (#2746). It doesn't hurt us to have the set_title/get_title as convenience functions still, and will be one less problem for people to fix in moving to ipywidgets 8, so I'm in favor of adding back in the methods to preserve compatibility with 7.x code.

@jasongrout jasongrout added this to the 8.0 milestone May 25, 2022
@vidartf
Copy link
Member

vidartf commented May 30, 2022

+1 but maybe with a deprecation warning? (I forget how noisy deprecation warnings are these days...)

@jasongrout
Copy link
Member Author

jasongrout commented May 30, 2022

I don't see a point in doing a deprecation warning - I'm happy to support those two convenience functions going forward, as they don't hurt us and make it possible for the existing base to just keep using the code. In other words, I don't plan to remove them once we've reintroduced them.

jasongrout added a commit to jasongrout/ipywidgets that referenced this issue Jun 7, 2022
Also, we eagerly compute a title list that updates when the children are updated, so that .titles is always reasonable.

Fixes jupyter-widgets#3471.
jasongrout added a commit to jasongrout/ipywidgets that referenced this issue Jun 7, 2022
Also, we eagerly compute a title list that updates when the children are updated, so that .titles is always reasonable.

Fixes jupyter-widgets#3471.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants