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

Show the notification center if the status bar is disabled #15568

Merged
merged 1 commit into from Dec 27, 2023

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Dec 27, 2023

References

Fixes #15567

jupyterlab-notification-center-no-statusbar.webm

Code changes

If the status bar is not available, still position the notificationStatus widget on the page, so it can be used as the anchor for the popup.

User-facing changes

Notebook 7 users will be able to open the notification center via the same View > Show Notifications menu entry as in JupyterLab: jupyter/notebook#7195

Backwards-incompatible changes

None

@jtpio jtpio added the bug label Dec 27, 2023
Copy link

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

@jtpio
Copy link
Member Author

jtpio commented Dec 27, 2023

Labeling this PR as bug, as this fixes a behavior that might be confusing to users who have disabled the status bar.

@jtpio jtpio added this to the 4.1.0 milestone Dec 27, 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.

Thanks, @jtpio!

The failing usage2 tests are unrelated (#15570).

@krassowski krassowski merged commit 0a5e152 into jupyterlab:main Dec 27, 2023
77 of 79 checks passed
@jtpio jtpio deleted the notification-center-statusbar branch December 27, 2023 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show the notification center when the status bar plugin is disabled
2 participants