Skip to content
This repository has been archived by the owner on Aug 25, 2018. It is now read-only.

Sticky projects #121

Merged
merged 1 commit into from
Mar 9, 2018
Merged

Sticky projects #121

merged 1 commit into from
Mar 9, 2018

Conversation

simurai
Copy link
Contributor

@simurai simurai commented Feb 14, 2018

Description of the Change

This makes the project root in the tree-view stick to the top when scrolling.

sticky

Benefits

Easier to orient with multiple project roots and lots of files/folders.

Possible Drawbacks

Scrolling performance suffers, not too bad, but still noticeably slower.

@simurai
Copy link
Contributor Author

simurai commented Feb 14, 2018

Scrolling performance suffers

Ok, after some more testing, I think performance is ok (60fps) when nothing interferes. The slowdown I noticed seems to be caused whenever the background tips update. But that also happens without sticky projects.

Maybe the background tips should be static?

@simurai
Copy link
Contributor Author

simurai commented Mar 9, 2018

Ok, let's merge and publish a new release. There should still be some time to test on the dev channel before the next Atom release.

@simurai simurai merged commit b3cb989 into master Mar 9, 2018
@simurai simurai deleted the sm-sticky-projects branch March 9, 2018 10:45
@Ben3eeE
Copy link

Ben3eeE commented Mar 9, 2018

🎉

@simurai simurai added this to the 1.26 milestone Mar 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants