docs: restore full-width search layout - #131
Merged
Merged
Conversation
Move the Doxygen Awesome theme toggle into the documentation header, keep the search row full width, and assert the layout at desktop and mobile viewports.
Treat horizontal or vertical separation as non-overlap now that the theme toggle lives above the full-width search row.
Wait for the custom header container to own the dark-mode toggle before measuring the generated layout.
Relocate and deduplicate the Doxygen Awesome toggle before measuring the generated layout.
Allow the theme resize handler and custom relocation observer to settle before asserting generated page geometry.
Relocate the toggle immediately from mutation and resize handlers so the theme cannot leave it in the search row after layout changes.
Reconcile the custom header placement periodically so Doxygen Awesome cannot move the toggle back into the search row.
Print the measured toggle parent and geometry when diagnosing generated documentation layout in CI.
Validate the rendered header placement and dimensions instead of depending on a fragile parent id after Doxygen rewrites the header DOM.
Keep the generated documentation smoke test focused on stable placement and geometry assertions without temporary DOM diagnostics.
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
1 participant
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.
Summary\n- move the Doxygen Awesome theme toggle into a dedicated header container\n- restore the full-width desktop sidebar search row with project-specific CSS\n- extend layout smoke coverage to wide desktop and assert search width/toggle placement\n\n## Validation\n- node --check for both custom scripts\n- git diff --check\n- Playwright smoke in publish workflow\n- full CI workflow