add title for color box (microsoft#19121)

GazimbeIsaac · Apr 15, 2022 · 5ec567a · 5ec567a
1 parent 794daa3
commit 5ec567a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/sql/base/browser/ui/colorbox/colorbox.ts b/src/sql/base/browser/ui/colorbox/colorbox.ts
@@ -41,6 +41,7 @@ export class Colorbox extends Widget {
 		}
 		if (opts.label) {
 			this.domNode.setAttribute('aria-label', opts.label);
+			this.domNode.title = opts.label;
 		}
 
 		container.appendChild(this.domNode);

diff --git a/src/sql/workbench/services/serverGroup/browser/serverGroupDialog.ts b/src/sql/workbench/services/serverGroup/browser/serverGroupDialog.ts
@@ -193,7 +193,7 @@ export class ServerGroupDialog extends Modal {
 			const colorColorBox = new Colorbox(container, {
 				name: 'server-group-color',
 				class: ['server-group-color'],
-				label: `Colobox Color: ${color}`,
+				label: localize('serverGroup.colorValue', "Color: {0}", color),
 			});
 
 			this._register(colorColorBox.onSelect((viaKeyboard) => {