Code

Opened 7 months ago

Last modified 6 months ago

#21220 new Cleanup/optimization

Improve admin user friendliness

Reported by: ryangallen Owned by: nobody
Component: contrib.admin Version: 1.5
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: yes

Description

https://github.com/django/django/pull/1707

Links to the admin index and site index in the header
Green success messages
Tone down the red error messages
Min-width for date and time fields (could not read whole date in input box when clicking 'Today')
A handful of other subtle details

Attachments (5)

01-overview.png (115.0 KB) - added by anonymous 7 months ago.
02-errors.png (116.8 KB) - added by ryangallen 7 months ago.
Errors
03-successes.png (119.7 KB) - added by ryangallen 7 months ago.
Successes
06-link-to-admin-home.png (80.4 KB) - added by ryangallen 7 months ago.
Link to Admin Home
07-link-to-site.png (77.7 KB) - added by ryangallen 7 months ago.
Link to Site Home

Download all attachments as: .zip

Change History (13)

Changed 7 months ago by anonymous

Changed 7 months ago by ryangallen

Errors

Changed 7 months ago by ryangallen

Successes

Changed 7 months ago by ryangallen

Link to Admin Home

Changed 7 months ago by ryangallen

Link to Site Home

comment:1 Changed 7 months ago by ryangallen

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 7 months ago by timo

  • Easy pickings unset
  • Patch needs improvement set
  • Triage Stage changed from Unreviewed to Accepted

It's better not to include so many changes in one ticket. For example, the issue of linking the header has already been proposed in #13749.

I'm inclined to avoid changing colors at this time -- otherwise we'll end up having someone else come along and make different suggestions. I know a redesign or reskins of the admin has been proposed on django-developers in the past, but as far as I know there are no active efforts on this front.

If you could split the pull request into smaller parts (for example, the width changes that Marc said looked good) that will make it easier to review and merge.

comment:3 Changed 7 months ago by ryangallen

For what it's worth, obviously design choices are a special case since everyone has their own idea of what's best. But isn't the whole point of open-source to have contributors come and make different suggestions? Django has an extraordinarily powerful backend but is falling behind on the frontend, where non-technical users are experiencing it. There are just a handful of styling updates in this patch, but they modernize the admin design immensely, while holding true to Django's personality. Could be a good start to reactivating those efforts from before.

Thanks -- I'll get to work splitting up the changes.

comment:4 Changed 7 months ago by ryangallen

Changes have been split up into smaller pull requests:

https://github.com/django/django/pull/1713 - min-width for date and time inputs
https://github.com/django/django/pull/1715 - update admin alerts to appropriate colors
https://github.com/django/django/pull/1712 - header and modules design improvements
https://github.com/django/django/pull/1714 - link to site home in header if it exists
https://github.com/django/django/pull/1710 - admin index link in header
https://github.com/django/django/pull/1711 - admin/base.css cleanup (indentation, minor optimization)

comment:5 Changed 7 months ago by Tim Graham <timograham@…>

In 259a6ea82cfb8bf242449666e4b643a19e54a423:

Added a link in admin branding block to admin index.

Refs #13749 and #21220.

comment:6 Changed 7 months ago by Tim Graham <timograham@…>

In 3eab22a0452f976727ba21b4b6e25dabba85111e:

Added min-width for date and time inputs in admin.

refs #21220.

comment:7 Changed 7 months ago by Tim Graham <timograham@…>

In 935001c4f1ab8660c76323854945e9e660514046:

Cleaned up admin CSS for consistency.

  • header selected twice -merged
  • match indentation throughout
  • color names to hex

refs #21220.

comment:8 Changed 6 months ago by Tim Graham <timograham@…>

In f719d4afc74e750b34247a574f800598c3daf1cc:

Updated admin alert colors

refs #21220.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as new
The owner will be changed from nobody to anonymous. Next status will be 'assigned'
as The resolution will be set. Next status will be 'closed'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.