Closed Bug 1521583 Opened 6 years ago Closed 6 years ago

Incorrect vertically centred layout components

Categories

(Firefox :: New Tab Page, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 66
Iteration:
66.4 - Jan 21 - 27
Tracking Status
firefox66 --- fixed

People

(Reporter: thecount, Assigned: thecount)

References

Details

(Keywords: github-merged)

Attachments

(3 files)

If you have a single component in the left column and a bunch in the right, and the right is taller then the left, the items in the left get vertically centred.

Assignee: nobody → sdowne
Iteration: --- → 66.4 - Jan 21 - 27
Priority: -- → P1

Testing instructions.

  1. Ensure the new tab is the regular version by default.
  2. In about:config, change browser.newtabpage.activity-stream.discoverystream.config to be {"enabled":true,"layout_endpoint":"https://gist.githubusercontent.com/ScottDowne/981a4e3946df5b6a1e7622987007c88a/raw/ee48fa6e8cd6b76e53b29bcdb2deed1d48e199d7/horizontal-center"}
  3. Ensure new tab is rendering something like the screen shot I'm about to attach to this bug.
  4. Reset the pref, ensure everything is back to normal.
Attached image gridgap.png (deleted) —
Commit pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/85088466a8225ef2ddbfa4590060c35ca8e4e8a5
Bug 1521583 - vertical alignment of grid. (#4692)

* Bug 1521583 - vertical alignment of grid.

* Update vertical gap on layouts
Commit pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/29bc624b5bb0db7599d977f24fd3a571909b4fda
Followup Bug 1521583 - Fix css style regression from grid column alignment and add tests (#4694)
Blocks: 1521836
Keywords: github-merged
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Attached image QA Result : bug - 1521583.png (deleted) —

QA Results:

Tested on :

FF Nightly version : 66.0a1 (2019-01-25)
OS : Mac and Windows 10 Pro

Results attached QA Result : bug - 1521583.png

Closing as verified

Status: RESOLVED → VERIFIED
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: