#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 Simon Litchfield <[EMAIL PROTECTED]>):

 We're getting wires crossed - I don't think anyone here lacks CSS
 understanding.
 
  1. The reason 3515's CSS control is bad is because of MVC leakage;
 although it's convenient, model is perhaps not the cleanest place for CSS
 definition
  1. #id_ CSS selectors can be used to control individual field sizes,
 styles etc
  1. Another alternative is to make a css class name '''filter''' for
 templates, I'll think some more and post thoughts about that later
  1. 3512 is the more MVC-compliant approach but '''needs implementing for
 all output methods''' not just the as_ functions
 
 Nice work people now lets finish it off and hopefully get it into trunk.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/3512#comment:13>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to