From b3e6b6eb66c61f04b18119b0c4eb8619f8e26a1f Mon Sep 17 00:00:00 2001
From: Lance Edgar <lance@edbob.org>
Date: Wed, 15 Aug 2012 09:30:41 -0700
Subject: [PATCH] improve fieldset template

---
 edbob/pyramid/templates/forms/fieldset.mako | 46 ++++++++++++---------
 1 file changed, 26 insertions(+), 20 deletions(-)

diff --git a/edbob/pyramid/templates/forms/fieldset.mako b/edbob/pyramid/templates/forms/fieldset.mako
index 77789b9..7af621b 100644
--- a/edbob/pyramid/templates/forms/fieldset.mako
+++ b/edbob/pyramid/templates/forms/fieldset.mako
@@ -9,27 +9,33 @@
 
   % for field in fieldset.render_fields.itervalues():
 
-      <div class="field-wrapper ${field.name}">
-        % for error in field.errors:
-            <div class="field-error">${error}</div>
-        % endfor
-        ${field.label_tag()|n}
-        <div class="field">
-          ${field.render()|n}
-        </div>
-        % if 'instructions' in field.metadata:
-            <span class="instructions">${field.metadata['instructions']}</span>
-        % endif
-      </div>
+      % if field.requires_label:
+          <div class="field-wrapper ${field.name}">
+            % for error in field.errors:
+                <div class="field-error">${error}</div>
+            % endfor
+            ${field.label_tag()|n}
+            <div class="field">
+              ${field.render()|n}
+            </div>
+            % if 'instructions' in field.metadata:
+                <span class="instructions">${field.metadata['instructions']}</span>
+            % endif
+          </div>
 
-      % if (fieldset.focus == field or fieldset.focus is True) and not _focus_rendered:
-          % if not field.is_readonly():
-              <script language="javascript" type="text/javascript">
-                $(function() {
-                    $('#${field.renderer.name}').focus();
-                });
-              </script>
-              <% _focus_rendered = True %>
+          % if not _focus_rendered and (fieldset.focus == field or fieldset.focus is True):
+              % if not field.is_readonly():
+                  <script language="javascript" type="text/javascript">
+                    $(function() {
+                        % if hasattr(field.renderer, 'focus_name'):
+                            $('#${field.renderer.focus_name}').focus();
+                        % else:
+                            $('#${field.renderer.name}').focus();
+                        % endif
+                    });
+                  </script>
+                  <% _focus_rendered = True %>
+              % endif
           % endif
       % endif