Change:
window a comment :bug: as you can see from the scrollbar, the comment text doesn't fill the full editor !
<!-- UNABLETOLOCATECOMMITMESSAGE issue --Issue marked as unreleased but unable to locate closing commit in issue timeline. You can manually reference a commit by commenting , or directly add the label if you know this has already been releaased
closedWith

Discussion:
import { MarkdownRenderer } from 'vs/editor/contrib/markdownRenderer/browser/markdownRenderer'; import { IInstantiationService } from 'vs/platform/instantiation/common/instantiation'; import { ICommentService } from 'vs/workbench/contrib/comments/browser/commentService'; <del> import { LayoutableEditor, STARTING_EDITOR_HEIGHT, SimpleCommentEditor, calculateEditorHeight } from 'vs/workbench/contrib/comments/browser/simpleCommentEditor'; </del> <ins> import { LayoutableEditor, MIN_EDITOR_HEIGHT, SimpleCommentEditor, calculateEditorHeight } from 'vs/workbench/contrib/comments/browser/simpleCommentEditor'; </ins> import { Selection } from 'vs/editor/common/core/selection'; import { Emitter, Event } from 'vs/base/common/event'; import { INotificationService } from 'vs/platform/notification/common/notification';

private _commentEditor: SimpleCommentEditor | null = null; private _commentEditorDisposables: IDisposable[] = []; private _commentEditorModel: ITextModel | null = null; <del> private _editorHeight = STARTING_EDITOR_HEIGHT; </del> <ins> private _editorHeight = MIN_EDITOR_HEIGHT; </ins> private _isPendingLabel!: HTMLElement; private _timestamp: HTMLElement | undefined;

import { CommentContextKeys } from 'vs/workbench/contrib/comments/common/commentContextKeys'; import { ICommentThreadWidget } from 'vs/workbench/contrib/comments/common/commentThreadWidget'; import { ICellRange } from 'vs/workbench/contrib/notebook/common/notebookRange'; <del> import { LayoutableEditor, STARTING_EDITOR_HEIGHT, SimpleCommentEditor, calculateEditorHeight } from './simpleCommentEditor'; </del> <ins> import { LayoutableEditor, MIN_EDITOR_HEIGHT, SimpleCommentEditor, calculateEditorHeight } from './simpleCommentEditor'; </ins> const COMMENT_SCHEME = 'comment'; let INMEM_MODEL_ID = 0;

private _commentFormActions!: CommentFormActions; private _commentEditorActions!: CommentFormActions; private _reviewThreadReplyButton!: HTMLElement; <del> private _editorHeight = STARTING_EDITOR_HEIGHT; </del> <ins> private _editorHeight = MIN_EDITOR_HEIGHT; </ins> constructor( readonly owner: string,

import { ICodeEditor } from 'vs/editor/browser/editorBrowser'; export const ctxCommentEditorFocused = new RawContextKey<boolean>('commentEditorFocused', false); <del> export const STARTING_EDITOR_HEIGHT = 5 * 18; </del> <ins> export const MIN_EDITOR_HEIGHT = 5 * 18; </ins> export const MAX_EDITOR_HEIGHT = 25 * 18; export interface LayoutableEditor {

const lineHeight = editor.getOption(EditorOption.lineHeight); const contentHeight = (editor.getModel()?.getLineCount()! * lineHeight) ?? editor.getContentHeight(); // Can't just call getContentHeight() because it returns an incorrect, large, value when the editor is first created. if ((contentHeight > layoutInfo.height) || <del> (contentHeight < layoutInfo.height && currentHeight > STARTING_EDITOR_HEIGHT)) { </del> <ins> (contentHeight < layoutInfo.height && currentHeight > MIN_EDITOR_HEIGHT)) { </ins> const linesToAdd = Math.ceil((contentHeight - layoutInfo.height) / lineHeight); <del> const maxCommentEditorHeight = Math.max(Math.min(MAX_EDITOR_HEIGHT, parentEditor.getLayoutInfo().height - 90), STARTING_EDITOR_HEIGHT); const newEditorHeight = Math.min(maxCommentEditorHeight, layoutInfo.height + (lineHeight * linesToAdd)); return newEditorHeight; </del> <ins> const lowerBoundOfMaxEditorHeight = Math.max(parentEditor.getLayoutInfo().height - 90, MIN_EDITOR_HEIGHT); const maxCommentEditorHeightForParentEditor = Math.min(MAX_EDITOR_HEIGHT, lowerBoundOfMaxEditorHeight); const proposedHeight = layoutInfo.height + (lineHeight * linesToAdd); return Math.min(maxCommentEditorHeightForParentEditor, Math.max(MIN_EDITOR_HEIGHT, proposedHeight)); </ins> } return currentHeight; }