Last modified: 2011-05-15 10:09:39 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T5043, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 3043 - RSS feed link should have silly orange icon
RSS feed link should have silly orange icon
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
unspecified
All All
: Low enhancement with 2 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
http://en.wikipedia.org/wiki/Special:...
:
Depends on:
Blocks: feeds css
  Show dependency treegraph
 
Reported: 2005-08-04 12:40 UTC by Brion Vibber
Modified: 2011-05-15 10:09 UTC (History)
8 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Sample 12x12 feed icon colored to match monobook bullets (578 bytes, image/png)
2007-03-06 05:45 UTC, Brion Vibber
Details
Sample patch to monobook.css using icon for the list bullet (417 bytes, patch)
2007-03-06 05:46 UTC, Brion Vibber
Details
Screen shot of toolbox with the icon on the feeds line (7.96 KB, image/png)
2007-03-06 05:48 UTC, Brion Vibber
Details
Screen shot IE7 showing borked display (2.98 KB, image/png)
2009-01-28 19:52 UTC, Brion Vibber
Details

Description Brion Vibber 2005-08-04 12:40:35 UTC
For consistency with other sites, the RSS and Atom feed links should use little icons in the common style.
The text links ("rss", "atom") are not very visible or standard.
Comment 1 Rowan Collins [IMSoP] 2005-08-05 18:28:22 UTC
This depends how much attention we want to draw to the feeds, really. Browsers
such as Firefox can already garner the fact that there are feeds from tags in
the <head> element, and nothing else in the navigation gets an icon. If the
links *are* turned into icons rather than text, I would suggest not using a
"silly orange icon" (by which I presume you mean something like [1]), but
something slightly subtler - more on the lines of [2] or [3], perhaps, which are
equally "standard" in terms of look, but easier to tweak to not be *too* visible...

[1] - http://dmxzone.com/downloads/images/rss_button.jpg
[2] - http://www.kent.ac.uk/is/computing/img/rss-button.png
[3] - http://www.frontfrauen.de/bilder/button_rss.gif
Comment 2 mike waggoner 2005-10-18 20:12:37 UTC
Regardless of whether we use the 'standard' buttons, I do think the rss and atom
links in the sidebar of recentchanges are a bit unlikely to be found unless sought.

I would recommend each (rss, atom) having it's own line, and the optional use of
icons.
Comment 3 Xerol Oplan 2006-05-17 21:42:18 UTC
I think the closest thing to a standard right now is this: 
http://blogs.msdn.com/rssteam/archive/2005/12/14/503778.aspx
While still not an actual standard, per se, being adapted by both major browser camps puts it 
close enough. (The orange text buttons are out, supposedly. Doesn't stop everyone from 
continuing to use them though.)
However, I like the idea behind this site: http://feedicons.com/
The idea being that the IMAGE, and not the COLOR is what should be kept universal. I think 
even more important is that the image in use should mesh well with the existing skin/theme, 
and perhaps that having a feedicon parameter in the skin definition might be a good solution 
to this (so it can be easily customised for whatever skin is being used).
Comment 4 Tadpole9 2007-02-26 04:36:43 UTC
My preference would be to replace the list bullet with a small icon (10x10, even?) in any color. I don't 
think we need them on separate lines, as the two feeds are essentially the same thing.
Comment 5 Titoxd 2007-02-26 04:42:40 UTC
I think the light blue icon in feedicons would match the Monobook and Cologne
Blue skins the most. Also, the 12x12 icon feedicons uses is the perfect size, IMO.
Comment 6 Brion Vibber 2007-03-06 05:45:42 UTC
Created attachment 3292 [details]
Sample 12x12 feed icon colored to match monobook bullets

Derived from the 12x12px linked above...
Comment 7 Brion Vibber 2007-03-06 05:46:28 UTC
Created attachment 3293 [details]
Sample patch to monobook.css using icon for the list bullet
Comment 8 Brion Vibber 2007-03-06 05:48:21 UTC
Created attachment 3294 [details]
Screen shot of toolbox with the icon on the feeds line

Taken w/ Firefox 2.0.0.2 on Mac OS X.

The position looks a little funny to me (same in Safari).
Also I think it might be odd that the icon itself is not clickable;
additionally there is no equivalent list item in the current usage of the feed
links on Cologne Blue and other such skins.
Comment 9 Aryeh Gregor (not reading bugmail, please e-mail directly) 2007-03-06 17:31:01 UTC
It could perhaps be made clickable using some kind of voodoo-magic like

li#feedlinks { list-style: none; }
li#feedlinks a { display: list-item; list-style: square url(...); margin:
something; }

But who knows how browsers would take that (IE is especially persnickety with
display: anything).  And CSS is completely vague on how to position list-items,
so if we want it to be positioned prettily we'll have to hope that all the
browsers position the same so we can use some pixel-specific image . . . or
maybe hack in different images for different browsers . . . yuck.
Comment 10 Titoxd 2007-03-12 02:45:32 UTC
Perhaps include the bullet, as it is currently done, and then put the icon to
the right of the RSS / Atom link? That way, a hard <a href="..."> can be used,
and all of those problems with the bullet are bypassed.
Comment 11 Brion Vibber 2007-03-12 16:22:04 UTC
Well, now you've got the problem of which feed version are you linking? Just
RSS, just Atom? Or an icon for each?
Comment 12 Phil Boswell 2007-03-12 16:36:50 UTC
Try delimiting them so:

