Skip to content

Conversation

@staskus
Copy link
Contributor

@staskus staskus commented Sep 30, 2025

Description

Figma: NFWCJetWSgHq2jPLKjpKYw-fi-8027_1152
WOOMOB-1142

One review is enough.

Updating the order list and order details UI based on the designer updates.

Changes:

  • Order List row view arrangement, font sizes, order status badge color, ghost rows
  • Order Details header arrangement, action placement, card spacing, font sizes, and dividers

Better VoiceOver support updates will be done in another PR.

Steps to reproduce

  1. Open POS -> Menu -> Orders
  2. Confirm ghost rows appear for both list and details
  3. Confirm the view appears according to designs NFWCJetWSgHq2jPLKjpKYw-fi-8027_1152
  4. Check light & dark mode, larger and smaller font sizes

Testing information

Tested on iPad Air 11' 26.0 simulator

Screenshots

https://github.com/user-attachments/assets/41b7cca5-ac5d-49da-8b8a-36bd842149e5
https://github.com/user-attachments/assets/25dbbf06-bd5a-442c-a0a1-859cf2c0331e


  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Added makePreviewOrderWithRefund() method to support preview of order details with refunds including formatted net amount.
Modified header title format to show order number directly and moved email receipt action to trailingContent. Updated back button to use xmark icon and apply POSSpacing.none.
Removed separate actions section since email receipt is now in header. Updated scroll view padding to use POSPadding constants and changed background to posSurfaceBright.
Moved order badge from trailingContent to bottomContent within header with proper spacing and top padding.
Updated products section to include dividers between line items with enumerated array and conditional divider display.
Changed product row alignment from top to center and removed vertical padding for cleaner layout.
Removed product attributes display, changed spacing to none, and updated fonts to use posBodySmallBold and posBodySmallRegular for consistency.
Changed totals section spacing to small and replaced hardcoded divider with reusable divider component for consistency.
Updated all totals row components to use posBodySmallBold/Regular fonts, proper colors, and consistent spacing. Added dividers between refunds and updated payment method font to posCaptionRegular.
Removed old actions section code and added reusable divider ViewBuilder with proper styling and spacing.
Added second preview showing order details with refund scenario to test refund display functionality.
@staskus staskus added this to the 23.4 milestone Sep 30, 2025
@staskus staskus added type: task An internally driven task. feature: POS labels Sep 30, 2025
@dangermattic
Copy link
Collaborator

dangermattic commented Sep 30, 2025

2 Warnings
⚠️ This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.
⚠️ This PR is assigned to the milestone 23.4. This milestone is due in less than 2 days.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Sep 30, 2025

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Numberpr16184-1718654
Version23.3
Bundle IDcom.automattic.alpha.woocommerce
Commit1718654
Installation URL0gveg787ojhb8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@iamgabrielma iamgabrielma self-assigned this Oct 1, 2025
@iamgabrielma
Copy link
Contributor

@staskus a couple of hours after this was submitted for review, this was published: pdfdoF-7Av-p2 . Let me know if I should just go ahead and review the PR or wait 🙇

@staskus
Copy link
Contributor Author

staskus commented Oct 1, 2025

@iamgabrielma I did these updates according to the new designs but I will then add a few more details so it would be full in this PR.

@staskus staskus marked this pull request as draft October 1, 2025 11:10
@staskus staskus marked this pull request as ready for review October 1, 2025 14:53
Copy link
Contributor

@iamgabrielma iamgabrielma left a comment

Choose a reason for hiding this comment

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

Looks great! Well done! :shipit:

@staskus staskus enabled auto-merge October 2, 2025 06:47
@staskus staskus merged commit b40ac0f into trunk Oct 2, 2025
13 checks passed
@staskus staskus deleted the woomob-1142-woo-poshistorical-orders-order-list-designer-ui branch October 2, 2025 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: POS type: task An internally driven task.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants