#3512: [patch] Add HTML classes to fields in newforms as_methods: "required" &
"error"
-------------------------------------------------+--------------------------
Reporter: Waylan Limberg <[EMAIL PROTECTED]> | Owner:
adrian
Status: new | Component:
django.newforms
Version: SVN | Resolution:
Keywords: | Stage:
Design decision needed
Has_patch: 1 | Needs_docs: 1
Needs_tests: 1 | Needs_better_patch: 0
-------------------------------------------------+--------------------------
Comment (by [EMAIL PROTECTED]):
I strongly prefer this fix to the one suggested in #3515. There is a bug
in the patch at line 122 though, it says:
{{{class_list = html_class_list # Reset for each loop.}}}
This does in fact not reset the list for each loop step, but results in
"required" being appended over and over again. In my example code I tried
I got {{{class="required required required"}}} in one case :P
If this is changed to {{{class_list=[]}}} everything is fine as far as I
can see. I would very much like this patch to go into trunk.
--
Ticket URL: <http://code.djangoproject.com/ticket/3512#comment:5>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
