Opened 12 years ago
Closed 12 years ago
#20111 closed Cleanup/optimization (fixed)
Admin message levels are too ambigious
Reported by: | Kaspars Sprogis | Owned by: | Kaspars Sprogis |
---|---|---|---|
Component: | contrib.admin | Version: | 1.5 |
Severity: | Normal | Keywords: | messages |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Currently admin uses messages.INFO level for both - info messages and success messages.
But even text states it should be messages.SUCCESS instead of messages.INFO:
"The %(name)s "%(obj)s" was added successfully"
Therefore it is impossible to use different styles for these levels.
One more example:
In changelist, if you don't choose any checkbox and use some action, you receive message:
"Items must be selected in order to perform actions on them..."
which looks exactly like success messages and gives us feeling everything went ok, but it wasn't and this is confusing.
I suggest to correct these message levels, so that they are more precise and we could style them separately.
Patch is on it's way.
Attachments (2)
Change History (9)
comment:1 by , 12 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 12 years ago
by , 12 years ago
Attachment: | messages_level_old.png added |
---|
by , 12 years ago
Attachment: | messages_level_new.png added |
---|
comment:3 by , 12 years ago
comment:4 by , 12 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:6 by , 12 years ago
Triage Stage: | Accepted → Ready for checkin |
---|
I've looked over the patch. It's simple and solid. No tests/docs needed.
Marking Ready For Checkin as per https://docs.djangoproject.com/en/dev/internals/contributing/triaging-tickets/#ready-for-checkin
comment:7 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Patch is available on my github branch. Opened PR.
See diff: https://github.com/darklow/django/compare/ticket_20111