Skip to content

Commit

Permalink
avoid top toolbar overflow
Browse files Browse the repository at this point in the history
  • Loading branch information
rebornix committed Aug 13, 2020
1 parent 4a0543f commit 728c1ed
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/vs/workbench/contrib/notebook/browser/media/notebook.css
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,10 @@
z-index: 100;
}

.monaco-workbench .notebookOverlay .cell-list-container {
overflow: hidden;
}

.monaco-workbench .notebookOverlay .cell-list-container .overflowingContentWidgets > div {
z-index: 600 !important;
/* @rebornix: larger than the editor title bar */
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1678,6 +1678,7 @@ export class NotebookEditorWidget extends Disposable implements INotebookEditor

this._localStore.clear();
this._list?.dispose();
this._listTopCellToolbar?.dispose();

this._overlayContainer.remove();
this.viewModel?.dispose();
Expand Down
2 changes: 2 additions & 0 deletions src/vs/workbench/contrib/notebook/test/testNotebookEditor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ import { IConfigurationService } from 'vs/platform/configuration/common/configur
import { TestConfigurationService } from 'vs/platform/configuration/test/common/testConfigurationService';
import { IThemeService } from 'vs/platform/theme/common/themeService';
import { TestThemeService } from 'vs/platform/theme/test/common/testThemeService';
import { ScrollEvent } from 'vs/base/common/scrollable';

export class TestCell extends NotebookCellTextModel {
constructor(
Expand Down Expand Up @@ -69,6 +70,7 @@ export class TestNotebookEditor implements INotebookEditor {
multipleKernelsAvailable: boolean = false;
onDidChangeAvailableKernels: Event<void> = new Emitter<void>().event;
onDidChangeActiveCell: Event<void> = new Emitter<void>().event;
onDidScroll = new Emitter<ScrollEvent>().event;

uri?: URI | undefined;
textModel?: NotebookTextModel | undefined;
Expand Down

0 comments on commit 728c1ed

Please sign in to comment.