Skip to content

Update navbar CTA and fix Inkeep dark overlay#141

Open
danielclickh wants to merge 1 commit into
ClickHouse:mainfrom
danielclickh:feat/navbar-cta-inkeep
Open

Update navbar CTA and fix Inkeep dark overlay#141
danielclickh wants to merge 1 commit into
ClickHouse:mainfrom
danielclickh:feat/navbar-cta-inkeep

Conversation

@danielclickh

@danielclickh danielclickh commented Jun 8, 2026

Copy link
Copy Markdown

Summary

  • Rename navbar CTA button from "Cloud console" to "Get Started"
  • Reduce CTA button border radius from 8px to 4px
  • Fix Inkeep search dark-mode overlay: the widget's .dark:bg-overlay-dark class was rendering a light grey semi-transparent background; override it to use a proper dark overlay (rgba(0,0,0,0.75))

Test plan

  • Navbar shows "Get Started" button with 4px border radius (light and dark mode)
  • Opening Inkeep search in dark mode shows a dark overlay behind the dialog, not a grey one

🤖 Generated with Claude Code


Note

Low Risk
Cosmetic navbar and Inkeep theme CSS only; no auth, data, or API behavior changes.

Overview
Updates docs site chrome: the custom navbar CTA is relabeled Get Started (still links to clickhouse.cloud/signUp) and its border-radius is tightened from 8px to 4px.

For Inkeep search, a new shadow-DOM theme.styles entry overrides .dark:bg-overlay-dark so the modal backdrop uses rgba(0,0,0,0.75) in dark mode instead of a light grey overlay.

Reviewed by Cursor Bugbot for commit dca2a16. Bugbot is set up for automated code reviews on this repo. Configure here.

- Rename CTA button from "Cloud console" to "Get Started"
- Reduce CTA button border radius from 8px to 4px
- Add dark-mode overlay fix to Inkeep search: the widget's
  `.dark:bg-overlay-dark` class was rendering a light grey overlay;
  override it with a proper dark semi-transparent background

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit dca2a16. Configure here.

key: 'dark-search-overlay',
type: 'style',
value: '.dark\\:bg-overlay-dark { background-color: rgba(0, 0, 0, 0.75) !important; }',
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overlay override ignores dark scope

Medium Severity

The new Inkeep theme style targets .dark\:bg-overlay-dark without a .dark ancestor. Tailwind dark utilities usually stay on the element in light mode while only taking effect under .dark, so this unscoped !important rule can force a 75% black backdrop when opening search in light mode.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit dca2a16. Configure here.

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.

2 participants