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

Collapse all directories recursively #976

Merged
merged 4 commits into from Oct 16, 2017

Conversation

Projects
None yet
5 participants
@darkowlzz
Contributor

darkowlzz commented Nov 5, 2016

Implementation of shortcut to collapse all directories recursively #960 .

ctrl-r shortcut for recursively collaping all the directories.

tree-view-collapse-all

Show outdated Hide outdated keymaps/tree-view.cson
Show outdated Hide outdated lib/tree-view.coffee
Show outdated Hide outdated lib/tree-view.coffee
Show outdated Hide outdated spec/tree-view-spec.coffee

@50Wliu 50Wliu added the needs-review label Nov 6, 2016

@darkowlzz

This comment has been minimized.

Show comment
Hide comment
@darkowlzz

darkowlzz Nov 6, 2016

Contributor

@50Wliu thanks for the feedback. Is this what you meant by placing collapseAll under collapseDirectory?

Contributor

darkowlzz commented Nov 6, 2016

@50Wliu thanks for the feedback. Is this what you meant by placing collapseAll under collapseDirectory?

@50Wliu

This comment has been minimized.

Show comment
Hide comment
@50Wliu

50Wliu Nov 6, 2016

Member

What you did looks fine to me.

Member

50Wliu commented Nov 6, 2016

What you did looks fine to me.

@50Wliu

50Wliu approved these changes Nov 6, 2016

@50Wliu

This comment has been minimized.

Show comment
Hide comment
@50Wliu

50Wliu Nov 30, 2016

Member

Can you also add the collapse all option to the context menus?

Member

50Wliu commented Nov 30, 2016

Can you also add the collapse all option to the context menus?

@darkowlzz

This comment has been minimized.

Show comment
Hide comment
@darkowlzz

darkowlzz Dec 9, 2016

Contributor

@50Wliu collapse all option in project-root context menu only, looks good?

screen shot 2016-12-10 at 1 45 01 am

Contributor

darkowlzz commented Dec 9, 2016

@50Wliu collapse all option in project-root context menu only, looks good?

screen shot 2016-12-10 at 1 45 01 am

@sfeldkamp

This comment has been minimized.

Show comment
Hide comment
@sfeldkamp

sfeldkamp May 7, 2017

This would be nice to have. Is it able to be merged into master for a release? If not what needs to happen?

sfeldkamp commented May 7, 2017

This would be nice to have. Is it able to be merged into master for a release? If not what needs to happen?

@eugenet8k

This comment has been minimized.

Show comment
Hide comment
@eugenet8k

eugenet8k Aug 30, 2017

This is a very desirable feature on multiple projects Tree View setups. Please merge.

eugenet8k commented Aug 30, 2017

This is a very desirable feature on multiple projects Tree View setups. Please merge.

@ywysuibian

This comment has been minimized.

Show comment
Hide comment
@ywysuibian

ywysuibian Sep 26, 2017

I am also expecting this feature~

ywysuibian commented Sep 26, 2017

I am also expecting this feature~

@50Wliu

This comment has been minimized.

Show comment
Hide comment
@50Wliu

50Wliu Sep 26, 2017

Member

Thanks @darkowlzz! Sorry that this hasn't been merged yet. Can you resolve the conflicts?

Member

50Wliu commented Sep 26, 2017

Thanks @darkowlzz! Sorry that this hasn't been merged yet. Can you resolve the conflicts?

@darkowlzz

This comment has been minimized.

Show comment
Hide comment
@darkowlzz

darkowlzz Oct 16, 2017

Contributor

@50Wliu ping!

Contributor

darkowlzz commented Oct 16, 2017

@50Wliu ping!

@50Wliu 50Wliu merged commit 56dfd81 into atom:master Oct 16, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment