EncyclopediaOfLife/eol — comment on commit ef7b6563f13254bf99b4ecb17943ef29ce61e2a1
File:  line 0
By lwalley:

One thing I think I would like to change is when this the link tag is added to the page. I think we should be more cautious about where is it used so I think moving it out of the application layout and into individual views (using content_for :head) is wise, plus we only need it on pages the search engines can index, we don't need the overhead on the other pages.

My only hesitation with using content_for :head is how much rendering time that adds to the view. Is that something I should worry about?

On a separate note Patrick mentioned using the params variable instead of manipulating the request.url string. I thought about this as well and I'm not sure which would be better, but the rationale I used for choosing to manipulate the request.url is that I am calling this method after the controller action has executed and I wasn't sure if the params variable would persist with the same values as the request URL during the execution of the controller action?