Skip to content

Conversation

@alexey-milovidov
Copy link
Member

@alexey-milovidov alexey-milovidov commented Jan 24, 2026

Summary

Replace DocSearchButton with a progressive enhancement approach:

This fixes the issue where search was unusable on slow connections until all scripts (~25MB) finished loading.

Checklist

Replace DocSearchButton with a progressive enhancement approach:
- Use native <a href="search"> link that works without JavaScript
- After React hydrates, intercept clicks to open DocSearch modal
- Add CSS to style the link element as a button

This fixes the issue where search was unusable on slow connections
until all scripts (~25MB) finished loading.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@alexey-milovidov alexey-milovidov requested a review from a team as a code owner January 24, 2026 00:51
@vercel
Copy link

vercel bot commented Jan 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
clickhouse-docs Ready Ready Preview, Comment Jan 24, 2026 1:02am
3 Skipped Deployments
Project Deployment Review Updated (UTC)
clickhouse-docs-jp Ignored Ignored Jan 24, 2026 1:02am
clickhouse-docs-ru Ignored Ignored Preview Jan 24, 2026 1:02am
clickhouse-docs-zh Ignored Ignored Preview Jan 24, 2026 1:02am

Request Review

@alexey-milovidov
Copy link
Member Author

I tried to test it, but didn't end up validating. Please check if it's alright.

@alexey-milovidov
Copy link
Member Author

@Blargian, I need help - for some reason it goes to
https://clickhouse-docs-git-progressive-search-button-clickhouse.vercel.app/search
instead of
https://clickhouse-docs-git-progressive-search-button-clickhouse.vercel.app/docs/search

@Blargian
Copy link
Member

@Blargian, I need help - for some reason it goes to https://clickhouse-docs-git-progressive-search-button-clickhouse.vercel.app/search instead of https://clickhouse-docs-git-progressive-search-button-clickhouse.vercel.app/docs/search

Thanks @alexey-milovidov, sorry for the delay. On my to-do list for first thing tomorrow.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants