Skip to content

Commit

Permalink
Removed toolbar scrollbar (#10790)
Browse files Browse the repository at this point in the history
* Removed toolbar scrollbar (#10189)

* Updated responsive toolbar styles to fit content

Co-authored-by: Piyush Jain <pijain@amazon.com>
  • Loading branch information
3coins and Piyush Jain committed Aug 13, 2021
1 parent c490c1e commit e66585b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 7 deletions.
5 changes: 3 additions & 2 deletions packages/notebook/style/toolbar.css
Expand Up @@ -36,9 +36,10 @@

.jp-Toolbar-responsive-popup {
position: absolute;
height: var(--jp-private-toolbar-height);
height: fit-content;
display: flex;
flex-direction: row;
flex-wrap: wrap;
justify-content: flex-end;
border-bottom: var(--jp-border-width) solid var(--jp-toolbar-border-color);
box-shadow: var(--jp-toolbar-box-shadow);
Expand All @@ -50,6 +51,6 @@
top: 0px;
}

.jp-toolbar > .jp-Toolbar-responsive-opener {
.jp-Toolbar > .jp-Toolbar-responsive-opener {
margin-left: auto;
}
1 change: 0 additions & 1 deletion packages/ui-components/src/components/toolbar.tsx
Expand Up @@ -161,7 +161,6 @@ export class Toolbar<T extends Widget = Widget> extends Widget {
constructor() {
super();
this.addClass(TOOLBAR_CLASS);
this.addClass('jp-scrollbar-tiny');
this.layout = new ToolbarLayout();
this.insertItem(
0,
Expand Down
4 changes: 0 additions & 4 deletions packages/ui-components/style/toolbar.css
Expand Up @@ -24,10 +24,6 @@
overflow-x: hidden;
}

.jp-Toolbar:hover {
overflow-x: auto;
}

/* Toolbar items */

.jp-Toolbar > .jp-Toolbar-item.jp-Toolbar-spacer {
Expand Down

0 comments on commit e66585b

Please sign in to comment.