Skip to content

Conversation

@chandrasekharan-zipstack
Copy link
Contributor

@chandrasekharan-zipstack chandrasekharan-zipstack commented Jan 29, 2026

What

  • Added pointer-events: none to sidebar tooltips via the .sidebar-tooltip CSS class
  • Applied overlayClassName="sidebar-tooltip" to Tooltip components in SideNavBar

Why

  • Sidebar tooltips were blocking mouse cursor events from reaching adjacent menu items when the sidebar was collapsed
  • This prevented users from hovering over or clicking on menu items that appeared behind the tooltip overlay
image

How

  • Added .sidebar-tooltip { pointer-events: none; } to SideNavBar.css
  • Added overlayClassName="sidebar-tooltip" to Tooltip components to apply the class to tooltip overlays
  • Mouse events now pass through the tooltip to underlying menu items

Can this PR break any existing features. If yes, please list possible items. If no, please explain why.

  • No. This only affects the visual behavior of tooltips in the collapsed sidebar state. The CSS change makes tooltips non-interactive, which is the intended behavior. No functional logic is altered.

Notes on Testing

  • Test with collapsed sidebar: hover over menu items to verify tooltips display
  • Verify that cursor can reach adjacent menu items without tooltips blocking interaction
  • Test with expanded sidebar to ensure normal behavior is unchanged

Screenshots

  • Unable to add a good screenshot, but the tooltip disappears while going to the option above it

Checklist

This adds pointer-events: none to sidebar tooltips so mouse events pass
through to underlying menu items when the sidebar is collapsed.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026

Walkthrough

A styling enhancement is applied to tooltips in the side navigation bar by adding a pointer-events: none; CSS property and applying it via a new overlayClassName prop on all Tooltip components. This allows mouse events to pass through tooltips to underlying menu items.

Changes

Cohort / File(s) Summary
Side Navigation Tooltip Styling
frontend/src/components/navigations/side-nav-bar/SideNavBar.css, frontend/src/components/navigations/side-nav-bar/SideNavBar.jsx
Added .sidebar-tooltip CSS class with pointer-events: none; and applied it to all Tooltip components via overlayClassName prop to prevent tooltips from blocking mouse events to underlying elements.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing sidebar tooltips that were blocking adjacent menu items on hover. It accurately reflects the primary objective of the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description covers all required sections with clear explanations of the problem, solution, testing approach, and potential impacts.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

@sonarqubecloud
Copy link

@chandrasekharan-zipstack
Copy link
Contributor Author

Closing this since #1768 will take care of this issue altogether

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