Remove empty placeholder icon when a page has no favicon

RESOLVED FIXED in Firefox 28
(NeedInfo from)

Status

()

Firefox
Tabbed Browser
--
enhancement
RESOLVED FIXED
6 years ago
2 years ago

People

(Reporter: Dan Jacobson, Assigned: mconley, NeedInfo)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 28
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Australis:M3])

Attachments

(6 attachments, 3 obsolete attachments)

(Reporter)

Description

6 years ago
User Agent: Mozilla/5.0 (X11; Linux i686; rv:8.0a2) Gecko/20110905 Firefox/8.0a2 Iceweasel/8.0a2
Build ID: 20110905042011

Steps to reproduce:

Gentlemen, observe each tab carries a empty dashed rectangle to its left.
These rectangles are in the same spot spinning red circles are in when the page is loading.
But after it is loaded, these useless rectangles just sit there, wasting precious tag title space.
Kindly revert this no-brainer decoration.

Comment 1

6 years ago
That rectangle is the replacement of the olde empty page acting as a placeholder for the pages favicon. So reverting it would mean switching to the old icon.

Updated

6 years ago
Severity: normal → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: uiwanted
Summary: empty rectangle left of each tab wasting space → Remove empty placeholder icon when a page has no favicon
(In reply to Cork from comment #1)
> That rectangle is the replacement of the olde empty page acting as a
> placeholder for the pages favicon. So reverting it would mean switching to
> the old icon.

This bug aims to remove the generic icon, instead of reverting it to anything. Use this style until then - http://userstyles.org/styles/49060/ff3-hide-blank-tab-favicon
(Reporter)

Comment 3

6 years ago
Yes, no need for 'reserved parking spaces' wasting space.

Comment 4

6 years ago
If the icon is hidden after the page finishes loading, the title will need to suddenly shift over to occupy that empty space, which seems like it would be pretty jarring. Even if we add a transition for that, I think it would draw too much attention to the tab, when the user really wants to be looking at the page that just finished loading.
(In reply to Margaret Leibovic [:margaret] from comment #4)
> If the icon is hidden after the page finishes loading, the title will need
> to suddenly shift over to occupy that empty space, which seems like it would
> be pretty jarring. Even if we add a transition for that, I think it would
> draw too much attention to the tab, when the user really wants to be looking
> at the page that just finished loading.

This is a concern, but I'm not sure how jarring it really would be. I suspect it would be more jarring with a transition.

On the positive side, this would make tabs without icons more recognizable by showing more of their titles.
(Reporter)

Comment 6

6 years ago
(In reply to Margaret Leibovic [:margaret] from comment #4)
Hahahaha "jarring". Don't worry, nobody will notice one more jar amongst the million, at least on my system.

Updated

6 years ago
Duplicate of this bug: 690195

Updated

6 years ago
OS: Linux → All
Hardware: x86 → All
Version: 8 Branch → Trunk
The popping when the loading icon is finished and the broken tab consistency feels kind of gross but that might just be my "OMG Change!" reflex kicking in.

It does feel cleaner without all the placeholder icons though.
OS: All → Linux
Hardware: All → x86
Version: Trunk → 8 Branch

Updated

6 years ago
OS: Linux → All
Hardware: x86 → All
Blocks: 648668
Created attachment 564324 [details]
screenshot of default favicons on bookmarks toolbar

related: the default favicon takes up unnecessary space in my bookmark toolbar too, it would be great to nothing there if there's no favicon (or a config setting to turn them off in the toolbar?)
Attachment #564324 - Attachment is patch: false
Attachment #564324 - Attachment mime type: text/plain → image/png
Version: 8 Branch → Trunk

Comment 10

6 years ago
(In reply to Lukas Blakk [:lsblakk] from comment #9)
> Created attachment 564324 [details]
> screenshot of default favicons on bookmarks toolbar
> 
> related: the default favicon takes up unnecessary space in my bookmark
> toolbar too, it would be great to nothing there if there's no favicon (or a
> config setting to turn them off in the toolbar?)

This would require a separate fix -- would you mind filing a new bug in Bookmarks & History?
> This would require a separate fix -- would you mind filing a new bug in
> Bookmarks & History?

done. bug 691594

Comment 12

6 years ago
This should be looked at urgently for the next release.

The decision made in https://bugzilla.mozilla.org/show_bug.cgi?id=648668 was a very misguided interface mistake, as evidenced by the hundreds of users complaining about this:
https://support.mozilla.com/en-US/search?w=2&a=1&created=0&created_date=&updated=0&updated_date=&sortby=0&asked_by=&answered_by=&num_voted=0&num_votes=&q_tags=&q=favicon+8

The new "dotted rounded square" is an unsuitable replacement for the "blank document" icon, because it incorrectly implies a missing or broken element where there is none (since favicons are entirely optional). At the very least, this should have been implemented as a user-selectable interface option.

Visual change for its own sake is illogical. Remember:
http://www.codinghorror.com/blog/2005/08/the-user-interface-is-the-application.html

Comment 13

6 years ago
Is this a different bug than the dotted square replacing favicons that do, in fact, exist, and were working in FF7 but are now "missing" in FF8?

It appears that this is a more complex bug than simply bad / unexpected UI design.  Take this page, for instance: in the title bar, the Bugzilla favicon appears ( https://bugzilla.mozilla.org/extensions/BMO/web/images/favicon.ico ), but on the tab is the dotted box.  Even Google Canada's favicon is missing; although it is not coded in the HTML, it is sitting in the expected spot ( http://www.google.com/favicon.ico ).

Comment 14

6 years ago
I have discovered that the favicon will appear in the address bar (for sites that have one, whether coded with rel="icon" or in the default root location) after clicking on the "blank" icon beside the address.  (Moving the tab may have the same effect; I am sure that it happened, once, but I have not been able to reproduce it!)

The favicon does not persist, however.  Once the page is reloaded or after clicking a link to another page on the same site, the behaviour repeats ("blank" favicon until clicked).

I have not discovered any way to force the favicon to display on the tab.

I have tested pages that do not have the rel="icon" instruction but do have a default favicon.ico in the site's root, pages that have both a (different) default favicon and a rel="icon" direction, pages that change icons as the site is navigated (different favicons for different pages) and a variety of capitalization ("Icon" "ICON" &c) in the rel="icon" (or rel="shortcut icon") instruction.

Perhaps this is a bug that is deeper than just the change in the picture used for sites without favicons.

Comment 15

6 years ago
Previous versions were fine, but this week's update to v 8.0 effectively removed the favicons from my bookmarks as well as the location bar. Being a visual learner this is a real disappointment to me. I know the favicons are on the servers because some of them are from my own websites.

Comment 16

6 years ago
The favicon also does not appear in the Page Info / Media preview until you click on another picture and then click back on the favicon entry.

This has no effect on the address bar or tab display, and the favicon sometimes persists and sometimes does not when you exit the Page Info window and open it, again.  I have not figured out if there is any pattern to this latter behaviour.

Comment 17

6 years ago
This bug is only about removing the new placeholder icon when a page does not have a favicon.

Valid favicons disappearing is a separate (and bad) bug that somehow managed to get through Aurora and Beta without being filed.

R. and Peter, it sounds like you guys are experiencing bug 701297. If so, could you comment in there with steps to reproduce.

Comment 18

6 years ago
Thanks, Margaret.  I will post these over in that bug; I did not find it when I was looking, earlier.
You're entirely right that the current dotted-border favicon isn't working very well.  It's a bit too small of field to really merit a marching-ant-style marquee.  However, I don't think just showing nothing is the way to handle this problem, but rather improving the placeholder icon.  After all, showing nothing could just suggest that the favicon has not loaded yet, rather than the page has none.  You'd end up with a tab bar which had some favicons and some empty spaces, making it visually jarring and seemingly taking longer to load ("how long must I wait for this favicon to appear?".  I'm going to close this bug, since we don't want some tabs to simply have no favicon, but improving that icon should certainly be another bug.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Keywords: uiwanted
Resolution: --- → WORKSFORME

Comment 20

6 years ago
For the record, I disagree. The only use I have for favicons is to identify a particular web site, or distinguish it from its neighbors in a list, faster than I can read its title. A generic icon would not accomplish this, serving instead only to add clutter and (in the case of horizontal lists like to bookmark bar) to waste space. For sites with no favicon, and for other kinds of bookmark (e.g. bookmarklets), showing no icon is *exactly* what I want.

Also, I would certainly not sit around wondering how long I must wait for a favicon to appear. It's a favicon, not my email. I don't care if it takes a while to load, and I'm perfectly aware that not all sites have them.

If you're really bent on wasting space with icons that have no purpose, I hope you'll at least make it a configurable option.
Boriss, according to your explanation, this is a typical WONTFIX rather than WORKSFORME. See <https://bugzilla.mozilla.org/page.cgi?id=fields.html#status>.
Resolution: WORKSFORME → WONTFIX

Comment 22

6 years ago
@dao: 
However, what you could do, is make the icon more looking like: "hé this icon is missing" in stead of "what kind of bug does firefox have now, with these dotted boxes".

I installed the Silvermel 1.4.4 (http://www.silvermel.net/) theme. It solves this problem by showing the icon of a bookmark for these cases. Looks beautiful and does the job well of revealing that the site doesn't have an icon of its own.

So, WONTFIX should not mean: "we'll never look at this again", in stead it should mean: "we'll never look at this again, unless someone comes along with a very good idea".

Comment 23

6 years ago
(In reply to Forest from comment #20)

> If you're really bent on wasting space with icons that have no purpose, I
> hope you'll at least make it a configurable option.

Forest, for the record, I totally agree with you.
Discussed with Boriss and we want this for the current Australis theme work.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Target Milestone: --- → Firefox 12

Updated

5 years ago
Target Milestone: Firefox 12 → ---
The default blank favicon should all the same be updated for bookmarks (Chrome do a pretty good job concerning this as in many other UI area).

Updated

5 years ago
Blocks: 702730
(Assignee)

Updated

4 years ago
Duplicate of this bug: 847444
(Assignee)

Updated

4 years ago
Assignee: nobody → mconley
It's not completely related but all Firefox in-content pages should have a favicon (maybe before landing this). I'm especially thinking about the new tab page ( http://people.mozilla.com/~shorlander/files/australis-design-specs/images/Australis-i01-DesignSpec-InContentUI-Dashboard-%28NewTab%29.jpg ).
Blocks: 732583
Status: REOPENED → ASSIGNED
(Assignee)

Comment 28

4 years ago
Created attachment 738151 [details] [diff] [review]
Patch v1
(Assignee)

Comment 29

4 years ago
Need to test this on Linux and OSX first, and then I'll request review.
(Assignee)

Comment 30

4 years ago
Created attachment 738650 [details] [diff] [review]
Patch v1.1

As per UX request, the tab labels have been re-centered on OSX.

Testing Linux next, and then I'll huck up some screenshots.
Attachment #738151 - Attachment is obsolete: true
(Assignee)

Comment 31

4 years ago
Created attachment 738661 [details]
Patch v1 on OSX
(Assignee)

Comment 32

4 years ago
Created attachment 738697 [details]
Patch v1 on Ubuntu Linux
(Assignee)

Comment 33

4 years ago
Created attachment 738700 [details]
Patch v1 on Windows 7
(Assignee)

Comment 34

4 years ago
Created attachment 738701 [details]
Patch v1 on Windows XP
(Assignee)

Updated

4 years ago
Attachment #738650 - Flags: ui-review?(shorlander)

Comment 35

4 years ago
Can you add screen shots showing how sites without favicons look in the Bookmark Toolbar?  I'd love to be rid of those stupid placeholder icons that appear next to my bookmarklets.
(Assignee)

Comment 36

4 years ago
(In reply to Forest from comment #35)
> Can you add screen shots showing how sites without favicons look in the
> Bookmark Toolbar?  I'd love to be rid of those stupid placeholder icons that
> appear next to my bookmarklets.

At this point, I'm only concerned about the placeholder favicons in the tabs, as this is a potential blocker for the Australis theme changes.
(In reply to Forest from comment #35)
> Can you add screen shots showing how sites without favicons look in the
> Bookmark Toolbar?  I'd love to be rid of those stupid placeholder icons that
> appear next to my bookmarklets.

See bug 747620.
(Assignee)

Updated

4 years ago
Whiteboard: [Australis:M3]
Comment on attachment 738650 [details] [diff] [review]
Patch v1.1

Review of attachment 738650 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good!
Attachment #738650 - Flags: ui-review?(shorlander) → ui-review+
(Assignee)

Comment 39

4 years ago
Comment on attachment 738650 [details] [diff] [review]
Patch v1.1

If you're overloaded Matt, let me know and I'll redirect the review request.
Attachment #738650 - Flags: review?(mnoorenberghe+bmo)
Comment on attachment 738650 [details] [diff] [review]
Patch v1.1

>--- a/browser/base/content/browser.css	Fri Apr 12 18:33:01 2013 -0400
>+++ b/browser/base/content/browser.css	Wed Apr 17 14:18:35 2013 -0400
>@@ -51,16 +51,17 @@ tabbrowser {
>   max-width: 0.1px;
>   min-width: 0.1px;
>   opacity: 0 !important;
>   transition: min-width 200ms ease-out,
>               max-width 250ms ease-out,
>               opacity 50ms ease-out 180ms /* hide the tab for the last 20ms of the max-width transition */;
> }
> 
>+.tab-icon-image:not([src]):not([pinned]),
> .tab-throbber:not([fadein]):not([pinned]),
> .tab-label:not([fadein]):not([pinned]),
> .tab-icon-image:not([fadein]):not([pinned]),
> .tab-close-button:not([fadein]):not([pinned]) {
>   display: none;
> }

These rules are about tab animations, which your selector has nothing to do with. You should add it here at line 44: http://hg.mozilla.org/mozilla-central/annotate/d8202613aaea/browser/base/content/tabbrowser.css#l44

>--- a/browser/themes/osx/browser.css	Fri Apr 12 18:33:01 2013 -0400
>+++ b/browser/themes/osx/browser.css	Wed Apr 17 14:18:35 2013 -0400

>+.tab-text {
>+  margin: 0;
>+}

.tab-label

>--- a/browser/themes/shared/tabs.inc.css	Fri Apr 12 18:33:01 2013 -0400
>+++ b/browser/themes/shared/tabs.inc.css	Wed Apr 17 14:18:35 2013 -0400

>+.tab-throbber,
>+.tab-icon-image {
>+  width: 16px;
>+  height: 16px;
>+  list-style-image: url("chrome://mozapps/skin/places/defaultFavicon.png");
>+}

defaultFavicon.png doesn't make sense for .tab-throbber. This is particularly confusing because you put this in tabs.inc.css while leaving the .tab-throbber specific rules in the individual browser.css files.
Attachment #738650 - Flags: review?(mnoorenberghe+bmo) → review-
(Assignee)

Comment 41

4 years ago
Created attachment 741433 [details] [diff] [review]
Patch v1.2 (ui-r+'d by shorlander)

(In reply to Dão Gottwald [:dao] from comment #40)
> Comment on attachment 738650 [details] [diff] [review]
> Patch v1.1

Hey Dão, thanks for the drive-by!

> 
> >--- a/browser/base/content/browser.css	Fri Apr 12 18:33:01 2013 -0400
> >+++ b/browser/base/content/browser.css	Wed Apr 17 14:18:35 2013 -0400
> >@@ -51,16 +51,17 @@ tabbrowser {
> >   max-width: 0.1px;
> >   min-width: 0.1px;
> >   opacity: 0 !important;
> >   transition: min-width 200ms ease-out,
> >               max-width 250ms ease-out,
> >               opacity 50ms ease-out 180ms /* hide the tab for the last 20ms of the max-width transition */;
> > }
> > 
> >+.tab-icon-image:not([src]):not([pinned]),
> > .tab-throbber:not([fadein]):not([pinned]),
> > .tab-label:not([fadein]):not([pinned]),
> > .tab-icon-image:not([fadein]):not([pinned]),
> > .tab-close-button:not([fadein]):not([pinned]) {
> >   display: none;
> > }
> 
> These rules are about tab animations, which your selector has nothing to do
> with. You should add it here at line 44:
> http://hg.mozilla.org/mozilla-central/annotate/d8202613aaea/browser/base/
> content/tabbrowser.css#l44
> 

Good call - thanks. Fixed.

> >--- a/browser/themes/osx/browser.css	Fri Apr 12 18:33:01 2013 -0400
> >+++ b/browser/themes/osx/browser.css	Wed Apr 17 14:18:35 2013 -0400
> 
> >+.tab-text {
> >+  margin: 0;
> >+}
> 
> .tab-label
> 

Alright, switched. Although, for my own education, why the preference for .tab-label over .tab-text?

> >--- a/browser/themes/shared/tabs.inc.css	Fri Apr 12 18:33:01 2013 -0400
> >+++ b/browser/themes/shared/tabs.inc.css	Wed Apr 17 14:18:35 2013 -0400
> 
> >+.tab-throbber,
> >+.tab-icon-image {
> >+  width: 16px;
> >+  height: 16px;
> >+  list-style-image: url("chrome://mozapps/skin/places/defaultFavicon.png");
> >+}
> 
> defaultFavicon.png doesn't make sense for .tab-throbber. This is
> particularly confusing because you put this in tabs.inc.css while leaving
> the .tab-throbber specific rules in the individual browser.css files.

Fixed!
Attachment #738650 - Attachment is obsolete: true
Attachment #741433 - Flags: review?(dao)
(In reply to Mike Conley (:mconley) from comment #41)
> Alright, switched. Although, for my own education, why the preference for
> .tab-label over .tab-text?

.tab-label is tabbrowser specific, whereas .tab-text applies to all XUL tabs in the window.
Comment on attachment 741433 [details] [diff] [review]
Patch v1.2 (ui-r+'d by shorlander)

You don't seem to be setting defaultFavicon.png anymore. It's still needed for pinned tabs.
Attachment #741433 - Flags: review?(dao) → review-
(Assignee)

Comment 44

4 years ago
Created attachment 741449 [details] [diff] [review]
Patch v1.3 (ui-r+'d by shorlander)

Ah, good call. Does this cover all of the bases?
Attachment #741433 - Attachment is obsolete: true
Attachment #741449 - Flags: review?(dao)

Updated

4 years ago
Attachment #741449 - Flags: review?(dao) → review+
(Assignee)

Comment 45

4 years ago
Thanks Dão!

Landed on UX as https://hg.mozilla.org/projects/ux/rev/fd26cf8fc03a
Whiteboard: [Australis:M3] → [Australis:M3][fixed-in-ux]
(Assignee)

Comment 46

4 years ago
Landed on Jamun as https://hg.mozilla.org/projects/jamun/rev/8bf747097b31
(Assignee)

Updated

4 years ago
Whiteboard: [Australis:M3][fixed-in-ux] → [Australis:M3][fixed-in-ux][fixed-in-jamun]

Updated

4 years ago
Blocks: 865776
(Assignee)

Updated

4 years ago
Depends on: 865728
(Assignee)

Updated

4 years ago
Blocks: 865728
No longer depends on: 865728
Will this land on Nightly before Australis lands (like the separate bookmarks button did)?
(Assignee)

Comment 48

4 years ago
(In reply to Siddhartha Dugar [:sdrocking] from comment #47)
> Will this land on Nightly before Australis lands (like the separate
> bookmarks button did)?

Hey Siddhartha,

This will land when Australis lands.

-Mike

Comment 49

4 years ago
there should at least be an about:config entry on whether to show the placeholder or not. for if some people want to get it without user script hacks
No longer blocks: 865776
Depends on: 865776

Comment 50

4 years ago
https://hg.mozilla.org/mozilla-central/rev/8bf747097b31
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago4 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:M3][fixed-in-ux][fixed-in-jamun] → [Australis:M3]
Target Milestone: --- → Firefox 28

Comment 51

4 years ago
(In reply to rexyrexy2 from comment #49)
> there should at least be an about:config entry on whether to show the
> placeholder or not. for if some people want to get it without user script
> hacks

Agreed. When using vertical tabs the text of non-favicon tabs doesn't line up with other tabs.

Comment 52

2 years ago
Why is this bug marked "fixed"? The problem still exists in the latest versions of Firefox
Flags: needinfo?(mconley)
(Assignee)

Comment 53

2 years ago
(In reply to Omar from comment #52)
> Why is this bug marked "fixed"? The problem still exists in the latest
> versions of Firefox

Can you please post a screenshot of the icon you're seeing?f
Flags: needinfo?(mconley) → needinfo?(omaryak)
You need to log in before you can comment on or make changes to this bug.