diff --git a/src/vs/code/electron-browser/issue/issueReporter.html b/src/vs/code/electron-browser/issue/issueReporter.html index 629aab5ea4447c8a53c35f6a61b4ba2961e2e2ca..695de78a4cb9e455069110872b7e640864cd0936 100644 --- a/src/vs/code/electron-browser/issue/issueReporter.html +++ b/src/vs/code/electron-browser/issue/issueReporter.html @@ -8,8 +8,7 @@ - - Issue Reporter + diff --git a/src/vs/code/electron-browser/issue/issueReporterMain.ts b/src/vs/code/electron-browser/issue/issueReporterMain.ts index 8ba1dd95e004df31fdc1f0704b276e3d5d372747..93be7de5db058896720022a99c97e964ec00ef1d 100644 --- a/src/vs/code/electron-browser/issue/issueReporterMain.ts +++ b/src/vs/code/electron-browser/issue/issueReporterMain.ts @@ -115,7 +115,8 @@ export class IssueReporter extends Disposable { } if (styles.inputErrorBorder) { - content.push(`.invalid-input, .invalid-input:focus { border: 1px solid ${styles.inputErrorBorder}; }`); + content.push(`.invalid-input, .invalid-input:focus { border: 1px solid ${styles.inputErrorBorder} !important; }`); + content.push(`.validation-error { color: ${styles.inputErrorBorder}; }`); } if (styles.inputActiveBorder) { @@ -256,8 +257,8 @@ export class IssueReporter extends Disposable { const processBlock = document.querySelector('.block-process'); const workspaceBlock = document.querySelector('.block-workspace'); - const descriptionTitle = document.querySelector('.block-description .block-title'); - const descriptionSubtitle = document.querySelector('.block-description .block-subtitle'); + const descriptionTitle = document.getElementById('issue-description-label'); + const descriptionSubtitle = document.getElementById('issue-description-subtitle'); // 1 - Bug if (issueType === 0) { diff --git a/src/vs/code/electron-browser/issue/issueReporterPage.ts b/src/vs/code/electron-browser/issue/issueReporterPage.ts index 2551db67e312fa87f57298395e68dd201a0695ca..5836f45bd1081dc8b364b3de24ed748380d54c88 100644 --- a/src/vs/code/electron-browser/issue/issueReporterPage.ts +++ b/src/vs/code/electron-browser/issue/issueReporterPage.ts @@ -40,7 +40,7 @@ export default (): string => ` -
+
${escape(localize('systemInfo', "My System Info"))} @@ -79,18 +79,19 @@ export default (): string => `
-
- - - - -
- ${escape(localize('githubMarkdown', "We support GitHub-flavored Markdown. You will still be able to edit your issue when we preview it on GitHub."))} - - -
+
+ +
+ + + + +
+ ${escape(localize('githubMarkdown', "We support GitHub-flavored Markdown. You will still be able to edit your issue when we preview it on GitHub."))} + +
diff --git a/src/vs/code/electron-browser/issue/media/issueReporter.css b/src/vs/code/electron-browser/issue/media/issueReporter.css index d7e3c8ec2d8f96a6dbcb902d42d2af39c01aaaf0..1444e6cd9446a165bf7f81350721bbe6163400db 100644 --- a/src/vs/code/electron-browser/issue/media/issueReporter.css +++ b/src/vs/code/electron-browser/issue/media/issueReporter.css @@ -104,14 +104,7 @@ body { } #block-container { - margin-top: 20px; -} - -.block { - margin-bottom: 20px; -} -.block summary { - margin-bottom: 16px; + margin-top: 1em; } .block .block-info { @@ -170,23 +163,29 @@ select, input, textarea { summary { border: 1px solid transparent; + padding: 10px; + margin-bottom: 5px; } .validation-error { font-size: 12px; - font-weight: bold; margin-top: 1em; } .caption { display: inline-block; font-size: 12px; + vertical-align: middle; + height: 18px; } input[type="checkbox"] { margin-left: 1em; + height: 18px; width: auto; display: inline-block; + margin-top: 0; + vertical-align: middle; } input:disabled { @@ -213,7 +212,7 @@ a { border: 1px solid #be1100; } -.required-input { +.required-input, .validation-error { color: #be1100; }