Skip to content

fix(web): match settings search shortcut styling to command palette's - #5841

Merged
maria-rcks merged 2 commits into
pingdotgg:mainfrom
UtkarshUsername:fix/settings-search-shortcut-style
Aug 9, 2026
Merged

fix(web): match settings search shortcut styling to command palette's#5841
maria-rcks merged 2 commits into
pingdotgg:mainfrom
UtkarshUsername:fix/settings-search-shortcut-style

Conversation

@UtkarshUsername

@UtkarshUsername UtkarshUsername commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

What Changed

The "/" shortcut badge in the settings sidebar search box now uses the same subtle keycap style as the command palette shortcut badge in the legacy sidebar, so the two look consistent.

Why

The "/" settings badge previously looked weird and was not consistent with the existing style. It rendered as a bordered, outlined pill, which stood out against the plain keycap used for the command palette hint. This change makes the settings search shortcut match the existing style.

Existing Style (command palette shortcut badge in the legacy sidebar):
image
The "/" shortcut didn't match this, and looked weird.

UI Changes

Before:
image

After:
image

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes

Note

Low Risk
Visual-only className change on a keyboard hint with no behavior or data impact.

Overview
The / keyboard hint in the settings sidebar search field no longer uses sidebar-specific pill styling (extra background, border ring, muted text color, and right margin). It now relies on the shared Kbd defaults with the same compact sizing classes as the legacy sidebar command palette shortcut badge.

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

Note

Match settings search Kbd shortcut styling to command palette

Removes sidebar-specific styles (background color, ring/border, muted-foreground text color, right margin) from the / keyboard hint in SettingsSidebarNav.tsx, aligning it visually with the command palette shortcut badge.

Macroscope summarized 124ffcf.

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e08569c7-6450-46dd-ab3d-1475c2d3bb37

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 9, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 36ccab4

This is a purely cosmetic change removing some Tailwind CSS classes from a keyboard shortcut indicator to match styling with the command palette. No behavioral changes.

You can customize Macroscope's approvability policy. Learn more.

@maria-rcks
maria-rcks merged commit 83d769f into pingdotgg:main Aug 9, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants