Skip to content

799 page titles - #800

Merged
AdrianDAlessandro merged 3 commits into
mainfrom
799-page-titles
Oct 9, 2026
Merged

AdrianDAlessandro merged 3 commits into
mainfrom
799-page-titles

Conversation

@AdrianDAlessandro

Copy link
Copy Markdown
Collaborator

Description

This PR fixes a small bug where none of the pages has unique titles because the {% block title %} tag wasn't working correctly. Previously it was defined inside the head.html snippet, which was imported into the base template with an include statement. The title blocks were then updated in sub-pages. This did not work because the block tags are resolved before the include tags. So all the titles were the same default of "DIRECT Framework".

Done in this PR

  • Moved the block title tags from head.html to the base.html and base.auth.html templates
  • Added a sensible title block to all the templates of visible pages
  • Resolved inconsistency with "Sign out" being used instead of "Log out" in a few instances

Fixes #799

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Technical work (non-breaking, change which is work as part of a new feature)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • The documentation builds and looks OK (eg. mkdocs serve)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

@codecov

codecov Bot commented Oct 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@anenadic anenadic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@AdrianDAlessandro
AdrianDAlessandro merged commit c1ad8f9 into main Oct 9, 2026
6 checks passed
@AdrianDAlessandro
AdrianDAlessandro deleted the 799-page-titles branch October 9, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Page titles not updating

2 participants