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

Partition select #15

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

mseabold
Copy link

@mseabold mseabold commented Oct 8, 2014

Added the ability to change the partition used for Disk Usage by clicking on the tile

I'm definitely open to any suggestions or corrections.

Addresses #13

Clicking on the Disk Usage tile will show a dialog to select the
partition. Still some kinks though.
Separate the runnable to load the usage into a standalone class. Use this to force a usage update immediately after selecting a partition. Also, don't stop the running task.
Maintain the selected partition in a SharedPreference in order to keep it persistently.
Move the creation of the dialog and managing of preference into MainActivity as it make more sense within the Android design paradigm. Needed to add a callback for when the available partitions are loaded.
@PaulMaddox
Copy link
Member

Sorry @mseabold - I did not see this PR come in.
Will load it up this afternoon and have a play.

@mseabold
Copy link
Author

I did realize that when I save the selected partition to the preferences, I don't key it off of which session selected the partition. So when you select a partition, it persists that selection across all juice connections. I'll probably look into correcting that.

@jimmydjwu83
Copy link

#13

@AltyFox
Copy link

AltyFox commented May 28, 2016

This is a Longshot but is there any hope for this

@tbnorris
Copy link

I would also love this, but guessing it was not useful to the maintainer. This is a huge feature add-on I would love to have in a very awesome plugin. My root partion is pretty static, my /home is always more important to me

@Slamology2
Copy link

Continue

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.

6 participants