Skip to content

Comments

[WOOMOB-2274] Fix order detail shimmer loading screen to match loaded layout#15397

Open
malinajirka wants to merge 4 commits intotrunkfrom
woomob-2274-pos-booking-view-order-shimmer-loading-screen-doesnt-match
Open

[WOOMOB-2274] Fix order detail shimmer loading screen to match loaded layout#15397
malinajirka wants to merge 4 commits intotrunkfrom
woomob-2274-pos-booking-view-order-shimmer-loading-screen-doesnt-match

Conversation

@malinajirka
Copy link
Contributor

@malinajirka malinajirka commented Feb 20, 2026

Fixes WOOMOB-2274

Description

Updates the POS order detail shimmer loading screen to match the actual loaded order detail layout. The header structure now mirrors WooPosOrderDetails — action shimmer sits in a toolbar-height row at the top, with date/email/status shimmer below. In single-order mode (from bookings), the order number shimmer is hidden and the excessive top padding is removed so the action shimmer aligns with the toolbar. Totals card spacing is also updated to match the loaded state.

Test Steps

  1. Open POS
  2. Go to Bookings tab, select a booking, tap "View Order"
  3. Observe the shimmer loading screen — the action button shimmer should appear at the top (toolbar level), and the overall layout should closely match the loaded order detail
  4. Also verify the orders list shimmer still looks correct when opening the Orders tab (you might need to add delay to loading orders or just check out the video I shared)

Images/gif

Screen.Recording.2026-02-20.at.9.55.44.mov
  • I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary. Use the "[Internal]" label for non-user-facing changes.

@malinajirka malinajirka requested a review from kidinov February 20, 2026 08:41
@malinajirka malinajirka marked this pull request as draft February 20, 2026 08:41
@malinajirka malinajirka marked this pull request as ready for review February 20, 2026 08:58
@malinajirka malinajirka added this to the 24.2 milestone Feb 20, 2026
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Feb 20, 2026

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

App NameWooCommerce Android
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Build Number727
Version24.1-rc-3
Application IDcom.woocommerce.android.prealpha
Commitc89a120
Installation URL0n5to96bq755o
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.29%. Comparing base (ab7c42b) to head (1bf5f23).

Additional details and impacted files
@@            Coverage Diff            @@
##              trunk   #15397   +/-   ##
=========================================
  Coverage     39.29%   39.29%           
  Complexity    10977    10977           
=========================================
  Files          2238     2238           
  Lines        127540   127540           
  Branches      17802    17802           
=========================================
+ Hits          50113    50116    +3     
+ Misses        72318    72316    -2     
+ Partials       5109     5108    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@kidinov kidinov left a comment

Choose a reason for hiding this comment

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

LGTM!

I would just maybe consider hiding "Order #" behind shimmer as well, as we kinda of loading order number. Btw do we need to load it or we have the order id already?

Image

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.

4 participants