Skip to content

Conversation

@jijo-OO7
Copy link
Contributor

@jijo-OO7 jijo-OO7 commented Jan 14, 2026

Header tabs overlap when search expands [Fixes #6332]

BEFORE

problem

Proposed Changes

  • Fixed navigation layout shift when search is activated
  • Restructured header to use separate sticky navigation container
  • Maintained responsive design across all breakpoints

Reference: https://squidfunk.github.io/mkdocs-material/tutorials/

AFTER

Screenshot (17) Screenshot (18)

@netlify
Copy link

netlify bot commented Jan 14, 2026

Deploy Preview for knative ready!

Name Link
🔨 Latest commit 6b64dc1
🔍 Latest deploy log https://app.netlify.com/projects/knative/deploys/696920fba8f76ac900ee1090
😎 Deploy Preview https://deploy-preview-6556--knative.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@knative-prow
Copy link

knative-prow bot commented Jan 14, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jijo-OO7
Once this PR has been reviewed and has the lgtm label, please assign dprotaso for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow knative-prow bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 14, 2026
@codeEvolveZenith345
Copy link
Contributor

Hi @jijo-OO7 I don't think you have actually followed the issue discussion, read it then open the PR.

@codeEvolveZenith345
Copy link
Contributor

This is totally mis-aligned from what @dprotaso described there.

@dprotaso
Copy link
Member

Yeah I don't see any difference on Chrome Desktop.

@dprotaso
Copy link
Member

Note the idea is for the home, docs etc in the top row to not move when the search bar has focus

@jijo-OO7
Copy link
Contributor Author

Thanks for the pointer. I’ve re-read the original issue discussion and I see where the intent differs from the current implementation.
Based on @dprotaso’s clarification, I’ll update the PR so the top navigation remains static when the search bar gains focus

@knative-prow knative-prow bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 15, 2026
@jijo-OO7
Copy link
Contributor Author

Hi @dprotaso Updated the implementation based on your feedback. The top navigation now remains static when search gains focus by moving to a separate sticky container, following the Material for MkDocs pattern.

Would appreciate your review to ensure this matches the intended behavior. Let me know if any further adjustments are needed.

@codeEvolveZenith345
Copy link
Contributor

@jijo-OO7 I am not sure if this is what was described? the before version is not complete, initially there were elements in nav beside the search bar, which have been removed in after version,
You explicitly hid that part in PR description? why?

@codeEvolveZenith345
Copy link
Contributor

Final review to @dprotaso . Does not seem optimal to me.

@codeEvolveZenith345
Copy link
Contributor

Thanks for the pointer. I’ve re-read the original issue discussion and I see where the intent differs from the current implementation. Based on @dprotaso’s clarification, I’ll update the PR so the top navigation remains static when the search bar gains focus

@jijo-OO7 Again, you did not follow the issue discussion, as Far as I know the nav elements were described to be overlayed by search bar expansion and not removed.

@dprotaso
Copy link
Member

Yeah this doesn't look correct

Screenshot 2026-01-15 at 12 33 10 PM

@jijo-OO7
Copy link
Contributor Author

@dprotaso Could you try reloading the preview? The navigation elements remain visible on my local build and the search expands as expected per the Material for MkDocs pattern. A cache refresh might resolve any discrepancies.
Screenshot (19)

@dprotaso
Copy link
Member

I would expect Home etc. to be at the same horizontal line as the search bar

@jijo-OO7
Copy link
Contributor Author

Got it, thank you for clarifying. I understand now, the navigation items should remain aligned on the same horizontal line as the search bar. I'll revisit the implementation to ensure proper alignment and will push an updated commit shortly with Home, Docs, About, Blog, and Community properly aligned alongside the search bar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When search bar is expanded it causes top section titles to be scrunched

3 participants