Closed Bug 1259573 Opened 8 years ago Closed 8 years ago

Default favicon should be inverted for highlighted awesomebar entries

Categories

(Firefox :: Address Bar, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- fixed

People

(Reporter: adw, Assigned: adw)

References

Details

Attachments

(1 file)

(In reply to Stephen Horlander [:shorlander] from bug 1181078 comment #42)
> Also it would be good to invert the default favicon if we can.
> Do we have an inverted version?
See also bug 1238907 (there is a white globe here).
Blocks: 1262507
No longer depends on: 1181078
Thanks Guillaume.

Stephen says the icons in that bug are fine.  They're png, unlike the other icons in the popup though.  The default favicon is used in other places besides the awesomebar, and I don't know if other consumers can safely use svg.  For example bug 1174284 says that svg doesn't work in OS X menus -- but bug 1218041 may have fixed it?  Anyway I'll try to see if svg would be OK.
This uses the png icons from bug 1238907.  I don't think it's a good idea to try and switch to svg since the default favicon is used pervasively throughout Firefox.

The icons live in toolkit, so this just sets the image of .ac-site-icon in toolkit's autocomplete.css.  Pretty simple I think.  This way we also get the high-res versions where appropriate, which does not happen currently.

Since the icons are the same across platforms, I created a new shared subdirectory.

Review commit: https://reviewboard.mozilla.org/r/46833/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/46833/
Attachment #8741919 - Flags: review?(mak77)
The leaks on try are due to an unrelated commit that was backed out of fx-team shortly after I refreshed my local copy.
Attachment #8741919 - Flags: review?(mak77) → review+
Comment on attachment 8741919 [details]
MozReview Request: Bug 1259573 - Invert the default favicon for selected awesomebar popup entries. r?mak

https://reviewboard.mozilla.org/r/46833/#review44189
https://hg.mozilla.org/mozilla-central/rev/45f98cbddc01
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Assignee: nobody → adw
You need to log in before you can comment on or make changes to this bug.