Skip to content

Move theme, language, and copy switchers to the burger menu (#199) - #467

Open
Raghaddahi wants to merge 3 commits into
mainfrom
feature/move-switchers-to-burger-menu
Open

Move theme, language, and copy switchers to the burger menu (#199)#467
Raghaddahi wants to merge 3 commits into
mainfrom
feature/move-switchers-to-burger-menu

Conversation

@Raghaddahi

@Raghaddahi Raghaddahi commented Aug 5, 2026

Copy link
Copy Markdown
Member

Relates to #199

Description

Moves the theme toggle, copy button, and language selector out of .page-tools (above the article content) and into the burger menu overlay on mobile, so they no longer take up space above the content on small screens.

Approach: render the tools in both places (.page-tools in base.html and .primary-nav__tools in navigation.html) and toggle visibility with CSS breakpoints — no JS DOM manipulation. The theme toggle button id was replaced with a .js-theme-toggle class so the JS wires up both instances.

Worth careful review:

  • The copy-button Bootstrap dropdown inside the scrollable .primary-nav overlay may clip instead of overlay content on mobile — verify on a narrow viewport.
  • page/request were added to the header inclusion-tag context; request is currently unused (kept defensively).

Testing

  • before
Screenshot From 2026-08-05 17-18-55 Screenshot From 2026-08-05 17-18-50
  • After
Screenshot From 2026-08-05 17-19-04 Screenshot From 2026-08-05 17-18-39

AI usage

  • Agent: AI assisted with the PR description and review only. The code was written by a human; AI reviewed the diff and prepared this description.

@RealOrangeOne
RealOrangeOne temporarily deployed to wagtail-guid-feature-mo-yusezs August 5, 2026 14:48 Inactive
@RealOrangeOne
RealOrangeOne temporarily deployed to wagtail-guid-feature-mo-yusezs August 5, 2026 16:13 Inactive
@RealOrangeOne
RealOrangeOne temporarily deployed to wagtail-guid-feature-mo-yusezs August 5, 2026 16:19 Inactive
@Raghaddahi
Raghaddahi requested a review from thibaudcolas August 5, 2026 16:19
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