#17050: add id attribute to individual model divs in the admin index
-------------------------------+------------------------------------
     Reporter:  scytale        |                    Owner:  nobody
         Type:  New feature    |                   Status:  new
    Component:  contrib.admin  |                  Version:  SVN
     Severity:  Normal         |               Resolution:
     Keywords:  admin          |             Triage Stage:  Accepted
    Has patch:  1              |      Needs documentation:  0
  Needs tests:  0              |  Patch needs improvement:  1
Easy pickings:  1              |                    UI/UX:  1
-------------------------------+------------------------------------
Changes (by julien):

 * needs_better_patch:  0 => 1
 * stage:  Unreviewed => Accepted


Comment:

 That sounds reasonable. Using slugify on the app's title would probably be
 error prone, so instead I'd simply pass the app_label through to the
 template and use that instead. Also, you'd need to prefix the id, for
 example with "app-", to be sure it doesn't conflict with other styles
 (e.g. if you have an app called "changelist").

-- 
Ticket URL: <https://code.djangoproject.com/ticket/17050#comment:2>
Django <https://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