Add user setting for disabling comment max height (microsoft#185301)

Fixes microsoft#181846

Author: alexr00

src/vs/workbench/contrib/comments/browser/commentNode.ts

@@ -46,6 +46,7 @@ import { SmoothScrollableElement } from 'vs/base/browser/ui/scrollbar/scrollable
 import { DomEmitter } from 'vs/base/browser/event';
 import { CommentContextKeys } from 'vs/workbench/contrib/comments/common/commentContextKeys';
 import { FileAccess } from 'vs/base/common/network';
+import { COMMENTS_SECTION, ICommentsConfiguration } from 'vs/workbench/contrib/comments/common/commentsConfiguration';
 
 class CommentsActionRunner extends ActionRunner {
 	protected override async runAction(action: IAction, context: any[]): Promise<void> {
@@ -131,6 +132,9 @@ export class CommentNode<T extends IRange | ICellRange> extends Disposable {
 		this.createHeader(this._commentDetailsContainer);
 		this._body = document.createElement(`div`);
 		this._body.classList.add('comment-body', MOUSE_CURSOR_TEXT_CSS_CLASS_NAME);
+		if (configurationService.getValue<ICommentsConfiguration | undefined>(COMMENTS_SECTION)?.maxHeight !== false) {
+			this._body.classList.add('comment-body-max-height');
+		}
 
 		this.createScroll(this._commentDetailsContainer, this._body);
 		this.updateCommentBody(this.comment.body);

src/vs/workbench/contrib/comments/browser/comments.contribution.ts

@@ -39,6 +39,11 @@ Registry.as<IConfigurationRegistry>(ConfigurationExtensions.Configuration).regis
 			type: 'boolean',
 			default: true,
 			description: nls.localize('comments.visible', "Controls the visibility of the comments bar and comment threads in editors that have commenting ranges and comments. Comments are still accessible via the Comments view and will cause commenting to be toggled on in the same way running the command \"Comments: Toggle Editor Commenting\" toggles comments.")
+		},
+		'comments.maxHeight': {
+			type: 'boolean',
+			default: true,
+			description: nls.localize('comments.maxHeight', "Controls whether the comments widget scrolls or expands.")
 		}
 	}
 });

src/vs/workbench/contrib/comments/browser/media/review.css

@@ -136,6 +136,9 @@
 
 .review-widget .body .review-comment .review-comment-contents .comment-body {
 	padding-top: 4px;
+}
+
+.review-widget .body .review-comment .review-comment-contents .comment-body-max-height {
 	max-height: 20em;
 }
 

src/vs/workbench/contrib/comments/common/commentsConfiguration.ts

@@ -7,6 +7,7 @@ export interface ICommentsConfiguration {
 	openView: 'never' | 'file' | 'firstFile';
 	useRelativeTime: boolean;
 	visible: boolean;
+	maxHeight: boolean;
 }
 
 export const COMMENTS_SECTION = 'comments';