Skip to content

fix(navbar): remove conflicting docsearch deps and CSS overrides#892

Closed
shreya1510ss wants to merge 1 commit into
keploy:mainfrom
shreya1510ss:fix/search-bar-styling
Closed

fix(navbar): remove conflicting docsearch deps and CSS overrides#892
shreya1510ss wants to merge 1 commit into
keploy:mainfrom
shreya1510ss:fix/search-bar-styling

Conversation

@shreya1510ss

Copy link
Copy Markdown
Contributor

What has changed?
Issue: The navbar search box was not styled correctly, unlike the default clean look on stock Docusaurus sites.
image

Cause:
@docsearch/react/@docsearch/core were unnecessarily added directly in package.json, conflicting with the version already bundled by @docusaurus/preset-classic.
Custom !important CSS overrides on .DocSearch-Button in src/css/custom.css were fighting DocSearch's default styling.
Fix: Removed both, restoring the default clean search box look.

Fixes https://github.com/keploy/enterprise/issues/2291

Image after fix
image

Signed-off-by: Shreya Sharma <shreyasharma.1510001@gmail.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR restores Docusaurus’ default DocSearch styling by removing direct DocSearch dependencies and eliminating custom CSS overrides that were fighting the upstream styles.

Changes:

  • Removed @docsearch/core and @docsearch/react from package.json to avoid version conflicts with the DocSearch version pulled in via Docusaurus.
  • Updated yarn.lock accordingly (DocSearch remains as a transitive dependency via @docusaurus/theme-search-algolia).
  • Removed !important Tailwind @apply overrides targeting .DocSearch-Button from src/css/custom.css.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Drops direct DocSearch deps so Docusaurus controls the DocSearch version.
yarn.lock Reflects the dependency graph cleanup after removing direct DocSearch deps.
src/css/custom.css Removes conflicting .DocSearch-Button overrides to let default DocSearch styling apply.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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