#2624 Improve HTML generated by Scaffold - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#2624 ✓duplicate

Improve HTML generated by Scaffold

Reported by Emili Parreño | May 8th, 2009 @ 07:49 AM | in 2.x

This patch improves the HTML code generated by scaffold including all elements within <p></p>. This markup is HTML Strict valid.

for example, in show.html.erb the actual scaffold generates


<%= link_to 'Back', posts_path %>

and with this patch, the result is


<p><%= link_to 'Back', posts_path %></p>

This patch affects: index.html.erb, show.html.erb, edit.html.erb and new.html.erb

You flagged this item as spam.
Yaroslav Markin May 8th, 2009 @ 01:09 PM
You should probably use div for that — more generic.
You flagged this item as spam.
Daniel Schierbeck May 8th, 2009 @ 01:18 PM
Yaroslav: I disagree; this seems like perfectly correct use of the paragraph element type.

+1 from me; I think it's a good idea to use semantically correct XHTML.
You flagged this item as spam.
Ken Collins May 8th, 2009 @ 02:24 PM
DIV is way more appropriate. I hate seeing

tags for block level element usage.
CancelProfileIsBroken May 8th, 2009 @ 03:13 PM
- State changed from “new” to “duplicate”
See ticket #1760 where these issues are already being discussed.