Skip to content

Conversation

@stevenle
Copy link
Member

Motivation

  • Prevent the field deeplink anchor from being included in keyboard tab navigation so it doesn't interrupt tab order.
  • Reduce spurious matches when using browser find (Ctrl+F) on the editor list by moving preview text out of the direct DOM text node.

Description

  • Set the deeplink anchor to tabIndex={-1} in packages/root-cms/ui/components/DocEditor/DocEditor.tsx so the # link is skipped by tab navigation.
  • Render the array item preview title using a data-preview attribute (data-preview={previewText}) instead of a direct text child in DocEditor.ArrayFieldPreview.
  • Add a CSS rule in packages/root-cms/ui/components/DocEditor/DocEditor.css that displays the preview via .DocEditor__ArrayField__item__header__preview__title::before { content: attr(data-preview); } to preserve visual output while removing the preview from the element's text node.

Testing

  • Ran git diff and git status to verify the changes were applied and staged, which succeeded.
  • Committed the changes with message fix(cms): improve keyboard/find behavior in doc editor, which succeeded.
  • Attempted an automated Playwright page screenshot to validate the UI, but the run failed due to no local web server responding (net::ERR_EMPTY_RESPONSE). No other automated tests were run per the request to avoid running tests unless explicitly prompted.

Codex Task

@stevenle stevenle changed the title fix(cms): improve keyboard/find behavior in doc editor chore: improve keyboard/find behavior in doc editor Feb 13, 2026
@stevenle stevenle force-pushed the codex/add-tabindex-to-deeplink-icon-and-exclude-from-ctrl+f branch from 4ee8b78 to d6158c4 Compare February 13, 2026 01:19
@stevenle stevenle merged commit 8d0a9a5 into main Feb 13, 2026
1 check passed
@stevenle stevenle deleted the codex/add-tabindex-to-deeplink-icon-and-exclude-from-ctrl+f branch February 13, 2026 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant