Fix some compatibility and structure problems with admin templates.

Review Request #10956 — Created March 13, 2020 and submitted — Latest diff uploaded

Information

Review Board
release-4.0.x

Reviewers

The new revised administration UI templates left out some crucial blocks
that were necessary for some extensions to work, and broke the ability
to override the form submission buttons.

This was largely due to an attempt to simplify the templates, removing
some Djangoisms that didn't appear to make sense, and to also utilize
some other Djangoisms for calculating state needed for form buttons.
What we had just wasn't comprehensive enough.

What we're doing now is bringing back some of the blocks, fixing up some
bad assumptions with the buttons (and allowing them to be better
overridden), and also fixing some incorrect usage of some CSS components.

There's also fixes to the form that previously caused dependencies on an
adminform variable. Most of these are fixed in this change, with a fix
to media coming in a follow-up change.

There's room for improvement here, particularly when it comes to the
form buttons. Eventually we'll want to move away from Django's
{% submit_row %} template tag (which breaks if not working with
models), but that's a task for another day.

Tested the model change pages, settings pages, and the Power Pack
configuration page (which was broken before). They all now render as
expected.

Commits

Files

    Loading...