issue reporter style fixes (#223579)

fix some styling issues also bug
microsoft · Jul 25, 2024 · 64b7b6d · 64b7b6d
1 parent 3cae7b1
commit 64b7b6d
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/src/vs/workbench/contrib/issue/browser/issue.ts b/src/vs/workbench/contrib/issue/browser/issue.ts
@@ -163,6 +163,7 @@ export class BaseIssueReporterService extends Disposable {
 
 		if (styles.backgroundColor) {
 			content.push(`.monaco-workbench { background-color: ${styles.backgroundColor} !important; }`);
+			content.push(`.issue-reporter-body::-webkit-scrollbar-track { background-color: ${styles.backgroundColor}; }`);
 		}
 
 		if (styles.inputBorder) {
@@ -204,16 +205,13 @@ export class BaseIssueReporterService extends Disposable {
 			content.push(`a:hover, .workbenchCommand:hover { color: ${styles.textLinkActiveForeground}; }`);
 		}
 
-		if (styles.sliderBackgroundColor) {
-			content.push(`::-webkit-scrollbar-thumb { background-color: ${styles.sliderBackgroundColor}; }`);
-		}
-
 		if (styles.sliderActiveColor) {
-			content.push(`::-webkit-scrollbar-thumb:active { background-color: ${styles.sliderActiveColor}; }`);
+			content.push(`.issue-reporter-body::-webkit-scrollbar-thumb:active { background-color: ${styles.sliderActiveColor}; }`);
 		}
 
 		if (styles.sliderHoverColor) {
-			content.push(`::--webkit-scrollbar-thumb:hover { background-color: ${styles.sliderHoverColor}; }`);
+			content.push(`.issue-reporter-body::-webkit-scrollbar-thumb { background-color: ${styles.sliderHoverColor}; }`);
+			content.push(`.issue-reporter-body::--webkit-scrollbar-thumb:hover { background-color: ${styles.sliderHoverColor}; }`);
 		}
 
 		if (styles.buttonBackground) {
@@ -419,6 +417,10 @@ export class BaseIssueReporterService extends Disposable {
 			if (issueType === IssueType.PerformanceIssue && this.receivedSystemInfo && this.receivedPerformanceInfo) {
 				return true;
 			}
+
+			if (issueType === IssueType.FeatureRequest) {
+				return true;
+			}
 		}
 
 		return false;

diff --git a/src/vs/workbench/contrib/issue/electron-sandbox/issueFormService.ts b/src/vs/workbench/contrib/issue/electron-sandbox/issueFormService.ts
@@ -96,6 +96,7 @@ export class IssueFormService2 implements IIssueFormService {
 
 			// Store into config object URL
 			this.configuration = {
+				zoomLevel: data.zoomLevel,
 				appRoot: this.environmentService.appRoot,
 				windowId: 0,
 				userEnv: {},

diff --git a/src/vs/workbench/contrib/issue/electron-sandbox/media/newIssueReporter.css b/src/vs/workbench/contrib/issue/electron-sandbox/media/newIssueReporter.css
@@ -359,6 +359,8 @@ body.issue-reporter-body {
 	cursor: pointer;
 	appearance: auto;
 	border: none;
+	border-right: 6px solid transparent;
+	padding-left: 10px;
 }
 
 .issue-reporter-body #similar-issues {