diff --git a/src/vs/code/electron-browser/issue/issueReporterMain.ts b/src/vs/code/electron-browser/issue/issueReporterMain.ts index 4edeea2666a..03f4c565032 100644 --- a/src/vs/code/electron-browser/issue/issueReporterMain.ts +++ b/src/vs/code/electron-browser/issue/issueReporterMain.ts @@ -318,18 +318,31 @@ export class IssueReporter extends Disposable { label.addEventListener('click', (e) => { e.stopPropagation(); - // Stop propgagation not working as expected in this case https://bugs.chromium.org/p/chromium/issues/detail?id=809801 - // preventDefault does prevent outer details tag from toggling, so use that and manually toggle the checkbox - e.preventDefault(); const containingDiv = (e.target).parentElement; const checkbox = containingDiv.firstElementChild; if (checkbox) { - checkbox.checked = !checkbox.checked; this.issueReporterModel.update({ [checkbox.id]: !this.issueReporterModel.getData()[checkbox.id] }); } }); } + const showInfoElements = document.getElementsByClassName('showInfo'); + for (let i = 0; i < showInfoElements.length; i++) { + const showInfo = showInfoElements.item(i); + showInfo.addEventListener('click', (e) => { + const label = (e.target); + const containingElement = label.parentElement.parentElement; + const info = containingElement.lastElementChild; + if (info.classList.contains('hidden')) { + show(info); + label.textContent = localize('hide', "hide"); + } else { + hide(info); + label.textContent = localize('show', "show"); + } + }); + } + this.addEventListener('issue-source', 'change', (event: Event) => { const fileOnExtension = JSON.parse((event.target).value); this.issueReporterModel.update({ fileOnExtension: fileOnExtension, includeExtensions: !fileOnExtension }); diff --git a/src/vs/code/electron-browser/issue/issueReporterPage.ts b/src/vs/code/electron-browser/issue/issueReporterPage.ts index 4991cd98b70..af7bfd77a12 100644 --- a/src/vs/code/electron-browser/issue/issueReporterPage.ts +++ b/src/vs/code/electron-browser/issue/issueReporterPage.ts @@ -48,89 +48,6 @@ export default (): string => ` -
-
-
- ${escape(localize('systemInfo', "My System Info"))} -
- - -
-
-
- -
-
-
-
-
- ${escape(localize('processes', "Currently Running Processes"))} -
- - -
-
-
-					
-				
-
-
-
-
- ${escape(localize('workspaceStats', "My Workspace Stats"))} -
- - -
-
-
-					
-						
-					
-				
-
-
-
-
- ${escape(localize('extensions', "My Extensions"))} -
- - -
-
-
- -
-
-
-
-
- ${escape(localize('searchedExtensions', "Searched Extensions"))} -
- - -
-
-
- -
-
-
-
-
- ${escape(localize('settingsSearchDetails', "Settings Search Details"))} -
- - -
-
-
- -
-
-
-
-
+
+
+ + + +
+
+ + + +
+
+ + + +
+
+ + + +
+
+ + + +
+
+ + + +
+
+ `; \ No newline at end of file diff --git a/src/vs/code/electron-browser/issue/media/issueReporter.css b/src/vs/code/electron-browser/issue/media/issueReporter.css index 7e7217fe2ba..ff907fc88eb 100644 --- a/src/vs/code/electron-browser/issue/media/issueReporter.css +++ b/src/vs/code/electron-browser/issue/media/issueReporter.css @@ -21,7 +21,7 @@ th { text-align: inherit; } td { - padding: .75rem; + padding: .25rem; vertical-align: top; } @@ -111,6 +111,10 @@ body { display: none; } +.block { + font-size: 12px; +} + .block .block-info { width: 100%; font-family: 'Menlo', 'Courier New', 'Courier', monospace; @@ -184,40 +188,17 @@ textarea { margin-top: 1em; } -.system-info { - margin-bottom: 1.25em; -} - select, input, textarea { border: 1px solid transparent; margin-top: 10px; } -summary { - border: 1px solid transparent; - padding: 0 10px; - margin-bottom: 5px; - cursor: pointer; -} .validation-error { font-size: 12px; margin-top: 1em; } -.include-data { - display: inline-block; -} - -.include-data > .caption { - display: inline-block; - font-size: 12px; - cursor: pointer; -} - -.sendData { - margin-left: 1em; -} input[type="checkbox"] { width: auto;