[Cards][Lists] Align source and context for SPOCs to bottom of the item container
Categories
(Firefox :: New Tab Page, defect, P1)
Tracking
()
People
(Reporter: wolasi, Assigned: pdahiya)
References
Details
(Keywords: github-merged)
Attachments
(2 files)
(deleted),
text/x-github-pull-request
|
Details | |
(deleted),
text/x-phabricator-request
|
lizzard
:
approval-mozilla-beta+
|
Details |
Please make the following changes for Cards and Lists
- The height of items in row should match the height of the tallest item in the row
- The context for SPOC items and source should be bottom aligned
- For lists items without excerpts the space between the title and the source (and context for SPOCs should be 4px
- For cards and list items with excerpts, the minimum spacing between the excerpt and the source (and context for SPOCs should be 8px
- For lists change square image width and height to 80px
Figma Spec Links
Reporter | ||
Updated•6 years ago
|
Comment hidden (obsolete) |
Updated•6 years ago
|
Reporter | ||
Updated•6 years ago
|
Comment 2•6 years ago
|
||
Comment 3•6 years ago
|
||
Looks like bug 1519879 regressed this for cards although lists never had source bottom aligned.
Updated•6 years ago
|
Comment 4•6 years ago
|
||
QA steps:
- use dev-test-all (
about:config
setbrowser.newtabpage.activity-stream.discoverystream.config
to{"api_key_pref":"extensions.pocket.oAuthConsumerKey","enabled":true,"show_spocs":true,"layout_endpoint":"https://getpocket.com/v3/newtab/layout?version=1&consumer_key=$apiKey&layout_variant=dev-test-all"}
) - open a new tab and make sure all source domain text is at the bottom left corner of each item
- also check list images are 80px tall
Comment 5•6 years ago
|
||
Comment 6•6 years ago
|
||
Comment 7•6 years ago
|
||
I have verified that this issue is no longer reproducible with the latest Firefox Nightly (67.0a1 Build ID - 20190217214556) installed, on Windows 10 x64, Arch Linux and Mac 10.13.3. Now, the source domain text is displayed in the left corner of each item and the check list images are 80px tall.
Comment 8•6 years ago
|
||
Comment on attachment 9044384 [details]
Bug 1527437 - [Cards][Lists] Align spocs items to bottom of container
Beta/Release Uplift Approval Request
Feature/Bug causing the regression
User impact if declined
User facing bug, user experience is affected
Is this code covered by automated tests?
No
Has the fix been verified in Nightly?
Yes
Needs manual test from QE?
No
If yes, steps to reproduce
QA done
List of other uplifts needed
Risk to taking this patch
Low
Why is the change risky/not risky? (and alternatives if risky)
Small change that has already been tested
String changes made/needed
Comment on attachment 9044384 [details]
Bug 1527437 - [Cards][Lists] Align spocs items to bottom of container
OK for uplift for beta 10, mostly css fixes, verified in nightly.
Comment 10•6 years ago
|
||
bugherder uplift |
Comment 11•6 years ago
|
||
(In reply to Marius Coman [:mcoman], Experiments QA from comment #7)
I have verified that this issue is no longer reproducible with the latest Firefox Nightly (67.0a1 Build ID - 20190217214556)
Could you please verify this issue on an early 66.0b10 build:
Windows 64: https://queue.taskcluster.net/v1/task/enVix5uvRveeQnNfxxl5mQ/runs/0/artifacts/public/build/target.zip
OS X: https://queue.taskcluster.net/v1/task/BbyPjtloRuCbHM2gX_9Hlw/runs/0/artifacts/public/build/target.dmg
Linux 64: https://queue.taskcluster.net/v1/task/CTU-aOfCSZe2B2tBVk_1GQ/runs/0/artifacts/public/build/target.tar.bz2
Comment 12•6 years ago
|
||
I have verified this issue with the latest Firefox Beta (66.0b10 Build ID - 20190219202808) installed, on Windows 10 x64, Arch Linux and Mac 10.13.3. Now, the source domain text is displayed in the left corner of each item and the check list images are 80px tall.
Updated•6 years ago
|
Updated•5 years ago
|
Description
•