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

Fix TabbedContent __init__ signature. #2497

Merged
merged 1 commit into from
May 8, 2023
Merged

Conversation

rodrigogiraoserrao
Copy link
Contributor

@rodrigogiraoserrao rodrigogiraoserrao commented May 5, 2023

This fixes the signature of TabbedContent.__init__ so that it is initialised like any other widget.

Of course, this PR only makes sense under the assumption that these kwargs were missing due to our oversight in implementing and reviewing TabbedContent.

@rodrigogiraoserrao
Copy link
Contributor Author

Once this has been merged, we should uncomment the middle sub-test in tests/test_on.py::test_on_arbitrary_attributes in 83b0ae7

@rodrigogiraoserrao rodrigogiraoserrao self-assigned this May 5, 2023
@rodrigogiraoserrao rodrigogiraoserrao added the bug Something isn't working label May 5, 2023
@willmcgugan willmcgugan merged commit 1e2f632 into main May 8, 2023
@willmcgugan willmcgugan deleted the fix-tabbed-content-init branch May 8, 2023 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants