Skip to content

FIX: Repair responsive target configuration#2224

Open
romanlutz wants to merge 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-fix-responsive-target-config
Open

FIX: Repair responsive target configuration#2224
romanlutz wants to merge 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-fix-responsive-target-config

Conversation

@romanlutz

Copy link
Copy Markdown
Contributor

Description

The target configuration workflow could become partially inaccessible when the available width was narrow or target registry names were long. The page-level action row pushed New Target outside a 390px viewport, while Round Robin controls could expand beyond the dialog.

This change makes the configuration shell shrink-safe, reflows header actions at narrow widths, and keeps horizontal scrolling localized to the target table. The Round Robin dialog now constrains native selects and selected-target rows, stacks controls on narrow screens, and preserves complete registry-name access through wrapping on mobile and a keyboard-focusable tooltip on desktop.

Mobile configuration at 390x844

Mobile target configuration with fully visible actions

Round Robin dialog with long registry names

Contained Round Robin dialog with accessible full-name tooltip

Tests and Documentation

  • Added React Testing Library coverage for keyboard-accessible long registry names and controls.
  • Added Playwright containment checks at 390x844 and 1280x800 for page actions, dialog width, long selections, and localized overflow.
  • Ran 79 focused Config unit tests.
  • Ran frontend TypeScript type checking and ESLint on the touched files.
  • Ran all four responsive Playwright scenarios.
  • Documentation: N/A; this is a responsive behavior fix with regression coverage.

Copilot AI added 2 commits July 17, 2026 15:54
Keep configuration actions and Round Robin target controls contained at narrow widths while preserving access to long registry names.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 07dadfa9-6127-4169-a9ae-8de30d98b59d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants