Fix microsoft#52562, Process Explorer: NVDA reads the whole table dim…

…ension every time before reading each cell value
jvasileff · Jan 25, 2019 · a94b94f · a94b94f
1 parent 770beca
commit a94b94f
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/src/vs/code/electron-browser/processExplorer/media/processExplorer.css b/src/vs/code/electron-browser/processExplorer/media/processExplorer.css
@@ -56,7 +56,7 @@ table {
 	width: 100%;
 	table-layout: fixed;
 }
-th {
+th[scope='col'] {
 	vertical-align: bottom;
 	border-bottom: 1px solid #cccccc;
 	padding: .5rem;
@@ -80,6 +80,9 @@ td {
 .data {
 	white-space: pre;
 	padding-left: .5rem;
+	font-weight: normal;
+	text-align: left;
+	height: 22px;
 }
 
 tbody > tr:hover {

diff --git a/src/vs/code/electron-browser/processExplorer/processExplorer.html b/src/vs/code/electron-browser/processExplorer/processExplorer.html
@@ -8,7 +8,7 @@
 </head>
 
 <body aria-label="">
-	<table id="process-list" aria-live="polite"></table>
+	<table id="process-list"></table>
 </body>
 
 <!-- Startup via processExplorer.js -->

diff --git a/src/vs/code/electron-browser/processExplorer/processExplorerMain.ts b/src/vs/code/electron-browser/processExplorer/processExplorerMain.ts
@@ -152,7 +152,8 @@ function updateProcessInfo(processList): void {
 		pid.classList.add('centered');
 		pid.textContent = p.pid;
 
-		const name = document.createElement('td');
+		const name = document.createElement('th');
+		name.scope = 'row';
 		name.classList.add('data');
 		name.title = p.cmd;
 		name.textContent = p.formattedName;