Add ARIA labels to all icon-only buttons for screen reader accessibility

[Copilot]

~200 <IconButton> components and several icon-only <button> elements across web/src/ lacked aria-label attributes, making them completely opaque to screen readers.

IconButton components (~130 instances across 50+ files)

Key areas fixed:

• ImageEditorToolbar — tool buttons (Select, Crop, Draw, Erase, Fill, Text, Rectangle, Ellipse, Line, Arrow), transform (Rotate CCW/CW, Flip H/V), zoom, undo/redo
• DataTable/TableActions — Add row, Delete/Duplicate rows, Reset sorting, Undo/Redo, Paste, Export CSV, Toggle select/row-numbers, Copy data
• VerticalToolbar / PanelLeft — all navigation toggle buttons
• NodeToolButtons / NodeHeader / NodePropertyForm — Run, Bypass, Comment, Duplicate, Info, Add input/output
• asset_viewer, hugging_face, properties, panels, chat, collections, workspaces — all remaining icon-only IconButtons

Dynamic labels used where toggle state changes meaning:

// Before
<IconButton onClick={handleToggleBypass}>
  <PauseCircleIcon />
</IconButton>

// After
<IconButton
  aria-label={isBypassed ? "Enable node" : "Bypass node"}
  onClick={handleToggleBypass}
>
  <PauseCircleIcon />
</IconButton>

Icon-only <button> elements

• AssetSearchInput — search mode toggle (dynamic), clear search
• TabsBar — scroll left/right, create new workflow
• FindInWorkflowDialog — clear, previous result, next result
• EditableTitle — remove note
• TextEditorModal / DataframeEditorModal — find, format, word wrap, editor mode, fullscreen, close
• DraggableNodeDocumentation — close

Utility component changes

Added an ariaLabel?: string prop to CircularActionButton and ToolbarIconButton to support cases where the tooltip prop is a ReactNode (not a plain string), which previously caused aria-label to resolve to undefined.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Add ARIA labels to all icon-only buttons for screen reader accessibility</issue_title>
<issue_description>## Problem
Many IconButton components lack aria-label attributes, making them inaccessible to screen readers. Users who rely on assistive technology cannot understand what these buttons do.

What to do

1. Search for all <IconButton components in web/src/ that don't have an aria-label prop
2. Add a descriptive aria-label to each one (e.g. aria-label="Close", aria-label="Delete item", aria-label="Toggle settings")
3. Also check for <button> elements that only contain an icon (no text) — these need labels too
4. Ensure interactive elements that look like buttons use <button> or have role="button" + tabIndex={0} + keyboard handlers

Guidelines

• Labels should be concise and action-oriented
• Use the button's actual function, not the icon name (e.g. "Close dialog" not "X icon")
• Don't add labels to buttons that already have visible text</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Add ARIA labels to all icon-only buttons for screen reader accessibility #2129

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[cloudflare-workers-and-pages]

Deploying nodetool-staging with    Cloudflare Pages

Latest commit:	579b9db
Status:	✅  Deploy successful!
Preview URL:	https://7e0e4b0e.nodetool-staging.pages.dev
Branch Preview URL:	https://copilot-add-aria-labels-to-i.nodetool-staging.pages.dev

View logs

[cloudflare-workers-and-pages]

Deploying nodetool with    Cloudflare Pages

Latest commit:	579b9db
Status:	✅  Deploy successful!
Preview URL:	https://2bbf9915.nodetool.pages.dev
Branch Preview URL:	https://copilot-add-aria-labels-to-i.nodetool.pages.dev

View logs

[georgi]

Review: This PR now conflicts with #2142 (merged), which added aria-label to the same 4 IconButtons in JSONProperty.tsx, StringProperty.tsx, DataframeProperty.tsx, and ConstantStringNode.tsx.

The rest of the changes (~130 additional IconButtons across 50+ files) look good and are much broader in scope. Please rebase on latest main to resolve the conflicts from #2142, then this can move forward.

[georgi]

Unable to rebase this branch automatically — it has merge conflicts with main (likely in the files that overlap with the already-merged #2142: JSONProperty.tsx, StringProperty.tsx, DataframeProperty.tsx, ConstantStringNode.tsx). I don't have push access to Copilot-owned branches. @copilot or @georgi please rebase this branch on latest main.

[Copilot]

Review: This PR now conflicts with #2142 (merged), which added aria-label to the same 4 IconButtons in JSONProperty.tsx, StringProperty.tsx, DataframeProperty.tsx, and `ConstantStringNod...

Merged origin/main into this branch in commit 579b9dbf. The 4 conflicting files (JSONProperty.tsx, StringProperty.tsx, DataframeProperty.tsx, ConstantStringNode.tsx) both had identical aria-label="Open Editor" additions — resolved by using main's attribute ordering. No functional difference.