Skip to content

Fix search icon not showing in navbar#6393

Merged
Alek99 merged 1 commit into
mainfrom
carlos/fix-search-icon-not-showing
Apr 27, 2026
Merged

Fix search icon not showing in navbar#6393
Alek99 merged 1 commit into
mainfrom
carlos/fix-search-icon-not-showing

Conversation

@carlosabadia
Copy link
Copy Markdown
Contributor

No description provided.

@carlosabadia carlosabadia requested review from a team and Alek99 as code owners April 27, 2026 09:33
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

This PR fixes the search icon not displaying in the navbar by correcting the asset path for the custom search icon used in the Inkeep search bar configuration. The path was updated from /icons/search.svg to /docs/icons/search.svg to match the actual location of the icon on the docs site.

Confidence Score: 5/5

Safe to merge β€” single-character path fix with no logic changes or side effects.

The change is a minimal, targeted fix correcting an asset URL path. No logic, state, or API behavior is altered.

No files require special attention.

Important Files Changed

Filename Overview
docs/app/reflex_docs/views/inkeep.py One-line fix: corrects the custom search icon asset path from /icons/search.svg to /docs/icons/search.svg in the Inkeep search bar props.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Inkeep SearchBar renders] --> B[Loads customIcons config]
    B --> C{Icon path}
    C -- Before fix --> D["/icons/search.svg ❌ 404"]
    C -- After fix --> E["/docs/icons/search.svg βœ… Found"]
    D --> F[Search icon missing in navbar]
    E --> G[Search icon displays correctly]
Loading

Reviews (1): Last reviewed commit: "Fix search icon not showing in navbar" | Re-trigger Greptile

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 27, 2026

Merging this PR will not alter performance

βœ… 9 untouched benchmarks


Comparing carlos/fix-search-icon-not-showing (675992d) with main (44271c5)

Open in CodSpeed

@Alek99 Alek99 merged commit 6d6ff1f into main Apr 27, 2026
92 of 93 checks passed
@Alek99 Alek99 deleted the carlos/fix-search-icon-not-showing branch April 27, 2026 16:46
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