Closed Bug 1071878 Opened 10 years ago Closed 10 years ago

Sites without favicons should show an empty rectangle rather than a globe

Categories

(Firefox for Android Graveyard :: General, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 35

People

(Reporter: mcomella, Assigned: mcomella)

References

Details

Attachments

(1 file, 1 obsolete file)

Showing a globe would be strange given our doorhanger solution (bug 1071267).
Alternatively, maybe we can symlink R.drawable.favicon?
Attachment #8494171 - Flags: review?(lucasr.at.mozilla)
Assignee: nobody → michael.l.comella
Status: NEW → ASSIGNED
Comment on attachment 8494171 [details] [diff] [review] Put an empty rectangle for websites without favicons on new tablet Review of attachment 8494171 [details] [diff] [review]: ----------------------------------------------------------------- Cool. ::: mobile/android/base/favicons/Favicons.java @@ +400,5 @@ > + // Decode the default Favicon ready for use. We'd preferably override the drawable for > + // different screen sizes, but since we need phone's default favicon on tablet (in > + // ToolbarDisplayLayout), we can't. > + final int defaultFaviconDrawableID = > + isNewTabletEnabled ? R.drawable.favicon_none : R.drawable.favicon; I wonder if favicon_none should actually be named new_tablet_default_favicon?
Attachment #8494171 - Flags: review?(lucasr.at.mozilla) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: