Skip to content

fix: add backdrop blur to sticky header for better readability#673

Open
VedantMadane wants to merge 1 commit intokubernetes:masterfrom
VedantMadane:fix/sticky-header-background
Open

fix: add backdrop blur to sticky header for better readability#673
VedantMadane wants to merge 1 commit intokubernetes:masterfrom
VedantMadane:fix/sticky-header-background

Conversation

@VedantMadane
Copy link

@VedantMadane VedantMadane commented Mar 2, 2026

This PR adds backdrop-filter: blur(10px) and adjusts the background opacity of the sticky navbar when scrolled to ensure header text remains readable against underlying content.

This PR adds backdrop-filter: blur(10px) to the sticky navbar when scrolled. This ensures that the header text remains readable when overlapping with page content.
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 2, 2026

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: VedantMadane / name: Vedant Madane (f0a5a5b)

@k8s-ci-robot k8s-ci-robot requested a review from natalisucks March 2, 2026 07:47
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: VedantMadane
Once this PR has been reviewed and has the lgtm label, please assign natalisucks 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

@k8s-ci-robot k8s-ci-robot requested a review from reylejano March 2, 2026 07:47
@k8s-ci-robot
Copy link
Contributor

Welcome @VedantMadane!

It looks like this is your first PR to kubernetes/contributor-site 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval.

You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation.

You can also check if kubernetes/contributor-site has its own contribution guidelines.

You may want to refer to our testing guide if you run into trouble with your tests not passing.

If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs!

Thank you, and welcome to Kubernetes. 😃

@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Mar 2, 2026
@TineoC
Copy link
Contributor

TineoC commented Mar 5, 2026

I can't see the difference between the new vs actual version.

Could you send some screenshots?

@natalisucks
Copy link
Contributor

@VedantMadane Hi there! You will need to sign the CLA before any meaningful review can take place on your pull request. Please follow the directions in the following message to do so: #673 (comment)

@VedantMadane
Copy link
Author

VedantMadane commented Mar 5, 2026

@TineoC The change is subtle but noticeable when scrolling over content-heavy pages:

Before: The sticky navbar uses background-color: rgba(, 0.95) which is a nearly opaque solid color that can still show slight content bleeding through on high-contrast pages.

After: The navbar uses background-color: rgba(, 0.9) with backdrop-filter: blur(10px.) This adds a frosted glass effect that makes the background content behind the header soft and blurred, improving text readability while maintaining a modern aesthetic. The slight transparency reduction (0.95 to 0.9) compensates by letting the blur effect be visible.

The visual difference is most apparent when scrolling a page with varied content colors behind the header. The blur prevents distracting text/images from showing through.

I'll work on getting the CLA signed and can provide screenshots once that's resolved.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Mar 6, 2026
@VedantMadane
Copy link
Author

Friendly ping — CLA is signed and the change is ready for review. Would appreciate an /lgtm when you get a chance. Thanks!

@TineoC
Copy link
Contributor

TineoC commented Mar 10, 2026

Please share the screenshot to see the difference of both versions

@TineoC
Copy link
Contributor

TineoC commented Mar 10, 2026

And this pr doesn't fixes #614 could you please remove that part from your description?

@TineoC
Copy link
Contributor

TineoC commented Mar 17, 2026

/hold

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 17, 2026
@jberkus
Copy link
Contributor

jberkus commented Mar 17, 2026

So, comparing these two I don't understand why the blur is supposed to be better?

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

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants