[3.0] Theme split (wave 3, part 5) — Move the top bar into the header as a user panel - #9373
Merged
live627 merged 2 commits intoAug 8, 2026
Merged
Conversation
albertlast
force-pushed
the
3.0/theme-header-panel
branch
from
August 5, 2026 05:56
6dc86f5 to
3ee0f5f
Compare
Closed
The top bar was a separate band above the header holding the user links, the language picker and the search box, which left the header itself holding nothing but the forum title and the logo. They merge: the header now has the title at one end and a user panel at the other. Keeps the search box and the language picker, which the theme branch drops from the template while leaving their CSS behind. The footer's Go up link pointed at #top_section, so it now points at Signed-off-by: albertlast <mathiaspapealbert@hotmail.com>
They hang off #top_info, which is positioned, and were anchored to its start edge. That was fine while #top_info sat at the left of the page, but it now sits at the end of the header, so a 300px wide menu ran off the right of the screen and gave the page a horizontal scrollbar on a narrow window. Anchored to the end edge instead, with logical properties so it works the same way in a right to left language. Signed-off-by: albertlast <mathiaspapealbert@hotmail.com>
Contributor
|
...and now these have merge conflicts and need a rebase |
Collaborator
Author
|
just gimme a minute |
albertlast
force-pushed
the
3.0/theme-header-panel
branch
from
August 8, 2026 05:50
3ee0f5f to
a83d69f
Compare
Collaborator
Author
|
everything fine now @live627 |
live627
approved these changes
Aug 8, 2026
This was referenced Aug 9, 2026
live627
pushed a commit
that referenced
this pull request
Aug 9, 2026
Reverts the layout half of #9373. The top bar goes back above the header holding the user links, the language picker and the search box, and the header goes back to the forum title and the logo on their own between it and the main menu. Reported as #9423, and confirmed with the author of #7933 as the direction to take: the merged header is not what people expect of this theme, whatever the theme branch does. The maintenance page and the profile export follow the forum's header, so their wrapper goes back onto the header element with it. Everything #9374 and #9375 changed stays as it is - the menu is still a band across the page and the footer is still a flex row. The Go up link points at the top bar again, since that is the first thing on the page once more. Signed-off-by: albertlast <mathiaspapealbert@hotmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Part 5 of wave 3 of the #7933 split. Stacked on #9372, #9371, #9370 and #9369 — the diff of this one is its last two commits.
This is the first part of wave 3 that changes how the forum looks.
The top bar was a separate white band above the header holding the user links, the language picker and the search box, which left the header itself holding nothing but the forum title and the logo. They merge.
#top_sectiongoes, and the header becomes:The panel is a grid, so the float classes those three carried are gone.
#headeris now full width with the wrapper inside it lining its contents up with the rest of the page.Two things kept that the theme branch loses
Its
index.template.phpno longer emits the search box or the clock, whileindex.css,responsive.cssandrtl.csson that same branch still carry#search_formrules. Leftover CSS for markup that no longer exists is usually the sign of something dropped by accident rather than on purpose, so both are kept here. The clock stays where it is for now and moves with the part that deals with the section below the header.A regression this introduced, and the fix
Moving
#top_infofrom the left of the page to the end of the header broke the profile, PM and alerts dropdowns..dropmenu, #top_info { position: relative }has always been there, so those menus are positioned against#top_info, and they were anchored to its start edge withleft: 0. While#top_infowas on the left that put them at the left of the page. With#top_infoat the end of the header,left: 0plus theirmin-width: 25emsent them off the right of the screen — 163px past the edge at a 701px viewport, with a horizontal scrollbar to go with it.They are anchored to the end edge instead, using logical properties so that right to left behaves too:
The footer's "Go up" link also pointed at
#top_section, so it now points at#header.Verification
Logged in as an admin in the running forum, since the interesting parts of the header only exist for a logged-in user.
Each of the three dropdowns opened and measured, at a 701px viewport:
Before the fix, at that same width:
left 547, right 864, 163px past the edge, horizontal scrollbar present. On the parent branch:left 0, right 317, no scrollbar — which is what identified this as mine rather than something already broken.At 1401px, the menu's end edge lines up with
#top_info's end edge, in both directions:Every
var()inindex.cssstill resolves againstvariables.css, with nothing referenced but undefined.Issues References (Fixes|Related|Closes)
Related to #7933.