#13564: Provide class attributes for form fields
-------------------------------+--------------------------------------------
          Reporter:  h3        |         Owner:  trebor74hr
            Status:  assigned  |     Milestone:            
         Component:  Forms     |       Version:  1.2-beta  
        Resolution:            |      Keywords:            
             Stage:  Accepted  |     Has_patch:  1         
        Needs_docs:  0         |   Needs_tests:  0         
Needs_better_patch:  0         |  
-------------------------------+--------------------------------------------
Comment (by trebor74hr):

 I agree with you, but one can notice that the patch I made solves only (2)
 and solution for (1) is given as an example in unit-test.
 In django documentation for this new feature one can put that example:
 {{{
 def get_field_css_classes(self, bf, css_classes):
     if "required" not in css_classes:
         css_classes.add("not_required")
     if "error" not in css_classes:
         css_classes.add("not_error")
     css_classes.add("django-%s" % bf.field.__class__.__name__.lower())
     return css_classes
 }}}

 Thus, I would suggest not to solve (1) at all, but only (2) (whether
 through this ticket or new one).

-- 
Ticket URL: <http://code.djangoproject.com/ticket/13564#comment:11>
Django <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 django-updates@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to