Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix tag render #1

Open
wants to merge 1 commit into from

1 participant

@tedchoward

I don't know why, but the rendering code for tags was escaping the html entities when it rendered the blog. This fixes it for me.

@tedchoward tedchoward Changed the rendering of tags to not result in escaped HTML tags.
For whatever reason, the html a tags were being escaped, so I changed
the tag rendering to use a basic each loop that generated the link in
the closure.
7fc20b0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 31, 2010
  1. @tedchoward

    Changed the rendering of tags to not result in escaped HTML tags.

    tedchoward authored
    For whatever reason, the html a tags were being escaped, so I changed
    the tag rendering to use a basic each loop that generated the link in
    the closure.
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 1 deletion.
  1. +4 −1 app/views/blog_entries/_tag_list.html.erb
View
5 app/views/blog_entries/_tag_list.html.erb
@@ -1,3 +1,6 @@
<div class="tags">
- <%= t('tags') + ': ' + e.tag_list.map {|tag| link_to tag, tag_path(tag) } * ", " %>
+ <%= t('tags') %>:
+ <% e.tag_list.each do |tag| %>
+ <%= link_to(tag, tag_path(tag)) %>
+ <% end %>
</div>
Something went wrong with that request. Please try again.