Skip to content

fix(breadcrumb): add cursor pointer to breadcrumb-link#820

Merged
rohilsurana merged 1 commit into
mainfrom
fix/breadcrumb-link-cursor
May 26, 2026
Merged

fix(breadcrumb): add cursor pointer to breadcrumb-link#820
rohilsurana merged 1 commit into
mainfrom
fix/breadcrumb-link-cursor

Conversation

@rohilsurana
Copy link
Copy Markdown
Member

Summary

  • Add cursor: pointer to .breadcrumb-link CSS class
  • The breadcrumb-link-active variant already sets cursor: default, and breadcrumb-dropdown-trigger/breadcrumb-dropdown-item both set cursor: pointer — the base link class was the only interactive breadcrumb element missing it
  • Without this, non-href breadcrumb items (used in SPA callback navigation) show a text cursor instead of pointer

Test plan

  • Render a Breadcrumb.Item with onClick but no href → verify pointer cursor on hover
  • Render a Breadcrumb.Item with current prop → verify default cursor (unchanged)
  • Render a Breadcrumb.Item with disabled prop → verify no pointer events (unchanged)

@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

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

Project Deployment Actions Updated (UTC)
apsara Ready Ready Preview, Comment May 26, 2026 10:30am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 26, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e605074c-7bc9-4f96-9449-25a6c30cc4bb

📥 Commits

Reviewing files that changed from the base of the PR and between 9943287 and 7600382.

📒 Files selected for processing (1)
  • packages/raystack/components/breadcrumb/breadcrumb.module.css

📝 Walkthrough

Walkthrough

This PR makes a single CSS styling update to the breadcrumb component. The .breadcrumb-link class rule now includes cursor: pointer;, which changes the mouse cursor appearance to a pointer when hovering over breadcrumb navigation links. This is a minimal, focused change affecting only one stylesheet file and adding one CSS property.

Suggested reviewers

  • rsbh
  • rohanchkrabrty
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding cursor pointer styling to the breadcrumb-link CSS class.
Description check ✅ Passed The description is directly related to the changeset, providing context about why the change was needed, what other breadcrumb elements already have, and a test plan to verify the fix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


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 and usage tips.

@rohilsurana rohilsurana merged commit c5554fa into main May 26, 2026
5 checks passed
@rohilsurana rohilsurana deleted the fix/breadcrumb-link-cursor branch May 26, 2026 10:32
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