Opened 14 years ago

Last modified 10 years ago

#12978 new New feature

Support in syndication framework for CSS stylesheet links

Reported by: intrepidweb Owned by: Yuval Adam
Component: contrib.syndication Version: 1.1
Severity: Normal Keywords: syndication css stylesheets
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

It would be very helpful to be able to add CSS stylesheets to feeds produced by the syndication framework. As far as I know, this is not possible now without overriding the write method of the feed class.

Attachments (4)

12968.diff (2.4 KB ) - added by Yuval Adam 14 years ago.
12978.diff (5.3 KB ) - added by Yuval Adam 14 years ago.
Updated proposed patch, with tests and docs
12978.2.diff (5.3 KB ) - added by Tim Graham 11 years ago.
12978.3.diff (5.4 KB ) - added by Tim Graham 10 years ago.

Download all attachments as: .zip

Change History (22)

comment:1 by Russell Keith-Magee, 14 years ago

Resolution: wontfix
Status: newclosed

I'm not aware of any CSS technique that works reliably for RSS or Atom feeds. This is largely because RSS is intended to be machine readable, not human readable.

If I'm mistaken, I'm open to suggestions on what technique we should be supporting.

comment:2 by intrepidweb, 14 years ago

Resolution: wontfix
Status: closedreopened

RSS feeds contain CSS and/or XSL stylesheets all the time. Here's a link on how to add an RSS stylesheet:

http://www.oreillynet.com/pub/a/network/2005/07/01/rss.html

I tried to include a list of major websites that use stylesheets in their RSS feeds, but the spam filter blocked it. However, if you check the feeds of CNN, CBS News, Reuters, Newsweek, Time and many more, you'll see XSL and/or CSS stylesheets linked in the source.

comment:3 by Jacob, 14 years ago

Triage Stage: UnreviewedSomeday/Maybe

comment:4 by Yuval Adam, 14 years ago

milestone: 1.3
Owner: changed from nobody to Yuval Adam
Status: reopenednew

by Yuval Adam, 14 years ago

Attachment: 12968.diff added

comment:5 by Yuval Adam, 14 years ago

Has patch: set
Needs documentation: set

Turns out easier than expected. saxutils supports adding XML processing instructions.

Added support for stylesheets, per the W3 spec, in the feedgenerator and in contrib.syndication.

comment:6 by Alex Gaynor, 14 years ago

Needs tests: set

by Yuval Adam, 14 years ago

Attachment: 12978.diff added

Updated proposed patch, with tests and docs

comment:7 by Yuval Adam, 14 years ago

Needs documentation: unset
Needs tests: unset
Triage Stage: Someday/MaybeUnreviewed

comment:8 by Yuval Adam, 14 years ago

Triage Stage: UnreviewedSomeday/Maybe

comment:9 by Luke Plant, 13 years ago

Type: New feature

comment:10 by Luke Plant, 13 years ago

Severity: Normal

comment:11 by Yuval Adam, 13 years ago

Easy pickings: unset
milestone: 1.31.4
Resolution: fixed
Status: newclosed
Triage Stage: Someday/MaybeReady for checkin
UI/UX: unset

comment:12 by Aymeric Augustin, 13 years ago

Resolution: fixed
Status: closedreopened
Triage Stage: Ready for checkinSomeday/Maybe

Revert apparent triage error.

comment:13 by Jacob, 13 years ago

milestone: 1.4

Milestone 1.4 deleted

comment:14 by Aymeric Augustin, 11 years ago

Triage Stage: Someday/MaybeAccepted

Accepting based on comment 2.

comment:15 by Aymeric Augustin, 11 years ago

Status: reopenednew

by Tim Graham, 11 years ago

Attachment: 12978.2.diff added

comment:16 by Tim Graham, 11 years ago

Updated the patch to apply cleanly, but noticed with the current implementation the stylesheets are only added to RSS feeds and not ATOM feeds. Does ATOM not support stylesheets?

comment:17 by Collin Anderson, 10 years ago

Is this still useful now that browsers have dropped the RSS button?

comment:18 by Tim Graham, 10 years ago

Patch needs improvement: set

I think it's still relevant. feedburner styles their feeds for example. I updated the patch to apply cleanly again, but we do need to add support for stylesheets in the ATOM feedgenerator (I started this, but didn't make the test updates). Also, I think we should avoid hardcoding text/css since stylesheets could also be xsl. Maybe extracting the stylesheet extension is enough or maybe the stylehseets attribute should be tuples of ('stylesheet', 'type'). Finally, the patch is also missing release notes.

by Tim Graham, 10 years ago

Attachment: 12978.3.diff added
Note: See TracTickets for help on using tickets.
Back to Top