albatrossandco / brubeck_cms

A Django-based open source CMS for newspapers
Other
5 stars 1 forks source link

Rework byline display #32

Open allanjamesvestal opened 13 years ago

allanjamesvestal commented 13 years ago

Let's change the way in which bylines are generated.

I would suggest two methods on the "Content" object — one for generating bylines with links to staffers' pages, the other for just flat text.

Additionally, doing a render_byline and render_byline_html methods this way could negate the need for any if statements in the templates. We could also add a DEFAULT_BYLINE setting so that that boilerplate need only be written in one place.

myersjustinc commented 13 years ago

Sounds reasonable to me.