Closed
Bug 1037091
Opened 10 years ago
Closed 10 years ago
Add gear button with doorhanger configuration of newtab page
Categories
(Firefox :: New Tab Page, defect)
Firefox
New Tab Page
Tracking
()
People
(Reporter: Mardak, Assigned: Mardak)
References
Details
Attachments
(4 files, 3 obsolete files)
12.46 KB,
patch
|
Details | Diff | Splinter Review | |
5.26 KB,
image/png
|
Details | |
9.29 KB,
image/png
|
Details | |
5.91 KB,
patch
|
adw
:
review+
|
Details | Diff | Splinter Review |
From the prototype: http://bg61zt.axshare.com/new_tab.html
There'll need to be a gear icon to customize the new tab page with presets:
- enhanced
- classic
- blank
Where these presets toggle various show/hide customizations for the page:
- enhanced images
- search box
- tiles
- messaging/explanation area
enhanced = all
classic = search + tiles
blank = none
Flags: firefox-backlog+
Assignee | ||
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8461258 [details] [diff] [review]
v1
Review of attachment 8461258 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good, the only thing is that I think "customize" would be a better name for everything than "gear," but it's a nit, so feel free to ignore.
Attachment #8461258 -
Flags: review?(adw) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8461258 -
Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Updated•10 years ago
|
Iteration: --- → 34.1
QA Whiteboard: [qa?]
Can anyone confirm that the gear button changes sizes on new tab openings?
https://bugzilla.mozilla.org/show_bug.cgi?id=1044570
Updated•10 years ago
|
QA Whiteboard: [qa?] → [qa+]
QA Contact: cornel.ionce
Comment 7•10 years ago
|
||
I've verified this using the latest Firefox 34 Nightly build (20140804030205) on: Win 7 x64, Win 8 x64, Win 8.1 x64, Ubuntu 12.04 x64, Mac OS X 10.9.4.
I was able to verify that the following work as expected:
- gear icon displays in the top right corner of the New Tab page, and has an alternative image when hovered/selected
- gear icon opens a list with options: Enhanced, Classic (default), Blank
- selecting Enhanced will set the following:
* browser.newtabpage.enabled=true
* browser.newtabpage.enhanced=true
* enhanced history tiles will display in all New Tabs for *.mozilla.org (bug 1042204 comment 6)
- selecting Classic will set the following:
* browser.newtabpage.enabled=true
* browser.newtabpage.enhanced=false
* non-enhanced regular history tiles will display in all New Tabs for *.mozilla.org (bug 1042204 comment 6)
- selecting Blank will set the following:
* browser.newtabpage.enabled=false
* browser.newtabpage.enhanced=false/true (depends if switched from Enhanced or Classic)
* all New Tab pages will display empty (with just the gear icon visible and accessible)
- transitions between the 3 options work fine
Potential issues:
1. For now I see that there's not much difference between Enhanced and Classic modes - the only difference is that in Enhanced mode we display enhanced history tiles with a rollover picture on hover (see bug 1042204 comment 6). This means that the "messaging/explanation area" is not seen... let me know if this is an issue (I'm guessing it's not or will not be implemented?).
2. Gear icon is invisible in High Contrast modes (white, black, color) in Windows - see attached image
3. Gear icon looks rather blurry while in hovered/selected state - see attached image
4. Can open/dismiss list of options via keyboard, but cannot select one of the options in the list via keyboard
(In reply to Mikel from comment #6)
> Can anyone confirm that the gear button changes sizes on new tab openings?
> https://bugzilla.mozilla.org/show_bug.cgi?id=1044570
I've tried to reproduce this in all 5 operating systems that I've used, but was unsuccessful. I could not get the gear icon to change size even after quickly opening 60-70 New Tabs.
Ed, can you review what was tested above, and especially the list of potential issues, and let me know whether I should file them separately (and close this) or reopen this issue.
Flags: needinfo?(edilee)
Comment 8•10 years ago
|
||
Comment 9•10 years ago
|
||
Updated•10 years ago
|
Iteration: 34.1 → 34.2
Comment 10•10 years ago
|
||
(In reply to Florin Mezei, QA (:FlorinMezei) from comment #7)
> 3. Gear icon looks rather blurry while in hovered/selected state - see
> attached image
Looks like tjis is bug 1046693.
Assignee | ||
Comment 11•10 years ago
|
||
(In reply to Florin Mezei, QA (:FlorinMezei) from comment #7)
> 1. For now I see that there's not much difference between Enhanced and
> Classic modes
Yup. For now it's just the enhancement of history and there's not many history tiles that are getting enhanced right now.
> 2. Gear icon is invisible in High Contrast modes (white, black, color) in
> Windows - see attached image
Please file a followup.
> 3. Gear icon looks rather blurry while in hovered/selected state - see
> attached image
Bug 1046693.
> 4. Can open/dismiss list of options via keyboard, but cannot select one of
> the options in the list via keyboard
A followup for this too. Bug 1044537 is related to the menu items but not about keyboard access. Also, I would think gear menu item related issues should be the same with the search engine selection menu.
Flags: needinfo?(edilee)
Comment 12•10 years ago
|
||
(In reply to Ed Lee :Mardak from comment #11)
> > 4. Can open/dismiss list of options via keyboard, but cannot select one of
> > the options in the list via keyboard
> A followup for this too. Bug 1044537 is related to the menu items but not
> about keyboard access. Also, I would think gear menu item related issues
> should be the same with the search engine selection menu.
Yes, please file individual follow-ups for each menu.
Comment 13•10 years ago
|
||
(In reply to Ed Lee :Mardak from comment #11)
> (In reply to Florin Mezei, QA (:FlorinMezei) from comment #7)
> > 2. Gear icon is invisible in High Contrast modes (white, black, color) in
> > Windows - see attached image
> Please file a followup.
Added bug 1051861.
> > 4. Can open/dismiss list of options via keyboard, but cannot select one of
> > the options in the list via keyboard
> A followup for this too. Bug 1044537 is related to the menu items but not
> about keyboard access. Also, I would think gear menu item related issues
> should be the same with the search engine selection menu.
Added bug 1051865 for the gear icon, and bug 1051873 for the Search Provider button.
Marking this as verified since follow up bugs are tracked separately, and have been added to the backlog.
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa+] → [qa!]
Assignee | ||
Comment 14•10 years ago
|
||
To land this for aurora, we need to not change strings. This means the functionality for aurora non-en-US users will see a gear icon that toggles the newtab page.
Assignee | ||
Comment 15•10 years ago
|
||
For uplift to aurora, we want to avoid string changes in the properties/dtd.
Attachment #8476446 -
Flags: review?(adw)
Assignee | ||
Updated•10 years ago
|
Attachment #8476446 -
Attachment is patch: true
Assignee | ||
Updated•10 years ago
|
Attachment #8476446 -
Flags: review?(adw)
Assignee | ||
Comment 16•10 years ago
|
||
This patch is on top of the one landed in m-c, but when landing this patch for aurora, it'll be squashed into the same commit so there's no churn of locales files.
Assignee | ||
Comment 17•10 years ago
|
||
Here's a patch on top of what landed in m-c but to make it string-freeze-friendly for aurora uplift.
Attachment #8476446 -
Attachment is obsolete: true
Attachment #8477109 -
Attachment is obsolete: true
Attachment #8477762 -
Flags: review?(adw)
Assignee | ||
Updated•10 years ago
|
status-firefox33:
--- → affected
Updated•10 years ago
|
status-firefox34:
--- → verified
Updated•10 years ago
|
Attachment #8477762 -
Flags: review?(adw) → review+
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Comment 18•10 years ago
|
||
Comment 19•10 years ago
|
||
Verified fixed on Windows 7 64bit, Mac OS X 10.8.5 and Ubuntu 14.04 32bit using latest Aurora, build ID: 20140828004002.
You need to log in
before you can comment on or make changes to this bug.
Description
•