-
Notifications
You must be signed in to change notification settings - Fork 115
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
Conversation
…st error card use case with small size.
…se large size icon.
… use case to use large size icon.
…acing constants.
…ize icon along with other tweaks to match the latest design.
|
There was a problem hiding this 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?
"Try.."
does seem to get cut off in any sizing (even smaller fonts):
Good point, I asked design in 1qcjzXitBHU7xPnpCOWnNM-fi-1076_15843#1146628827. |
Good catch, I can reproduce and replaced the hard-coded max width with a modifier to fit width to content in e8f0665:
|
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:WooCommerce/Classes/POS/Utils/POSErrorAndAlertIconSize.swift
: Added a new enumerationPOSErrorAndAlertIconSize
to define small, medium, and large sizes for error and alert icons. This change standardizes the icon sizes across the application.Updates to Error Icon Sizes:
WooCommerce/Classes/POS/Presentation/Reusable Views/POSErrorExclamationMark.swift
: ModifiedPOSErrorExclamationMark
to use the newPOSErrorAndAlertIconSize
enumeration for defining icon sizes.WooCommerce/Classes/POS/Presentation/Reusable Views/POSErrorXMark.swift
: UpdatedPOSErrorXMark
to utilize thePOSErrorAndAlertIconSize
enumeration for setting the icon size.Adjustments to Spacing and Layout:
WooCommerce/Classes/POS/Presentation/CardReaderConnection/UI States/PointOfSaleItemListErrorView.swift
: Adjusted the spacing and layout inPointOfSaleItemListErrorView
to use consistent spacing constants and the new icon size.WooCommerce/Classes/POS/Presentation/CardReaderConnection/UI States/Reader Messages/PointOfSaleCardPresentPaymentLayout.swift
: Updated spacing constants inPointOfSaleCardPresentPaymentLayout
to ensure consistent layout.Steps to reproduce
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):
Infinite scroll loading error:
Payment error (design 1qcjzXitBHU7xPnpCOWnNM-fi-1096_18311) (just to ensure the icon size matches):
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: