From 4902064aab084ead299d659b146b3a609da837d0 Mon Sep 17 00:00:00 2001
From: Jan Dittberner <jan@dittberner.info>
Date: Fri, 11 Jun 2010 23:40:00 +0200
Subject: [PATCH] hide showall field for non ECMAScript clients

---
 ddportfolioservice/templates/showform.mako       | 2 +-
 ddportfolioservice/templates/showformscript.mako | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/ddportfolioservice/templates/showform.mako b/ddportfolioservice/templates/showform.mako
index a569927..3d12ec8 100644
--- a/ddportfolioservice/templates/showform.mako
+++ b/ddportfolioservice/templates/showform.mako
@@ -42,7 +42,7 @@ ${h.form(h.url_for(action='urllist'), method='get')}
     ${h.text('email',
              h.escape(request.params.get('email', None), True), id='email')}<br />
   </div>
-  <div id="showallfield">
+  <div id="showallfield" class="hidden">
     ${h.checkbox('showall', value='1', checked=False, id='showall')}
     <label for="showall">${_(u'Show all form fields')}</label><br />
   </div>
diff --git a/ddportfolioservice/templates/showformscript.mako b/ddportfolioservice/templates/showformscript.mako
index ab9ba82..8c21447 100644
--- a/ddportfolioservice/templates/showformscript.mako
+++ b/ddportfolioservice/templates/showformscript.mako
@@ -118,7 +118,7 @@ $(document).ready(function() {
         $(defaultreadonlydivs[index]).hide();
     }
 
+    $('#showall').attr('checked', false).change(onChangeShowAll);
+    $('#showallfield').show();
     $('#email').blur(onBlurEmail).focus();
-    $('#showall').attr('checked', false);
-    $('#showall').change(onChangeShowAll);
 });