Opened 18 years ago

Closed 18 years ago

#4258 closed (fixed)

[newforms-admin] Models aren't grouped by application on index page

Reported by: Matthias Pronk <django@…> Owned by: Adrian Holovaty
Component: contrib.admin Version: newforms-admin
Severity: Keywords: newforms-admin
Cc: jm.bugtracking@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The newforms-admin doesn't group the models per application.

Attached patch fixes this.

Attachments (1)

admin-index.diff​ (1.5 KB ) - added by Matthias Pronk <django@…> 18 years ago.

Download all attachments as: .zip

Change History (7)

by Matthias Pronk <django@…>, 18 years ago

Attachment: admin-index.diff​ added

comment:1 by Simon G. <dev@…>, 18 years ago

Triage Stage: Unreviewed → Accepted
Version: other branch → newforms branch

comment:2 by anonymous, 18 years ago

Cc: jm.bugtracking@… added

comment:3 by Malcolm Tredinnick, 18 years ago

There's also an alternative patch in #4477. I haven't made any attempt to work out if one is slightly better than the other; both seem reasonable.

in reply to:  3 comment:4 by django@…, 18 years ago

Replying to mtredinnick:

There's also an alternative patch in #4477. I haven't made any attempt to work out if one is slightly better than the other; both seem reasonable.

I didn't know about 'django.utils.datastructures.SortedDict', which seems to be a slightly more elegant way of doing it.
So I would suggest using the patch in #4477.

comment:5 by Adrian Holovaty, 18 years ago

Version: newforms branch → newforms-admin

comment:6 by Adrian Holovaty, 18 years ago

Resolution: → fixed
Status: new → closed

(In [5441]) newforms-admin: Fixed #4258 and #4477 -- Changed admin index page to group by applications, and alphabetized things. Thanks to Matthias Pronk and Honza Kral for the patches; I ended up using a hybrid of both

Note: See TracTickets for help on using tickets.
Back to Top