-
Notifications
You must be signed in to change notification settings - Fork 8
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
Put 'Eject' into context menu for disks/mountpoints #41
Comments
What shall happen if a combination of files, folders and mountpoints is selected? (What happens on a Mac in this case)? |
Ejecting
|
This feature has been imlpemented upstream now: @moochris would you be interested in having a go at this? I think this is especially useful in conjunction with Spatial mode, because there you have no other way of ejecting/unmounting |
Sure, I'll look at this next when I get some time. If we had devices on the desktop then drag to trash/eject would be nicer 🙂 |
This just takes changing a few lines (not much effort) as explained in #9: But what we need is to have both devices and files/folders... |
Turns out we can solve this by a simple text file: helloSystem/ISO@8f4b2f0#diff-febf34f96aa46897233532c13ceeaef78597b401b13affefd08399013e8934d0 |
Especially in Spatial mode there is currently no way to eject/unmount disks/loop-mounted partitions.
Whenever the user right-clicks on a directory, we should check whether something is mounted at that directory (a mountpoint) and if yes, offer "Eject" rather than "Move to Trash" in the context menu.
The text was updated successfully, but these errors were encountered: