Skip to content

PivotGrid - A11y: show a full focus outline on the field panel fields - #34595

Open
aleksei-semikozov wants to merge 6 commits into
DevExpress:mainfrom
aleksei-semikozov:pivotgrid-field-focus-outline-4825
Open

PivotGrid - A11y: show a full focus outline on the field panel fields#34595
aleksei-semikozov wants to merge 6 commits into
DevExpress:mainfrom
aleksei-semikozov:pivotgrid-field-focus-outline-4825

Conversation

@aleksei-semikozov

Copy link
Copy Markdown
Contributor

No description provided.

@aleksei-semikozov
aleksei-semikozov marked this pull request as ready for review August 3, 2026 10:24
@aleksei-semikozov
aleksei-semikozov requested a review from a team as a code owner August 3, 2026 10:24
Copilot AI review requested due to automatic review settings August 3, 2026 10:24

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves PivotGrid keyboard accessibility by ensuring PivotGrid field panel items render a full, non-cropped focus outline when focused via keyboard navigation.

Changes:

  • Added a :focus-visible outline style for .dx-area-field so the focus indicator is drawn inside the field box.
  • Added an E2E TestCafe test validating the outline style and negative outline offset for keyboard-focused fields.

Reviewed changes

Copilot reviewed 2 out of 6 changed files in this pull request and generated 1 comment.

File Description
packages/devextreme-scss/scss/widgets/base/pivotGrid/_index.scss Adds an inner (outline-offset: -2px) focus-visible outline for PivotGrid field panel fields.
e2e/testcafe-devextreme/tests/common/pivotGrid/kbn/fields.ts Adds an E2E check to verify the focused field uses the intended outline styling.

Comment thread e2e/testcafe-devextreme/tests/common/pivotGrid/kbn/fields.ts Outdated
Copilot AI review requested due to automatic review settings August 3, 2026 10:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 6 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings August 3, 2026 19:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 5 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants