[Jifty-commit] r1351 - in jifty/trunk: lib/Jifty/Web/Form lib/Jifty/Web/Form/Field

jifty-commit at lists.jifty.org jifty-commit at lists.jifty.org
Thu Jun 22 15:50:36 EDT 2006

Author: seanmil
Date: Thu Jun 22 15:47:23 2006
New Revision: 1351

   jifty/trunk/   (props changed)

 r574 at pc102:  sean | 2006-06-22 15:47:16 -0400
 * Backed out r1349, re-add autocomplete="off" tag

Modified: jifty/trunk/lib/Jifty/Web/Form/Field.pm
--- jifty/trunk/lib/Jifty/Web/Form/Field.pm	(original)
+++ jifty/trunk/lib/Jifty/Web/Form/Field.pm	Thu Jun 22 15:47:23 2006
@@ -415,11 +415,15 @@
 =head2 other_widget_properties
 If your widget subclass has other properties it wants to insert into the html
-of the main widget and you haven't subclassed render_widget you can subclass
-this to do so.
+of the main widget and you haven't subclassed render_widget then you can just
+subclass this.
+If you have subclassed render_widget then just stick them in your local sub
+We use this for marking password fields as not-autocomplete so the browser does
+not try to use its form autocompletion on them.
-Otherwise, if you have already subclassed render_widget, just stick them in
-your local sub render_widget.

Modified: jifty/trunk/lib/Jifty/Web/Form/Field/Password.pm
--- jifty/trunk/lib/Jifty/Web/Form/Field/Password.pm	(original)
+++ jifty/trunk/lib/Jifty/Web/Form/Field/Password.pm	Thu Jun 22 15:47:23 2006
@@ -25,6 +25,10 @@
 Never render anything for autocompletion.
+No browser-based form auto-completion in password fields ;)
+Note: This has nothing to do with Jifty's Autocomplete mechanism.
 sub render_autocomplete {''}

More information about the Jifty-commit mailing list