Notebooks: Ensure WYSIWYG Mode for Keyboard Shortcuts (#14416)

* Ensure WYSIWYG Mode for kb shortcuts

* Move logic down to cell model
This commit is contained in:
Chris LaFreniere
2021-02-25 16:57:06 -08:00
committed by GitHub
parent e761eb12ef
commit dbc655a8f5
3 changed files with 24 additions and 3 deletions

View File

@@ -22,7 +22,7 @@ import { IMarkdownRenderResult } from 'vs/editor/browser/core/markdownRenderer';
import { NotebookMarkdownRenderer } from 'sql/workbench/contrib/notebook/browser/outputs/notebookMarkdown';
import { CellView } from 'sql/workbench/contrib/notebook/browser/cellViews/interfaces';
import { ICellModel } from 'sql/workbench/services/notebook/browser/models/modelInterfaces';
import { CellEditModes, ICellModel } from 'sql/workbench/services/notebook/browser/models/modelInterfaces';
import { NotebookModel } from 'sql/workbench/services/notebook/browser/models/notebookModel';
import { ISanitizer, defaultSanitizer } from 'sql/workbench/services/notebook/browser/outputs/sanitizer';
import { CodeComponent } from 'sql/workbench/contrib/notebook/browser/cellViews/code.component';
@@ -61,7 +61,7 @@ export class TextCellComponent extends CellView implements OnInit, OnChanges {
@HostListener('document:keydown', ['$event'])
onkeydown(e: KeyboardEvent) {
if (this.isActive()) {
if (this.isActive() && this.cellModel?.currentMode === CellEditModes.WYSIWYG) {
// select the active .
if ((e.ctrlKey || e.metaKey) && e.key === 'a') {
preventDefaultAndExecCommand(e, 'selectAll');