Document that "Securely clean available disk space" does not delete hidden files
Users cleaning available disk space* should be warn that sfill does not take hidden files in account, i.e previously delete files can still appear in trash folders.
This could be .trash folders from *nix, but also .DS_Store, ~/.Trash, /.Spotlight-V100 and others
- see the documentation:
#6 Updated by sajolida about 5 years ago
- File wipe_available_diskspace.png View added
- Assignee changed from sajolida to BitingBird
- QA Check changed from Ready for QA to Dev Needed
So I created a branch on our main Git base directly on master (as of today) and added your relevant commit on top of it.
When working on this again, if you didn't do more work on it, please try to do reset your local branch to this one:
- git fetch (to fetch the latest changes to our main repo)
- git checkout doc/7118-erase_available_disk_warning (to put yourself on your branch)
- git reset --hard origin/doc/7118-erase_available_disk_warning (to reset your branch to the state of the one in our main repo).
In the future, please make sure that the difference between your branch and master only matches the ticket that you are trying to address. To do so you can do:
- git diff origin/master.. (to see the raw difference between origin/master and your branch)
- git log -p origin/master.. (to see the Git log between origin/master and your branch)
Then regarding your work, I would explain people how to see hidden files. because the technical Unix explanation that "files and folders whose name
+start with . are hidden" is not enough if you only work through GUI. In Nautilus you can do View → Show hidden files.
Then, those hidden files can also be present in any parent or sub folder on that disk. So what about explaining this as well?
Regarding the hidden trash folder, I would give an example. The currrent document could actually be improved maybe. Instead of saying "2. Navigate to a folder on the disk that you want to clean." we could instead explain how to do directly to the root of that disk (clicking on the corresponding disk in the left pane). Then, if would easier to explain the hidden Trash thing as it appears on that same root.
Then I would replace the second screenshot with the one I attach to this ticket and update all the rest accordingly.
What do you think?
#10 Updated by BitingBird about 5 years ago
- Assignee changed from BitingBird to sajolida
- QA Check changed from Dev Needed to Ready for QA
- Feature Branch changed from doc/7118-erase_available_disk_warning to bitingbird:doc/7118-erase_available_disk_warning
Implemented your suggestions, and the new image seems good but I don't know how to include it, I let you do that :)
#14 Updated by sajolida about 5 years ago
- Assignee deleted (
- QA Check deleted (
Ready for QA)
- Feature Branch changed from bitingbird:doc/7118-erase_available_disk_warning to doc/7118-erase_available_disk_warning
Added a bunch of improvements on top of that and merged. Frontdesk is going to be happy. Thanks!