#11791: Incorrect HTML Code rendered in list_editable enabled changelists
------------------------------------------+---------------------------------
          Reporter:  pa...@fragstore.net  |         Owner:  mlavin              
                                 
            Status:  assigned             |     Milestone:                      
                                 
         Component:  Contrib apps         |       Version:  1.1                 
                                 
        Resolution:                       |      Keywords:  list_editable, 
admin_list.py, items_for_result, opera
             Stage:  Unreviewed           |     Has_patch:  0                   
                                 
        Needs_docs:  0                    |   Needs_tests:  0                   
                                 
Needs_better_patch:  0                    |  
------------------------------------------+---------------------------------
Comment (by mlavin):

 It looks like line 245 in contrib/admin/templatetags/admin_list.py is
 missing the cell tags wrapper:
 {{{
 yield mark_safe(force_unicode(form[cl.model._meta.pk.name]))
 }}}
 Changing this to
 {{{
 yield mark_safe(u'<td>%s</td>' %
 force_unicode(form[cl.model._meta.pk.name]))
 }}}
 would fix the HTML validation but I'm not sure if it will introduce any
 issues with how the table will render.  Working out some appropriate tests
 for this.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/11791#comment:3>
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-upda...@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