Closed Bug 904998 Opened 11 years ago Closed 10 years ago

Export "close" button icon for autocomplete.

Categories

(Firefox for Metro Graveyard :: Browser, defect, P1)

x86_64
Windows 8.1
defect

Tracking

(firefox28 verified, firefox29 verified)

VERIFIED FIXED
Firefox 29
Tracking Status
firefox28 --- verified
firefox29 --- verified

People

(Reporter: kjozwiak, Assigned: mmaslaney)

References

Details

(Whiteboard: [beta28] [ux] p=0)

Attachments

(5 files)

I think it would be a good idea to make the "X" button under the "Your Results" & "Search For" screen a lot darker as its currently really hard to see as the background color is almost the same shade as the button. Maybe make it as dark as the buttons under the "Navigation App Bar" so it matches the theme?

- Attached a screenshot to illustrate the issue

Yuan, your thoughts?

Steps to reproduce the issue:

1) Open Firefox Metro
2) Tap on the text box in the Navigation App Bar and start typing something (Should see the "Your Results" & "Search For" screen)
3) At the bottom right corner, you should see the "X" button (really hard to see)

Current Behavior:

- The close "X" button is almost the same color as the background making it very difficult to see

Expected Behavior:

- Button should be a bit darker so its a lot easier to see

Used the following build:
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-08-13-03-02-05-mozilla-central/
Flags: needinfo?(ywang)
Whiteboard: feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0 → [preview-triage] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0
I am aware of this. Thanks, Kamil.

Currently there are a few bugs awaiting assets from our visual designer. This is one of them. 

The icon supposed to be grey not white. See this screenshot: http://cl.ly/image/442R3F2p3V3D
Flags: needinfo?(shorlander)
Flags: needinfo?(ywang)
I can't seem to find the bug I thought I filed about getting a an icon slice, so morphing this and assigning to shorlander.

shorlander, can we get sprites at 100%, 140%, and 180% for the close icon in https://bug895520.bugzilla.mozilla.org/attachment.cgi?id=785773? Thanks!
Assignee: nobody → shorlander
Flags: needinfo?(shorlander)
Summary: Change - Making the "X" button darker under "Your Results" & "Search For" screen → Work - Export "close" button icon for autocomplete.
Whiteboard: [preview-triage] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0 → [preview] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0
No longer depends on: Backlog-MetroDesign
Priority: -- → P1
Summary: Work - Export "close" button icon for autocomplete. → [MP] Change - Export "close" button icon for autocomplete.
Hey Stephen, do you know when you can provide the "close" button icon for autocomplete?
Flags: needinfo?(shorlander)
No longer blocks: MetroPreviewRelease
Summary: [MP] Change - Export "close" button icon for autocomplete. → Change - Export "close" button icon for autocomplete.
Whiteboard: [preview] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0 → feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0
Assignee: shorlander → ywang
Flags: needinfo?(shorlander)
Whiteboard: feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0 → [release28] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0
Assignee: ywang → nobody
Assignee: nobody → mmaslaney
Whiteboard: [release28] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0 → [beta28] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0
Whiteboard: [beta28] feature=change c=firefox_app_bar_and_autocomplete u=metro_firefox_user p=0 → [work] [beta28]
Summary: Change - Export "close" button icon for autocomplete. → Export "close" button icon for autocomplete.
Attached image no_close_button.png
With the latest Nightly 29.0a1 (build ID: 20131218030217) on Win 8 64-bit, I can't see at all any "close" button icon for the autocomplete screen. Please see the attached screenshot for details.

Should I file a new bug for this issue?
Flags: needinfo?(mmaslaney)
(In reply to Manuela Muntean [:Manuela] [QA] from comment #6)
> Created attachment 8350015 [details]
> no_close_button.png
> 
> With the latest Nightly 29.0a1 (build ID: 20131218030217) on Win 8 64-bit, I
> can't see at all any "close" button icon for the autocomplete screen. Please
> see the attached screenshot for details.
> 
> Should I file a new bug for this issue?

Yes, dependent on this one. I think this bug was about getting the button graphics from design. We currently don't have a bug filed on integrating the button into the ui.
Attached file icon-navBar-close.zip
Updated Close button.
Flags: needinfo?(mmaslaney)
Blocks: 952906
Thank you Jim! I've logged bug  952906 accordingly.
Whiteboard: [work] [beta28] → [beta28]
Whiteboard: [beta28] → [beta28] [defect] p=0
No longer blocks: Backlog-MetroDesign
Whiteboard: [beta28] [defect] p=0 → [beta28] [feature] p=0
Whiteboard: [beta28] [feature] p=0 → [beta28] [ux] p=0
No longer blocks: metrov1backlog
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Marking "status-firefox28: fixed" to get this design-only bug off our Fx28 uplift radar.
Kamil, please verify this is fixed in the latest Firefox Nightly and Aurora builds.
Flags: needinfo?(kamiljoz)
Went through the following issue for iteration #21 without any issues. Used the following builds:
- http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-01-16-03-02-50-mozilla-central/

Going through this issue alongside with Bug # 952906.

- Went through the original test case from comment #0 without any issues
- Ensured that the "X" is visible when viewing the Search App Bar under about:start
- Ensured that the "X" is visible when viewing the Search App Bar under several other websites
- Ensured that the "X" hover status doesn't get stuck when taping/clicking on it
- Ensured that the "X" button is using anti-aliasing (smooth, clear without any rough edges.
- Ensured that all of the above test cases worked in both full and filled views (using a variety of different screen widths)

Checked today's Aurora build and the changes still haven't made it in, will take a look tomorrow to complete verification. Leaving needinfo until both channels have been verified.
Went through the following defect for iteration #21 without any issues. Used the following build:
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014-01-17-00-40-24-mozilla-aurora/

- Went through the original test case from comment #0 without any issues
- Went through all the test cases from comment #12 without any issues
- Also went through Bug # 952906 alongside this one
Flags: needinfo?(kamiljoz)
Status: RESOLVED → VERIFIED
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: