Skip to content

Orders 7877 accessibility returns entry points - #2725

Merged
bc-vivekaggarwal merged 1 commit into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7877-accessibility-returns-entry-points
Aug 12, 2026
Merged

Orders 7877 accessibility returns entry points#2725
bc-vivekaggarwal merged 1 commit into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7877-accessibility-returns-entry-points

Conversation

@bc-vivekaggarwal

@bc-vivekaggarwal bc-vivekaggarwal commented Aug 12, 2026

Copy link
Copy Markdown

What?

Accessibility for returns entry points on order list and order details page

Requirements

  • CHANGELOG.md entry added (required for code changes only)

Tickets / Documentation

ORDERS-7877

Screenshots (if appropriate)

Before changes (Navigation goes to returns link first and then to order ID):
Screen.Recording.2026-08-12.at.1.42.06.pm.mov
After changes(UI remains same with fixes to aria-labels and navigation improvements):
image

Note

Low Risk
Template, CSS layout, and i18n-only accessibility fixes with no auth, data, or behavioral logic changes. Visual layout is preserved via flex, but focus order and screen-reader naming should be spot-checked.

Overview
Improves accessibility of returns entry points on the orders list and order details pages (ORDERS-7877).

On the orders list, the order title link is moved before the status/return block in the DOM so keyboard focus reaches the order first (WCAG 2.4.3), while a new .account-product-heading flex row keeps the visual layout. Return links on both the list and details pages get an order-scoped aria-label via the new return_items_for string.

Also clarifies in the accessibility skill that prefers-reduced-motion targets substantial/auto-playing motion, not transient loading spinners.

Reviewed by Cursor Bugbot for commit 3668a6d. Bugbot is set up for automated code reviews on this repo. Configure here.

@bc-vivekaggarwal
bc-vivekaggarwal requested a review from a team as a code owner August 12, 2026 03:43
@bc-vivekaggarwal
bc-vivekaggarwal changed the base branch from master to feature/PROJECT-7525-new-returns-flow August 12, 2026 03:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Improves accessibility for “Return Items” entry points on account order list and order details by ensuring a more logical focus/reading order and providing more descriptive, translatable accessible names for return links/buttons.

Changes:

  • Reorders the orders list DOM so the order title/link precedes the status/return action (better focus order), with SCSS updates to preserve the visual layout.
  • Adds unique aria-label values (via a new lang/en.json key) to “Return Items” links/buttons that include the order number.
  • Updates the in-repo accessibility skill documentation with reduced-motion guidance.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
templates/pages/account/orders/details.html Adds an order-specific aria-label to the return button on order details.
templates/components/account/orders-list.html Moves order title ahead of status/return action in DOM and adds an order-specific aria-label to the return link.
lang/en.json Introduces a new translatable string for the order-specific return aria-label.
assets/scss/components/stencil/account/_account.scss Adds an override class to keep the order status block positioned correctly after DOM reordering.
.claude/skills/accessibility/SKILL.md Adds guidance about prefers-reduced-motion expectations.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread assets/scss/components/stencil/account/_account.scss
Comment thread lang/en.json
@bc-vivekaggarwal
bc-vivekaggarwal force-pushed the ORDERS-7877-accessibility-returns-entry-points branch from b5887f4 to 2bf9552 Compare August 12, 2026 04:30

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2bf9552. Configure here.

Comment thread assets/scss/components/stencil/account/_account.scss
@bc-vivekaggarwal
bc-vivekaggarwal force-pushed the ORDERS-7877-accessibility-returns-entry-points branch from 2bf9552 to 3668a6d Compare August 12, 2026 07:13
@bc-vivekaggarwal
bc-vivekaggarwal merged commit 5b74b7d into feature/PROJECT-7525-new-returns-flow Aug 12, 2026
1 check passed
@bc-vivekaggarwal
bc-vivekaggarwal deleted the ORDERS-7877-accessibility-returns-entry-points branch August 12, 2026 23:58
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.

3 participants