1045769 - Adjust new tab whitespace: whitespace sizing, positioning of undo box

Status: VERIFIED FIXED

(Firefox :: New Tab Page, defect)

Description

[Ed Lee :Mardak]

phlsa suggests that with the larger tiles, some of the whitespace around the edges / around the tiles / around the undo box should be adjusted. This might also suggest a movement of the undo box to be closer to the gear/customize button freeing up some of the extra padding that goes around the box.

Comment 1

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

Here's the annotated screenshot for reference: http://cl.ly/image/2E3S082V1A2W

Comment 2

[Ed Lee :Mardak]

Is this something we want? If so, where should the undo box move to? Anchored to the top-right right next to the gear with some padding? Any minimum white space between the top of the search and the bottom of the undo box?

Comment 3

[Ed Lee :Mardak]

Notes from dcrobot:

Space above Search Bar = 35
Space below Search Bar = 70
Add 5 px spacing between Tiles and their Lables (i.e. "Amazon"), as well as 5 px between Tile rows

Comment 4

[Ed Lee :Mardak]

Attachment: before screenshot

Comment 5

[Ed Lee :Mardak]

Attachment: after screenshot

The increase tile spacing basically uses up all the whitespace gained from moving the undo box from the top middle. This won't help with showing more rows in a given screen height, but it will help things not feel so cramped.

Comment 6

[Ed Lee :Mardak]

Attachment: v1

Moves undo to top left avoiding some alignment issues with the customize button now/later.

Comment 7

[Drew Willcoxon :adw]

Comment on attachment 8472730 [details] [diff] [review]
v1

Review of attachment 8472730 [details] [diff] [review]:
-----------------------------------------------------------------

Nice, this is much nicer for the common case where the undo container is hidden.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7545b8222ecb

Comment 9

[Cornel Ionce [:noni]]

Attachment: whitespace_on_top_of_searchbar.png

While testing this on latest Nightly (build ID: 20140817030204) I've noticed that there still is a large white space between the undo box and the search bar compared to the "after" screenshot. Same for Ubuntu and Mac OS.

Is it ok to leave it like this? Should I file a separate bug?

Comment 10

[Ed Lee :Mardak]

(In reply to Cornel Ionce [QA] from comment #9)
> that there still is a large white space between the undo box and the search
> bar compared to the "after" screenshot. Same for Ubuntu and Mac OS.
What happens if you shrink the window height? What's the closest you can get the undo box to the search box before a row of tiles disappears (causing the whitespace to then increase).

Comment 11

[Drew Willcoxon :adw]

That looks as designed.  The undo box is supposed to stick to the top left corner now, and the search box + tiles should be better vertically centered than before with respect to the top and bottom of the window, which they are in your screenshot.

Comment 12

[Cornel Ionce [:noni]]

Attachment: whitespace_on_resize.png

If I shrink the window height then indeed the whitespace also shrinks along with it. The closest is shown in the attached screenshot and it seems ok.
Thank you!

Comment 13

[Sylvestre Ledru [:Sylvestre]]

Uplift has been managed in bug 1057602

Comment 14

[Ed Lee :Mardak]

https://hg.mozilla.org/releases/mozilla-aurora/rev/0b918191c65a

Comment 15

[Cornel Ionce [:noni]]

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:33.0) Gecko/20100101 Firefox/33.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:33.0) Gecko/20100101 Firefox/33.0
Mozilla/5.0 (X11; Linux i686; rv:33.0) Gecko/20100101 Firefox/33.0

Verified fixed on Windows 7 64bit, Mac OS X 10.8.5 and Ubuntu 14.04 32bit using latest Aurora, build ID: 20140828004002.