Shopping card vertical padding is too small (should be 12px instead of 8px)
Categories
(Firefox :: Shopping, defect)
Tracking
()
People
(Reporter: Gijs, Assigned: Gijs)
References
Details
(Whiteboard: [fidefe-shopping])
Attachments
(2 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
diannaS
:
approval-mozilla-beta+
|
Details | Review |
We use 8px, and that is correct for the gap
between different items inside the card, but the card's outer padding should be 12px.
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Comment 1•1 year ago
|
||
Updated•1 year ago
|
Assignee | ||
Comment 2•1 year ago
|
||
Talking to Julian, there are 2 other things that UX would like to change here:
- the expandable headers with a button to the right (end) hand side should have 10px vertical padding and 10px end padding because the button otherwise makes the header "too tall" (or some alternative CSS solution that accomplishes the same thing, ie "pulls in" the vertical size of the header, to match the size it would have without the button, as well as "pulling it" closer to the end of the row)
- the "show more"/"show less" button should have an 8px gap above it instead of 12px.
I'll move the second one to bug 1856057.
Comment 4•1 year ago
|
||
bugherder |
Assignee | ||
Comment 5•1 year ago
|
||
Feedback from UX was that the button should not impact the vertical spacing of
the header text in the accordion cards, so this makes the button abspos
so it stops impacting the sizing of its container.
Original Revision: https://phabricator.services.mozilla.com/D189765
Updated•1 year ago
|
Comment 6•1 year ago
|
||
Uplift Approval Request
- Needs manual QE test: yes
- Fix verified in Nightly: no
- Explanation of risk level: CSS only patch
- Code covered by automated testing: no
- User impact if declined: visual design of review checker looks unpolished
- Risk associated with taking this patch: low
- String changes made/needed: no
- Steps to reproduce for manual QE testing: check the issues in comment 0 and 2 are addressed
- Is Android affected?: no
Comment 7•1 year ago
|
||
Comment on attachment 9356422 [details]
Bug 1856060 - increase shopping card vertical padding, r?#shopping-reviewers
Approved for 119.0b5
Updated•1 year ago
|
Updated•1 year ago
|
Comment 9•1 year ago
|
||
This issue is verified as fixed in our latest Nightly build 120.0a1 (2023-10-03) and our latest beta 119.0b5.
Description
•