Skip to content

fix(sidebar): stop propagation on sidebar container to prevent collap… - #2467

Open
NiranjanRSoorej06 wants to merge 3 commits into
utksh1:mainfrom
NiranjanRSoorej06:main
Open

fix(sidebar): stop propagation on sidebar container to prevent collap…#2467
NiranjanRSoorej06 wants to merge 3 commits into
utksh1:mainfrom
NiranjanRSoorej06:main

Conversation

@NiranjanRSoorej06

Copy link
Copy Markdown

Description

This PR fixes the sidebar collapse behavior by ensuring that the sidebar only collapses when the dedicated collapse (<<) button is clicked.

Changes Made

  • Removed the unintended collapse behavior triggered by clicks on empty/non-interactive areas of the sidebar.
  • Restricted the collapse action to the dedicated collapse button only.
  • Preserved all existing sidebar navigation and interaction behavior.
  • Improved the overall user experience by preventing accidental sidebar collapses.

Related Issues

Closes #2364

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

  • Launched the application with the updated changes.
  • Verified that clicking the dedicated collapse (<<) button collapses the sidebar.
  • Verified that clicking on empty/non-interactive areas inside the sidebar no longer collapses it.
  • Confirmed that sidebar navigation and other interactive elements continue to function as expected.
  • Tested multiple expand/collapse cycles to ensure consistent behavior.

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas (where applicable).
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

Copilot AI lite review requested due to automatic review settings August 5, 2026 14:51

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@NiranjanRSoorej06

Copy link
Copy Markdown
Author

Hi @utksh1

This PR fixes the unintended sidebar collapse behavior by ensuring that the sidebar only collapses when the dedicated collapse button is clicked, preventing accidental collapses caused by clicks on empty areas.

If everything looks good, could you please review and merge this PR? Also, kindly add the following labels:

type:bug
quality:clean
level:intermediate
gssoc:assigned

Thank you!

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All checks passed. Sidebar propagation fix looks good.

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.

[BUG] : Sidebar Collapses When Clicking Empty Areas Instead of Only the Collapse Button

3 participants