Fix mis-use of span within fieldset.
xinming [Mon, 22 Jun 2009 16:17:32 +0000 (16:17 +0000)]
div and span is inter-changable as they can be styled via css, But we should
really use div element within fieldset instead of span element by default. I
believe using div here is the sane default. ;-)

below urls are for references:
http://www.learnwebdesignonline.com/htmlcourse/span-div.htm
http://archive.thedevweb.com/2007/0502.html

share/skin/base/layout/field/container.tt
share/skin/base/layout/field/mutable.tt

index 78bbdc0..c504e55 100644 (file)
@@ -11,8 +11,8 @@
 
 =for layout field
 
-<span class="field_container_field">
+<div class="field_container_field">
   [% call_next %]
-</span>
+</div>
 
 =cut
index d7acb49..d21e979 100644 (file)
@@ -1,6 +1,6 @@
 =for layout widget
 
-<span class="[% field_class %]">[% label_fragment %] [% field %] [% message_fragment %]</span>
+<div class="[% field_class %]">[% label_fragment %] [% field %] [% message_fragment %]</div>
 
 =for layout label