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

Empty toolbar not updating when switching to simple interface #9665

Closed
jasongrout opened this issue Jan 22, 2021 · 2 comments
Closed

Empty toolbar not updating when switching to simple interface #9665

jasongrout opened this issue Jan 22, 2021 · 2 comments
Labels
bug status:Needs Triage Applied to new issues that need triage status:Needs User Testing
Milestone

Comments

@jasongrout
Copy link
Contributor

jasongrout commented Jan 22, 2021

Description

In

#jp-main-dock-panel[data-mode='single-document']
.jp-MainAreaWidget
> .jp-Toolbar.jp-Toolbar-micro {
padding: 0;
min-height: 0;
}
we see that a "micro toolbar", i.e., an activity toolbar that has been collapsed because nothing is in it, is supposed to be completely collapsed in simple interface mode. Normally it has a height of 8px, which is set by a CSS variable:
min-height: var(--jp-toolbar-micro-height);
--jp-toolbar-micro-height: 8px;

However, it seems that at least Launcher widgets are not updated when switching to simple interface, so they retain the 8px micro toolbar. New launchers respect the 0px micro toolbar, so you see a bit of jumping when creating a new launcher. When you create a new launcher in the simple interface, it starts out with a 0px microtoolbar, but toggling simple interface off and then back on reverts things back to an 8px toolbar.

While this is sort of a small visual issue, it points to something not being refreshed when it should be refreshed. In general, widgets may change their appearance when in Simple Interface, and this is showing that there are problems right now with the update being called when switching to Simple Interface.

Reproduce

  1. Make sure you are not in Simple Interface, create a launcher. It helps to see how things are working if you resize the window to be narrow enough that the launcher has a scrollbar.
  2. Switch to simple interface. Note the toolbar space still is 8px high, visible as the space between the scrollbar and the top of the launcher panel.
  3. Create a new launcher panel. Notice that the scrollbar now goes up to the top of the panel (i.e., the 8px high toolbar space is gone)
  4. Toggle off simple interface, then back on. Notice that the toolbar now is 8px high.

Expected behavior

The toolbar to go to 0px in Simple Interface when I toggle Simple Interface.

Context

  • Operating System and version: macOS Catalina
  • Browser and version: Firefox 84
  • JupyterLab version: 3.0.5
@jasongrout jasongrout added this to the 3.1 milestone Jan 22, 2021
@jasongrout jasongrout modified the milestones: 3.1, 4.0 Jun 23, 2021
@JasonWeill JasonWeill modified the milestones: 4.0.0, 4.1.0 Feb 1, 2023
@krassowski krassowski added bug status:Needs Triage Applied to new issues that need triage labels Feb 5, 2024
@krassowski krassowski modified the milestones: 4.1.0, 4.1.x Feb 5, 2024
@krassowski
Copy link
Member

The toolbars were rewritten in 4.1, does it still appear in this release? Do you have a screenshot to demonstrate the issue?

@JasonWeill
Copy link
Contributor

@jasongrout @krassowski Any update about whether this still appears in 4.1.x?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug status:Needs Triage Applied to new issues that need triage status:Needs User Testing
Projects
None yet
Development

No branches or pull requests

3 participants