react to @bpasero feedback

Author: isidorn

src/vs/base/browser/ui/actionbar/actionViewItems.ts

@@ -222,7 +222,7 @@ export class BaseActionViewItem extends Disposable implements IActionViewItem {
 			return;
 		}
 		const title = this.getTooltip() ?? '';
-		this.element.setAttribute('aria-label', title);
+		this.updateAriaLabel();
 		if (!this.options.hoverDelegate) {
 			this.element.title = title;
 		} else {
@@ -236,6 +236,13 @@ export class BaseActionViewItem extends Disposable implements IActionViewItem {
 		}
 	}
 
+	protected updateAriaLabel(): void {
+		if (this.element) {
+			const title = this.getTooltip() ?? '';
+			this.element.setAttribute('aria-label', title);
+		}
+	}
+
 	protected updateClass(): void {
 		// implement in subclass
 	}
@@ -392,8 +399,7 @@ export class ActionViewItem extends BaseActionViewItem {
 		}
 	}
 
-	override updateTooltip(): void {
-		super.updateTooltip();
+	override updateAriaLabel(): void {
 		if (this.label) {
 			const title = this.getTooltip() ?? '';
 			this.label.setAttribute('aria-label', title);