Skip to content

feat(returns): ORDERS-7771 tighten spacing on mobile and tablet viewports#2694

Open
bc-bryan wants to merge 2 commits into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7771-lists-returns-page-mobile-tablet-views
Open

feat(returns): ORDERS-7771 tighten spacing on mobile and tablet viewports#2694
bc-bryan wants to merge 2 commits into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7771-lists-returns-page-mobile-tablet-views

Conversation

@bc-bryan

@bc-bryan bc-bryan commented Jul 1, 2026

Copy link
Copy Markdown

What?

Adds css changes for the mobile and tablet viewport so they layout per the figma reference.

Requirements

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

Tickets / Documentation

Add links to any relevant tickets and documentation.

Screenshots (if appropriate)

localhost_3000_account php_action=view_returns(iPad Mini) localhost_3000_account php_action=view_returns(Samsung Galaxy A51_71)

Note

Low Risk
Presentation-only SCSS in shared account components; no logic, auth, or data changes. Minor cross-page visual impact on other account lists using the same title classes.

Overview
Tightens account list row layout on small and medium viewports so the returns list matches the Figma reference, with a Draft CHANGELOG entry for ORDERS-7771.

In _account.scss, .account-listItem-summary uses a smaller flex gap and pushes list buttons to the trailing edge via margin-left: auto. .account-listItem-summary-main adds min-width: 0 on the product title and order status label so long text wraps instead of overflowing in flex rows.

.account-product-title is smaller on narrow screens (fontSize("smallest"), stepping up at the medium breakpoint) with adjusted bottom margin; .account-product-detail-heading spacing is slightly reduced. Because these selectors are shared account styles, the typography tweaks can show on other account list views that reuse the same classes—not only returns.

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

@bc-bryan bc-bryan requested a review from a team as a code owner July 1, 2026 09:15

@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 2 potential issues.

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 87f3cd3. Configure here.


@include breakpoint("medium") {
font-size: fontSize("smaller");
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Shared title styles widen scope

Medium Severity

Mobile/tablet font and margin updates on .account-product-title and .account-product-detail-heading apply account-wide, not only returns. Order list, order line items, and account messages inherit smaller titles and tighter detail labels though the change targets the returns list.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 87f3cd3. Configure here.

margin: 0 0 spacing("eighth");
text-transform: inherit;

margin: 0 0 spacing("quarter");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Duplicate margin property ignored

Low Severity

.account-product-title declares margin twice in a row; the first spacing("eighth") value is always overridden by spacing("quarter"), so the earlier rule is dead and easy to misread during later edits.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 87f3cd3. Configure here.

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.

1 participant