Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add inline toolbar to sidebar #569

Merged
merged 8 commits into from
Nov 25, 2019
Merged

Add inline toolbar to sidebar #569

merged 8 commits into from
Nov 25, 2019

Conversation

cassidyjames
Copy link
Contributor

@cassidyjames cassidyjames commented Nov 2, 2018

Adds an inline toolbar to the sidebar with an open folder button.

Fixes #554
Fixes #656

@jeremypw
Copy link
Collaborator

The simple folder icon does not really indicate its purpose - need to see tooltip. Any reason not to use the usual + and - icons?

@cassidyjames
Copy link
Contributor Author

@jeremypw my reasoning was to use a semantic icon for explicitly opening a folder, instead of a generic "add" icon—especially since the contents of the sidebar are mixed (and not just folders).

I also worry about a "minus" icon being construed as "delete" a folder, instead of just removing it from the project view. I think for removing in that case, a context menu still might be best since the concept of which project folder is focused is also harder to see at a glance, and it might appear to be acting on a selected file instead.

@jeremypw
Copy link
Collaborator

jeremypw commented Jan 16, 2019

I do not think the minus icon is likely to be construed as actual deletion in the context of a sidebar as it does not mean this anywhere else (I think). I agree that special "folder-add" and "folder-remove" icons, with tooltips, would be preferable. The icons should only be sensitive when a folder is selected.

@roymckenzie
Copy link
Contributor

Was a bit curious about this. It seems it duplicates functionality in the HeaderBar? Is this to help de-clutter the HeaderBar?

@cassidyjames
Copy link
Contributor Author

@roymckenzie the idea was to declutter the header bar a bit, to more clearly associate the folder/project actions with the sidebar, and to support further actions like in #568. I'm not sure if it's more discoverable or not, but I do like being able to add more folder management options there quite a bit—and those actions would feel awkward on their own.

@roymckenzie
Copy link
Contributor

roymckenzie commented Mar 12, 2019

@cassidyjames OK cool. That makes sense. I think an "Add File" button might be cool, too. And a "Collapse All Folders" button. My only complaint is that the toolbar looks so floaty and lonely down there with just the one button and so thin. Almost anemic. Maybe with more buttons it will look better?

@cassidyjames
Copy link
Contributor Author

@roymckenzie yeah with just one icon it's kind of silly. But I imagine both an add/remove on the left, then perhaps a gear with a submenu for things like sorting/collapsing folders on the right.

cassidyjames and others added 3 commits October 3, 2019 12:02
…ollapse folders. (#657)

* User can see close focused project folder, alphabetize folders, and collapse folders.

* removed close folder icon/functionality in the inline toolbar

* Update src/Widgets/Pane.vala

Co-Authored-By: roymckenzie <1065321+roymckenzie@users.noreply.github.com>

* Update src/Widgets/Pane.vala

Co-Authored-By: roymckenzie <1065321+roymckenzie@users.noreply.github.com>

* Update src/Widgets/Pane.vala

Co-Authored-By: roymckenzie <1065321+roymckenzie@users.noreply.github.com>

* Update src/Widgets/Pane.vala

Co-Authored-By: roymckenzie <1065321+roymckenzie@users.noreply.github.com>

* Move headerbar open folder button to inline toolbar

* Fix lint errors

* Revert icon change

* Fix lint error (again)
Copy link
Collaborator

@jeremypw jeremypw left a comment

Choose a reason for hiding this comment

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

Does the job and looks good.

@jeremypw jeremypw merged commit 88f39f7 into master Nov 25, 2019
@jeremypw jeremypw deleted the sidebar-inline-toolbar branch November 25, 2019 18:20
@cassidyjames cassidyjames added this to the 3.2.0 milestone Nov 25, 2019
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.

Expand/collapse all projects in sidebar Move "Open folder" action to inline toolbar
3 participants