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.

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

Theme updates - breadcrumbs, side and mobile navigation. #48

Merged
merged 12 commits into from
Feb 19, 2024

Conversation

EarthlingDavey
Copy link
Contributor

@EarthlingDavey EarthlingDavey commented Feb 16, 2024

Please review / merge #46 first. It will simplify the files changed for this PR, as this branch is based off theme-updates.

  • Simplify the content-wrapper css with display:flex
  • Remove page-left-centre-right because that template is used for 99% of pages, and is better to be the default.
  • Add body classes to match squiz html
  • Add breadcrumbs class and template part.
  • Breadcrumbs can use short_name metadata. Similar behaviour to squiz dashboard.
  • Create Meta class for shortTitle.
  • Add mobile nav to footer.

@EarthlingDavey EarthlingDavey merged commit 4fc7aad into main Feb 19, 2024
7 checks passed
@wilson1000 wilson1000 deleted the theme-updates-2 branch June 11, 2024 18: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.

2 participants