diff --git a/src/vs/code/electron-browser/issue/issueReporterMain.ts b/src/vs/code/electron-browser/issue/issueReporterMain.ts index 228ee2235746a1461b36a43d244b76a1b2d992c8..17633097592ae1faed4b6df94cff8685a19c5642 100644 --- a/src/vs/code/electron-browser/issue/issueReporterMain.ts +++ b/src/vs/code/electron-browser/issue/issueReporterMain.ts @@ -245,6 +245,24 @@ export class IssueReporter extends Disposable { }); }); + const labelElements = document.getElementsByClassName('caption'); + for (let i = 0; i < labelElements.length; i++) { + const label = labelElements.item(i); + 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] }); + } + }); + } + document.getElementById('reproducesWithoutExtensions').addEventListener('click', (e) => { this.issueReporterModel.update({ reprosWithoutExtensions: true }); }); diff --git a/src/vs/code/electron-browser/issue/issueReporterPage.ts b/src/vs/code/electron-browser/issue/issueReporterPage.ts index 14742da3b7362f75ce9c58b70fa09ef428bf6705..638a365632da93a68567f71b62d9e288580ae117 100644 --- a/src/vs/code/electron-browser/issue/issueReporterPage.ts +++ b/src/vs/code/electron-browser/issue/issueReporterPage.ts @@ -36,9 +36,8 @@ export default (): string => `
${escape(localize('systemInfo', "My System Info"))}
- - - + +
@@ -50,9 +49,8 @@ export default (): string => `
${escape(localize('processes', "Currently Running Processes"))}
- - - + +
@@ -64,9 +62,8 @@ export default (): string => `
${escape(localize('workspaceStats', "My Workspace Stats"))}
- - - + +
@@ -80,9 +77,8 @@ export default (): string => `
 				
${escape(localize('extensions', "My Extensions"))}
- - - + +
diff --git a/src/vs/code/electron-browser/issue/media/issueReporter.css b/src/vs/code/electron-browser/issue/media/issueReporter.css index bb951d46cba872c2b5298aec483639861fe4133e..3cc8d87604188df3fbf10adcfcd4edd919e2e16c 100644 --- a/src/vs/code/electron-browser/issue/media/issueReporter.css +++ b/src/vs/code/electron-browser/issue/media/issueReporter.css @@ -225,6 +225,7 @@ summary { width: 80px; top: 2px; left: 30px; + cursor: pointer; } .sendData { @@ -236,6 +237,7 @@ input[type="checkbox"] { display: inline-block; margin-top: 0; vertical-align: middle; + cursor: pointer; } input:disabled {