Closed
Bug 1016366
Opened 10 years ago
Closed 10 years ago
Tiles in new tab page are too small on higher resolutions
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
RESOLVED
WONTFIX
Tracking | Status | |
---|---|---|
firefox30 | --- | unaffected |
firefox31 | - | wontfix |
firefox32 | - | wontfix |
People
(Reporter: bugzilla, Unassigned)
References
Details
(Keywords: regression, uiwanted)
Attachments
(3 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
Build ID: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:32.0) Gecko/20100101 Firefox/32.0
Nightly builds new tab page added a search bar, but it also made the tiles much smaller resulting in lots of unused white space. See attached screenshots.
Reporter | ||
Comment 1•10 years ago
|
||
Reporter | ||
Comment 2•10 years ago
|
||
What this bug requests is to make better use of the screen in higher resolutions like older versions of Firefox.
Updated•10 years ago
|
Blocks: 978338
Status: UNCONFIRMED → NEW
tracking-firefox31:
--- → ?
Ever confirmed: true
Keywords: regression
OS: Mac OS X → All
Hardware: x86 → All
Comment 3•10 years ago
|
||
Dão did this ever present differently in previous versions on hi-dpi screens? It looks like this is an enhancement request more than a regression unless it did and if so, how far back?
Flags: needinfo?(dao)
Reporter | ||
Comment 4•10 years ago
|
||
Just compare the screenshots to see how it looked before in hi-dpi screens. This is a regression since it used to work well, but not anymore since bug 978338 was checked in.
Comment 5•10 years ago
|
||
According to bug 978338, this is potentially a regression in Firefox 31. Given the possibility of regression, I'm flagging this for tracking 31. I'd like to get to the bottom of the expected result of the change in bug 978338.
Cornel - You represented QA on bug 978338. Can you comment on how that bug was verified given the finding in this bug? Is the smaller page expected for some reason?
status-firefox30:
--- → unaffected
status-firefox31:
--- → affected
status-firefox32:
--- → affected
tracking-firefox32:
--- → +
Flags: needinfo?(cornel.ionce)
Updated•10 years ago
|
Flags: needinfo?(dao)
Comment 6•10 years ago
|
||
The smaller page is expected because the tiles have a fixed ratio now (243x150 px) as you may see in bug 975208 (attachment https://bug975208.bugzilla.mozilla.org/attachment.cgi?id=8385835). I'm afraid this is not going to change due to this fact.
As a workaround, you could change the default number of rows/columns in order to see more tiles and so, you won't have so much unused space, with the following prefs from about:config
- browser.newtabpage.columns
- browser.newtabpage.rows
Flags: needinfo?(cornel.ionce)
Reporter | ||
Comment 7•10 years ago
|
||
(In reply to Cornel Ionce [QA] from comment #6)
> The smaller page is expected because the tiles have a fixed ratio now
> (243x150 px) as you may see in bug 975208 (attachment
> https://bug975208.bugzilla.mozilla.org/attachment.cgi?id=8385835). I'm
> afraid this is not going to change due to this fact.
This only explains why technically the tiles are smaller. I would like to know from a UX point of view why this change would be better.
Reporter | ||
Updated•10 years ago
|
Flags: needinfo?(jboriss)
Updated•10 years ago
|
Flags: firefox-backlog+
Comment 9•10 years ago
|
||
https://bug1030832.bugzilla.mozilla.org/attachment.cgi?id=8446620 is the spec for the new tiles design. Page 3 of that spec shows how larger screens will show more tiles, which can solve this bug.
We can implement the requested change of page 3 to fix this bug without having to implement the rest of the new Tiles design spec.
Blocks: 1030832
Flags: needinfo?(jboriss)
Comment 10•10 years ago
|
||
Based on the spec that I linked to in the earlier comment, we want to show more tabs when space permits.
I looked in to removing the preference values at first, since they are less useful now but that will affect add-ons that are referencing them and also removes the ability for people to limit the number of tiles that are displayed on the New Tab Page.
We will now show up to 15 tiles on the New Tab Page (5 columns, 3 rows) as is described by the spec. The search container now has a fixed max-width of 600px, and I put a 12px left and right margin on the search container to match the offset that the toggle button has from the top-right corner of the page (visible when the browser window is made very narrow).
The spec shows that a minimum of two columns will be shown when the browser window is < 640 pixels wide, but our tiles don't currently scale down when the viewport is narrower. This would mean that we would need to be OK with showing a horizontal scrollbar, which I'm pretty sure we don't want. In the case that the browser window is very narrow (320px for example), we will show a single column of tiles.
Comment 11•10 years ago
|
||
Added to Iteration 33.2
Jared, can you provide a point value and if the bug should be marked as [qa+] or [qa-]
Iteration: --- → 33.2
QA Whiteboard: {qa?]
Updated•10 years ago
|
QA Whiteboard: {qa?] → [qa?]
Whiteboard: p=2
Updated•10 years ago
|
QA Whiteboard: [qa?] → [qa+]
Comment 12•10 years ago
|
||
Doing a little bugzilla switcharoo here. This bug is now being marked as wontfix as the size of the tiles are by design and we aren't planning on changing them in v1.
Bug 1026561 will be unduped and will be used for the patch that I just attached (v2 will tweak the size of the tiles but since we will probably want to uplift this patch I went with a reduced approach to reduce the potential effects on telemetry experiments and QA risk.
Assignee: jaws → nobody
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Updated•10 years ago
|
Iteration: 33.2 → ---
QA Whiteboard: [qa+]
Flags: firefox-backlog+ → firefox-backlog-
Whiteboard: p=2
Updated•10 years ago
|
Attachment #8449028 -
Flags: review?(adw)
Comment 13•10 years ago
|
||
wontfix, updating the tracking flags.
You need to log in
before you can comment on or make changes to this bug.
Description
•