Icon ":" "RSS<link>" &bull; "Atom<link>"

Then it's more obvious that this is a "list of links" and further flavours can be added 
later if needed.
Comment 13 Aryeh Gregor (not reading bugmail, please e-mail directly) 2007-03-13 00:07:35 UTC
Then clicking the icon won't get you the link, though, which is unexpected.
Comment 14 Danny B. 2009-01-04 18:27:09 UTC
#feed-atom a, #feed-rss a
{
background: url("http://upload.wikimedia.org/wikipedia/commons/thumb/4/43/Feed-icon.svg/12px-Feed-icon.svg.png") center left no-repeat;
padding-left: 16px;
}

seems sane in all Monobook based skins.

Classic based skins would need to add appropriate id to such links.

Then I'd also suggest to remove the unnecessary waste - wrapping <span> of feed links in Monobook skins in favor of moving the id attribute to <a>. Then all skins could have the same code which would be

#feed-atom, #feed-rss
{
background: url("http://upload.wikimedia.org/wikipedia/commons/thumb/4/43/Feed-icon.svg/12px-Feed-icon.svg.png") center left no-repeat;
padding-left: 16px;
}

The URL to image should be to the /skins directory (or its subdir) of course where the image should be stored, here it is just for simple copypaste to see how it looks.

Changing padding and image position to right seems fine as well. Icon color - choose whatever, but I'd prefer the standard orange one since it will shout out "here are the links you're looking for".
Comment 15 Danny B. 2009-01-04 18:30:11 UTC
See http://cs.wikinews.org/wiki/Speci%C3%A1ln%C3%AD:Recentchanges for demo.
Comment 16 Brion Vibber 2009-01-05 19:09:15 UTC
Looks cute! Any browser compat or RTL issues?
Comment 17 Danny B. 2009-01-05 20:05:55 UTC
I did not notice any cross-browser problems but I noticed issues in RTL.

However as far as can I see on random RTL wikis, they don't even use those link icons for external, mailto:, https:, irc: etc. links, which are based on the same principle, so I guess it's known issue. Thus I'd suggest to have the RTL wikis without those icons as well. Thus perhaps some override in rtl.css would have to be done, such as

#feed-atom a, #feed-rss a
{
background-image: none;
padding-left: 0;
}

(or without a's depending on which selector will be used - see comment #14)

As far as I tested known CSS constructions, the wrong displaying seems to me rather like browser issues, which I guess is why those icons are disabled on RTL wikis.
Comment 18 Chad H. 2009-01-06 20:13:29 UTC
(In reply to comment #17)
> However as far as can I see on random RTL wikis, they don't even use those link
> icons for external, mailto:, https:, irc: etc. links, which are based on the
> same principle, so I guess it's known issue. Thus I'd suggest to have the RTL
> wikis without those icons as well...

See bug 16502.
Comment 19 Danny B. 2009-01-23 02:11:12 UTC
Fixed for Monobook and Modern skins in r46058.
Comment 20 Danny B. 2009-01-23 03:41:14 UTC
Improved r46058, now works in all skins.
Comment 21 Ahmad Sherif 2009-01-28 11:41:06 UTC
The icons is still shown in RTL wikis (e.g. Arabic and Hebrew wikipedia). It looks good in FF3 and Konqueror (and perhaps Safari since it acts like FF3). But in Opera 9 (and perhaps the rest of browsers), the text overrides the icons and there is an ugly padding to the right. As mentioned in comment 18 and bug 16502, the problem also exists for the external links icons. Those lines in common_rtl.css are useless /* feed links */
a.feedlink {
	background: none;
	padding-left: 0;
}
since common_rtl.css is  not loaded in RTL wikis for some reason. Instead, They need to be placed in rtl.css with the !important keyword like external icons in the same file, which maybe like
a.feedlink {
	background: none !important;
	padding: 0 !important;
}
Comment 22 Melancholie 2009-01-28 12:06:42 UTC
Although comment #21 is much more important, could those icon(s) be made a little bit smaller, slightly more discreet?
Let's say 2px less in height?!
Comment 23 Brion Vibber 2009-01-28 19:52:17 UTC
Created attachment 5740 [details]
Screen shot IE7 showing borked display
Comment 24 Brion Vibber 2009-01-28 19:57:45 UTC
Disabled for RTL in r46474.
Comment 25 Schnargel 2009-01-31 19:15:51 UTC
This is a vote against displaying feed icons.

Except Monobook no skin used any icons until now. These flashy orange feed icons are in strong contrast to the rest of the mostly blue/gray/black colored skins, draw too much attention to them and distract from more important parts of the page and other links. The effect is worse in skins less playful than Monobook. There's no reason to emphasize just the feed links and not any of the other two dozen or more.

Comment 26 Danny B. 2009-05-03 10:53:32 UTC
Broken by r47615 or some of it followups. Reopening until fixed again.
Comment 27 Brion Vibber 2009-05-08 19:13:17 UTC
Doesn't seem particularly broken. Re-resolving FIXED. Please open a new, specific bug if there are new problems.
Comment 28 Danny B. 2009-10-31 14:23:10 UTC
The icon no longer appears again. I was unable to find the revision which caused that, though.

Reopening until fixed again.
Comment 29 Alexandre Emsenhuber [IAlex] 2009-10-31 14:53:17 UTC
Should be related to r57119.
Comment 30 DieBuche 2011-04-14 16:15:36 UTC
It does now. No idea which rev

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links