Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Woo POS][Design System] Update error icon size #15245

Merged
merged 11 commits into from
Feb 25, 2025

Conversation

jaclync
Copy link
Contributor

@jaclync jaclync commented Feb 24, 2025

Part of #15234

Description

This pull request focuses on improving the consistency of error icon sizes based on the new design guide 1qcjzXitBHU7xPnpCOWnNM-fi-1114_19201, plus spacing tweaks in its use cases. The key changes include the introduction of a new enumeration for error icon sizes and adjustments to spacing constants.

Introduction of POSErrorAndAlertIconSize Enumeration:

Updates to Error Icon Sizes:

Adjustments to Spacing and Layout:

Steps to reproduce

  • Turn off network connection
  • Enter POS with at least one variable product --> the product list loading error UI should match the design (except for the Products header)
  • Turn on network connection and reload item list
  • Turn off network connection
  • Tap on a variable product --> the variation list loading error UI should match the design
  • Go back to the product list, scroll to the bottom --> the infinite scroll error card should have the ! icon fitting the image size
  • Add item(s) to cart and check out --> the order sync error should match the design

Testing information

I tested the order validation error (< minimum payment amount), and payment error with an invalid order amount.

Screenshots

Item list loading error (design 1qcjzXitBHU7xPnpCOWnNM-fi-1076_15843):

products variations
IMG_0456 IMG_0457

Infinite scroll loading error:

Order sync error (design 1qcjzXitBHU7xPnpCOWnNM-fi-1096_16832) valiadation error
IMG_0458 IMG_0460

Payment error (design 1qcjzXitBHU7xPnpCOWnNM-fi-1096_18311) (just to ensure the icon size matches):


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

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:

  • The PR is small and has a clear, single focus, or a valid explanation is provided in the description. If needed, please request to split it into smaller PRs.
  • Ensure Adequate Unit Test Coverage: The changes are reasonably covered by unit tests or an explanation is provided in the PR description.
  • Manual Testing: The author listed all the tests they ran, including smoke tests when needed (e.g., for refactorings). The reviewer confirmed that the PR works as expected on all devices (phone/tablet) and no regressions are added. of

@jaclync jaclync added type: task An internally driven task. feature: POS labels Feb 24, 2025
@jaclync jaclync added this to the 21.9 milestone Feb 24, 2025
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Feb 24, 2025

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

App NameWooCommerce iOS WooCommerce iOS
Build Numberpr15245-e8f0665
Version21.8
Bundle IDcom.automattic.alpha.woocommerce
Commite8f0665
App Center BuildWooCommerce - Prototype Builds #13140
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@jaclync jaclync marked this pull request as ready for review February 24, 2025 06:25
@iamgabrielma iamgabrielma self-assigned this Feb 24, 2025
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.

LGTM! There is one small issue with the text on the "try again" button when infinite scroll fails, and a question:

  • Question: We use "retry" on the product/variations buttons while we've changed to "try again" in other places. Is this on purpose?

Simulator Screenshot - iPad Air 11 - iOS 17 5 M2 (US site) - 2025-02-24 at 13 18 10

  • "Try.." does seem to get cut off in any sizing (even smaller fonts):

Simulator Screenshot - iPad Air 11 - iOS 17 5 M2 (US site) - 2025-02-24 at 13 18 32

@jaclync
Copy link
Contributor Author

jaclync commented Feb 25, 2025

  • Question: We use "retry" on the product/variations buttons while we've changed to "try again" in other places. Is this on purpose?

Good point, I asked design in 1qcjzXitBHU7xPnpCOWnNM-fi-1076_15843#1146628827.

@jaclync
Copy link
Contributor Author

jaclync commented Feb 25, 2025

  • "Try.." does seem to get cut off in any sizing (even smaller fonts):

Good catch, I can reproduce and replaced the hard-coded max width with a modifier to fit width to content in e8f0665:

no accessibility font size biggest font size
Simulator Screenshot - iPad mini (A17 Pro) - 2025-02-25 at 09 07 14 Simulator Screenshot - iPad mini (A17 Pro) - 2025-02-25 at 09 07 26

@jaclync jaclync enabled auto-merge February 25, 2025 01:11
@jaclync jaclync merged commit 430e2a5 into trunk Feb 25, 2025
13 checks passed
@jaclync jaclync deleted the feat/15234-ds-design-review-error-icon-view branch February 25, 2025 01:28
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.

3 participants