Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use safe and linebreaks filters for the text #74

Conversation

kampfschlaefer
Copy link
Contributor

This is just small sugar to increase the experience and make the blog usable for more then just a paragraph per post.

@@ -44,7 +44,7 @@ It works! But we want them to be displayed in a way we created earlier in the __
<div>
<p>published: {{ post.published_date }}</p>
<h1><a href="">{{ post.title }}</a></h1>
<p>{{ post.text }}</p>
<p>{{ post.text|safe|linebreaks }}</p>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why safe?

Just add the linebreaks filter so the new users don't get distressed by their carefully prepared multi-paragraph blogposts on how great Django is becoming garbled into an unreadable one-paragraph text blob...
oinopion added a commit that referenced this pull request Jul 27, 2014
…e_formatting

use safe and linebreaks filters for the text
@oinopion oinopion merged commit b2f170a into DjangoGirls:master Jul 27, 2014
@kampfschlaefer kampfschlaefer deleted the add_safe_marker_to_preserve_formatting branch July 28, 2014 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants