#31034 closed New feature (fixed)
Add a navigation sidebar to the admin
| Reported by: | Tom Carrick | Owned by: | Tom Carrick |
|---|---|---|---|
| Component: | contrib.admin | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | yes |
Description
Title says it all, really. The idea is to add an optional (probably enabled by default?) sidebar for navigation, doing roughly what the current homepage does. For now, it'll be a simple implementation only for large screens, and hopefully it can be iterated on in the future.
I opened a discussion here, only got one response but it was positive and I think it's not too controversial if it's behind a flag: https://groups.google.com/forum/#!topic/django-developers/PAZw8chsp2g
Change History (20)
comment:1 by , 5 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 5 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:3 by , 5 years ago
| Has patch: | set |
|---|---|
| Needs documentation: | set |
| Needs tests: | set |
| UI/UX: | set |
Needs tests and docs but wanted to get some feedback first.
comment:4 by , 5 years ago
| Needs tests: | unset |
|---|
comment:5 by , 5 years ago
| Needs documentation: | unset |
|---|
comment:6 by , 5 years ago
To control its display, I'd think an admin site attribute would be preferred to a setting. Generally, settings are only added as a last resort.
comment:7 by , 5 years ago
| Patch needs improvement: | set |
|---|
comment:8 by , 5 years ago
| Patch needs improvement: | unset |
|---|
Thanks. I've fixed that, now I'm just having a problem figuring out the test failure.
comment:9 by , 5 years ago
| Patch needs improvement: | set |
|---|
I posted to the mailing list thread to see if we can find a Design/UX/CSS expert to input on the PR.
If not, I think we should proceed. It's a improvement, and we can iterate on it over time.
There's one CSS issue that we should resolve (at least) commented on the PR.
comment:10 by , 5 years ago
| Patch needs improvement: | unset |
|---|
comment:11 by , 5 years ago
| Patch needs improvement: | set |
|---|
comment:12 by , 5 years ago
| Patch needs improvement: | unset |
|---|
comment:13 by , 5 years ago
| Triage Stage: | Accepted → Ready for checkin |
|---|
OK, thanks. Accepting on the basis of the discussion. 👍