Opened 91 minutes ago

Last modified 38 minutes ago

#36887 assigned Bug

Unnecessary space introduced at top of changelist pages in 6.0

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

Description

See ​report and screenshot, #36055 solved an overlap issue at the top of changelist pages by splitting the header and action button into two rows, when we could have just used flex layout to fix the overlap.

I have a patch ready along those lines.

Change History (3)

comment:1 by Natalia Bidart, 88 minutes ago

Triage Stage: Unreviewed → Accepted
UI/UX: set

Thank you!

comment:2 by Jacob Walls, 48 minutes ago

Well, my intended fix just re-causes #36331. I'm not sure we should have closed with #36331 with a revert; we could have just doc'd it. django-filer (source of bug report) already coped with the change, see ​PR.

What is our backwards compatibility guarantee for extending admin template blocks?

comment:3 by Jacob Walls, 38 minutes ago

Has patch: set
Needs tests: set
Severity: Release blocker → Normal

Given that the best solution here involves a change to template composition, I'm not convinced we should backport. I already have something ready for discussion, so I'll post it anyway as a ​draft PR.

It's just a little whitespace; we can defer it to 6.1 and ensure more testing occurs.

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