Closed Bug 1116009 Opened 10 years ago Closed 6 years ago

Icon to add search engine looks very out of place

Categories

(Firefox :: Search, defect)

36 Branch
x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: kats, Unassigned)

References

Details

(Whiteboard: [devedition-polish])

Attachments

(2 files)

I'm running the latest aurora and when I load any bugzilla page there's a green + icon on the browser search field that looks very out of place. Clicking on it shows a menu. See screenshots.
Also this is on OS X 10.6.
Seeing this on OS X 10.9.5 as well, latest aurora desktop build.
[Tracking Requested - why for this release]: See screenshot, the icon looks very wrong and is very obvious to the user.
Version: unspecified → 36 Branch
Can you elaborate on "very wrong"? Apart from the increased contrast from the dark Dev Edition theme, this is how this feature was intended to look (see bug 1106432).
(In reply to :Gavin Sharp [email: gavin@gavinsharp.com] from comment #4) > Can you elaborate on "very wrong"? Unfortunately I don't have the right vocabulary to elaborate much. The best I can say is that the color contrast and the sharp edges of the green plus make it look like the plus is "floating" on top of the search bar rather than being attached to the magnifying glass. > Apart from the increased contrast from the dark Dev Edition theme, this is > how this feature was intended to look (see bug 1106432). Man. If it's intended to look this way I guess that's fine, but we should try to make sure these things look good in the dark dev edition theme. This is the second time (see bug 1098401 for the first) I've seen new icons appear that just look wrong in Aurora but are "by design".
To me, the reason it looks out of place is that the icon design does not make it immediately clear that the plus sign is part of / an addition to the search icon; visually, it looks "on top"/"overlaid" rather than "part of" the underlying icon. It also wasn't clear to me what the purpose of the icon was (in particular, that it was connected to the current page I was viewing) until I looked at the bug you linked to.
Blocks: 1106432
We can reduce the opacity for DevEdition I guess.
Whiteboard: [devedition-polish]
See Also: → 1119273
No longer depends on: 1119273
Stephen, can you make a call on this
Flags: needinfo?(shorlander)
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
This looks much better in our current theming. Marking as fixed.
Flags: needinfo?(shorlander)
Resolution: INACTIVE → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: