Skip to content

[FEAT] A collapse button in the sidebar which when clicked closes all the open folders#2314

Merged
abose merged 8 commits intomainfrom
pluto/collapse
Jul 5, 2025
Merged

[FEAT] A collapse button in the sidebar which when clicked closes all the open folders#2314
abose merged 8 commits intomainfrom
pluto/collapse

Conversation

@devvaannsh
Copy link
Member

This PR adds a 'Collapse' button in the sidebar area, specifically 'project-files-header' section.
this button when clicked closes all the opened folders recursively. it is really useful to keep things organized.
Now, to keep the UI simpler, we hide the button by default and only show it when the sidebar area is hovered.

Issue reference: #2126

Visual Reference
https://github.com/user-attachments/assets/968a871d-9a17-4d85-838f-81b850b775b4

Copy link
Member

@abose abose left a comment

Choose a reason for hiding this comment

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

Cool Feature!

Just some notes:

  1. Align the icon position relative to other icons in top bar too. Check the button width too on hover to match buttons above, or if both lines are different scales, we can have different sizes too. Don't know if it is fully inline with top icons. but horizontal alignment looks ok.
  2. Also check what happens when file names are really long and git branch name is really long by editing the DOM in dev tools if the button will still be rendered properly. Reduce width of sidebar and see how it behaves on smaller width too.
  3. Integration tests

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 4, 2025

Copy link
Member

@abose abose left a comment

Choose a reason for hiding this comment

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

LGTM.
Merging.

Thanks for the changes @devvaannsh

@abose abose merged commit 21ecffe into main Jul 5, 2025
16 of 18 checks passed
@abose abose deleted the pluto/collapse branch July 5, 2025 05: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

Comments