Opened 16 years ago
Closed 9 years ago
#3738 closed task (fixed)
RSS icons missing
| Reported by: | Rainer Sokoll | Owned by: | osimons |
|---|---|---|---|
| Priority: | normal | Component: | FullBlogPlugin |
| Severity: | minor | Keywords: | RSS icons WebUI |
| Cc: | Trac Release: | 0.11 |
Description
In the sidebar, there is a RSS icon only for the archive, but it is missing for "Browse by author:" and "Browse by category:".
The links are there. Could you add the icons to the links?
Attachments (2)
Change History (7)
comment:1 Changed 16 years ago by
comment:2 Changed 16 years ago by
Maybe you could make a config option "show_icon_instead_of_textlink"?
I would like it to have it consistent: Either all links with icon or none.
comment:3 Changed 16 years ago by
One: Sorry for the duplicate ticket a few days ago.
Two: I'm going to attach a suggested patch. It adds two options:
- display_rss_icon (defaults to true, only displays icon in archive)
- display_many_rss_icons (defaults to false, displays icons everywhere else)
I hope you find it useful.
Changed 16 years ago by
| Attachment: | fullblogplugin.diff added |
|---|
Suggested patch for handling no/one/multiple rss icons in the fullblogplugin sidebar
Changed 16 years ago by
| Attachment: | fullblogplugin.2.diff added |
|---|
Updated suggested patch for handling no/one/multiple rss icons in the fullblogplugin sidebar
comment:4 Changed 16 years ago by
Oops ... my first suggested patch included an unrelated patch. Bad me. Corrected in the updated fullblogplugin.2.diff attachment.
I could, and did in fact have them there at some stage in earlier development. The reason I removed them was clutter - it drew way to much unneeded attention having a column of 40 RSS icons. I decided to keep it for the main feed to make that easy to notice, and tone down the other more specific feeds by just using a small text string.
I'm quite happy with current situation, but I'll listen to arguments as to why a large number of feed icons improves the user experience...