woocommerce / woocommerce-android

WooCommerce Android app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
277 stars 135 forks source link

Purchase button adjustments #12939

Closed atorresveiga closed 21 hours ago

atorresveiga commented 4 days ago

Part of: #12281

Description

This PR includes the purchase button below the shipment details bottom sheet and improves the landscape support for the main shipping label screen by including the mark order complete control. The changes on this PR are only focused on the UI. Additionally, it updates the mark order complete caption as recommended by design.

The UI adjustments were discussed here: p1732055886685969-slack-C02KUCFCSFP

Testing information

  1. Create an Order with the processing status on a store that can create shipping labels
  2. Tap on Orders
  3. Tap on the order created on step 1
  4. Tap con create shipping labels
  5. Chek that the purchase button is displayed on the main screen
  6. check that the purchase button is displayed after expanding the shipment details section
  7. Change the phone configuration to landscape
  8. Check that on the landscape, the mark order complete switch is displayed
  9. Check that expanding/collapsing the shipment details section works as expected on landscape.

The tests that have been performed

Images/gif

https://github.com/user-attachments/assets/7a02c070-040f-4c43-a47d-417981f66199

Reviewer (or Author, in the case of optional code reviews):

Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement:

dangermattic commented 4 days ago
1 Warning
:warning: This PR is assigned to the milestone 21.2. 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 :no_entry_sign: Danger

wpmobilebot commented 4 days ago
📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
App Name WooCommerce-Wear Android
Platform⌚️ Wear OS
FlavorJalapeno
Build TypeDebug
Commit0c2943a729dc96741693cf4c19ba8d3fbd398a15
Direct Downloadwoocommerce-wear-prototype-build-pr12939-0c2943a.apk
wpmobilebot commented 4 days ago

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

App Name WooCommerce Android
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Commit0c2943a729dc96741693cf4c19ba8d3fbd398a15
Direct Downloadwoocommerce-prototype-build-pr12939-0c2943a.apk
codecov-commenter commented 21 hours ago

Codecov Report

Attention: Patch coverage is 0% with 18 lines in your changes missing coverage. Please review.

Project coverage is 39.77%. Comparing base (5b945a1) to head (0c2943a). Report is 10 commits behind head on trunk.

Files with missing lines Patch % Lines
...oid/ui/orders/wooshippinglabels/ShipmentDetails.kt 0.00% 17 Missing :warning:
...oid/ui/orders/wooshippinglabels/PurchaseSection.kt 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #12939 +/- ## ========================================= Coverage 39.76% 39.77% Complexity 5961 5961 ========================================= Files 1262 1262 Lines 72942 72935 -7 Branches 9976 9978 +2 ========================================= Hits 29009 29009 + Misses 41356 41349 -7 Partials 2577 2577 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features: