Skip to content

Fix: Sidebar expander arrow should be cursor pointer - #794

Merged
Nihal4777 merged 1 commit into
CircuitVerse:masterfrom
SouparnaChatterjee:fix/expander-cursor-672
May 31, 2026
Merged

Fix: Sidebar expander arrow should be cursor pointer#794
Nihal4777 merged 1 commit into
CircuitVerse:masterfrom
SouparnaChatterjee:fix/expander-cursor-672

Conversation

@SouparnaChatterjee

@SouparnaChatterjee SouparnaChatterjee commented May 31, 2026

Copy link
Copy Markdown
Contributor

Fixes #672

Changes done:

  • Added cursor: pointer to .nav-list-expander in _sass/custom/custom.scss

Screenshots

Before: Expander arrow shows default arrow cursor
After: Expander arrow shows pointer cursor (hand icon)

Screen.Recording.2026-05-31.171452.mp4

✅️ By submitting this PR, I have verified the following

  • Checked to see if a similar PR has already been opened 🤔️
  • Reviewed the contributing guidelines 🔍️
  • Sample preview link added (add after checks complete)
  • Tried Squashing the commits into one

Summary by CodeRabbit

  • Style
    • Updated cursor styling for the sidebar expander control to indicate interactivity.

@coderabbitai

coderabbitai Bot commented May 31, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

This pull request adds a CSS rule to the custom SCSS stylesheet that sets the cursor to pointer for the sidebar navigation list expander element. The change addresses a bug where the expander control was displaying the default arrow cursor instead of the pointer cursor, improving the visual affordance that the element is clickable. The modification is a single, isolated styling addition with no impact on HTML structure or JavaScript behavior.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and specifically describes the main change: adding cursor pointer styling to the sidebar expander element.
Linked Issues check ✅ Passed The PR directly addresses issue #672 by adding cursor: pointer to .nav-list-expander, meeting the stated objective to change the sidebar expander cursor from arrow to pointer.
Out of Scope Changes check ✅ Passed The PR contains only the necessary CSS change to address issue #672; no unrelated or out-of-scope modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai 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.

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: fd33f867-efe7-4682-b460-4c1660ffb352

📥 Commits

Reviewing files that changed from the base of the PR and between bd99a34 and 50ba6e6.

📒 Files selected for processing (2)
  • _includes/components/nav.html
  • _sass/custom/custom.scss

Comment thread _sass/custom/custom.scss Outdated
@Me-Priyank

Copy link
Copy Markdown
Member

@Nihal4777 LGTM

@SouparnaChatterjee
SouparnaChatterjee force-pushed the fix/expander-cursor-672 branch from 8272ff2 to 3def805 Compare May 31, 2026 15:00
Comment thread _sass/custom/custom.scss Outdated
@SouparnaChatterjee
SouparnaChatterjee force-pushed the fix/expander-cursor-672 branch from 3def805 to 3ca0d4d Compare May 31, 2026 17:55
@Nihal4777
Nihal4777 merged commit 2be27da into CircuitVerse:master May 31, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sidebar items expander should be cursor pointer

3 participants