[3.0] Put the admin search box back beside its heading, not in front of it - #9449
Merged
live627 merged 1 commit intoAug 10, 2026
Merged
Conversation
Every admin page has a bar holding the quick search form and the heading for the page. The form came first in the markup and .admin_search floats right, so the heading sat on the left and the search box on the right. Making .cat_bar a flex row in b81b1fa took the float out of play - a float is ignored on a flex item - so the two come out in source order now, and the search box has been sitting on the left with the heading pushed over beside it. Putting the heading first restores the old layout, this time through div.cat_bar > h3 { flex: 1 }, which is how the other bars already do it. Two places emit that bar, and both needed it: template_generic_menu_tabs(), which draws it on every admin page, and template_admin_search_results() for the results page, which has a bar of its own. It reads better in that order too: the heading says what the page is before the control that changes it. Measured across six pages, bar box unchanged at 1038-1052px wide and 35px tall: before form at x=72, heading at x=396 after heading at x=72, form at x=786 The float: right on .admin_search is inert now rather than wrong, so it is left alone; it would matter again if that bar ever stopped being flex. Signed-off-by: Mathias Albert <mathiaspapealbert@hotmail.com> Signed-off-by: albertlast <mathiaspapealbert@hotmail.com>
albertlast
force-pushed
the
3.0/admin-search-order
branch
from
August 9, 2026 12:52
e30788c to
482dfa6
Compare
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
Every admin page carries a bar holding the quick search form and the heading for the page. The form comes first in the markup, and
.admin_searchcarriesfloat: right, so the heading sat on the left and the search box on the right.b81b1fa(#9390) madediv.cat_bara flex row so a sibling could sit beside the heading. A float is ignored on a flex item, so the two now come out in source order, and the search box has been sitting on the left with the heading pushed over beside it. That was an unintended side effect of #9390 — mine — and this puts it right.Putting the heading first restores the old layout, this time through the rule the other bars already use:
Two places emit that bar and both needed the same change:
template_generic_menu_tabs()inGenericMenu.template.php— every admin pagetemplate_admin_search_results()inAdmin.template.php— the results page, which builds a bar of its ownIt reads better in that order anyway: the heading says what the page is before the control that changes it.
Verification
Each page loaded in a 1200px frame, measuring
getBoundingClientRect()on the bar and its two children.area=adminarea=manageboardsarea=permissionsarea=maintainarea=viewmembersarea=search(results)The bar box itself is unchanged throughout — 1038–1052px wide, 35px tall — so only the two children swap ends.
float: righton.admin_searchis inert now rather than wrong, so it is left alone — it would matter again if that bar ever stopped being a flex row.Issues References (Fixes|Related|Closes)
Related: #9390, #7933