Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move guidance pages side nav out of main content #5237

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Move guidance pages side nav out of main content #5237

wants to merge 1 commit into from
+21 −13

Commits on Oct 3, 2024

  1. Move guidance pages side nav out of main content 

    Also stops content-template.html swapping the
withoutnav_template.html layout for
admin_template.html, so removing
withoutnav_template.html from the inheritance
chain.

This was done, simply because it's odd to have a
template with a side navigation use a layout
called withoutnav_template.html.

It also lets us remove the code for error
summaries because, as far as I can see, no pages
using content_template.html have forms. I inferred
this by searching their view code for the
error_summary_enabled flag.
    @tombye
    tombye committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    6f56f01 View commit details
    Browse the repository at this point in the history