[3.0] Centres the forum title and logo in the header - #9425
Closed
albertlast wants to merge 1 commit into
Closed
Conversation
The header used to be a band of its own that bottom-aligned its contents, so the title carried a top-heavy padding and the logo a top margin to drop them onto that line. The header lines everything up in the middle now, and those two rules push the title and the logo eight pixels below the user panel beside them. The title keeps the inline padding that lines it up with the page content below it, as padding-inline. The logo needs nothing. Signed-off-by: albertlast <mathiaspapealbert@hotmail.com>
This was referenced Aug 9, 2026
Collaborator
Author
|
Superseded by #9427. That one puts the header back to a band of its own between the top bar and the menu, per the decision on #9423, so the row this centred things in no longer exists — the top-heavy padding on |
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
Two rules in
index.cssare leftovers from the header as it was before #9373, when it was a band of its own that bottom-aligned its contents:The header lines its contents up in the middle now (
#header .content_wrapper { align-items: center }), so both of those push the title and the logo below the user panel sitting beside them. Measured centres on the board index at 1280px, before and after:The title keeps the horizontal padding that lines it up with the page content below it — as
padding-inline, so the RTL sheet does not need its own rule. Its start edge stays at x=76 against the breadcrumb's 75. The logo needs nothing; RTL's ownimg#smflogo { margin-right: 1em }still applies.Header height is unchanged at 93px either way —
--header-height: 80pxsets the floor.This is the visible half of what #9423 reports. The rest of that report is the intended layout from #9373 and #9374, which merged the old
#top_sectioninto the header; that matches the end state of #7933 and is not changed here.Issues References (Fixes|Related|Closes)
Related #9423
Related #7933