Skip to content

Commit a2c84ee

Browse files
committed
fix(web): label the browser keybinding notice with its full message
1 parent 926de96 commit a2c84ee

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

‎apps/web/src/components/settings/KeybindingsSettings.tsx‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1288,18 +1288,21 @@ function KeybindingsGroups(props: KeybindingsGroupsProps) {
12881288

12891289
/** Shown in the browser build only; the desktop app receives every shortcut. */
12901290
function BrowserKeybindingNotice() {
1291+
// The label carries the whole sentence so assistive tech reads it without
1292+
// opening the tooltip.
1293+
const message = "The browser may claim some shortcuts first. The desktop app receives them all.";
12911294
return (
12921295
<Tooltip>
12931296
<TooltipTrigger
12941297
delay={200}
12951298
render={
1296-
<Button size="icon-micro" variant="ghost-muted" aria-label="Browser shortcut limits">
1299+
<Button size="icon-micro" variant="ghost-muted" aria-label={message}>
12971300
<TriangleAlertIcon className="size-3.5 text-warning" />
12981301
</Button>
12991302
}
13001303
/>
13011304
<TooltipPopup side="top" className="max-w-72">
1302-
The browser may claim some shortcuts first. The desktop app receives them all.
1305+
{message}
13031306
</TooltipPopup>
13041307
</Tooltip>
13051308
);

0 commit comments

Comments
 (0)