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

Do not delete user files! #912

Open
yakushabb opened this issue Apr 17, 2024 · 9 comments · Fixed by #914
Open

Do not delete user files! #912

yakushabb opened this issue Apr 17, 2024 · 9 comments · Fixed by #914

Comments

@yakushabb
Copy link
Contributor

yakushabb commented Apr 17, 2024

Bug/Feature description

When I removed a library from Cozy, I didn't expect it to delete my files!

Why does Cozy need to delete user files? It should simply remove them from the Cozy library!

image

Even though the command is "Remove from library", the second question is:

msgid "Delete Audiobook?"
msgid "The audiobook will be removed from your disk and from Cozy's library."

image

I don't think users want their precious files to be deleted.

You can either create a command that says "Delete from disk" or add a checkbox like qBittorrent has.

also delete"

Steps to reproduce

Try to remove a library! Please do not delete!

System Information

  • Operating System: Fedora 40
  • Installation source: Flatpak build from source code
  • Version of cozy: 1.3.0
@rdbende
Copy link
Collaborator

rdbende commented Apr 18, 2024

I've brought this up some time ago, and @geigi said, that this behavior is intentional, though to me it seems like a workaround, not a feature. I'll link his reply if I can find it.

@yakushabb
Copy link
Contributor Author

yakushabb commented Apr 18, 2024

In that case, we need to make a few tweaks to the wording.

In GNOME, "remove" is typically used for a soft action that does not affect the actual files, while "delete" is used for a hard action that cannot be undone.

In Cozy, both terms are used interchangeably, and even I, as the translator of the project, cannot distinguish whether my files are going to be permanently deleted and lost a few files while testing the translation.

@yakushabb
Copy link
Contributor Author

yakushabb commented Apr 18, 2024

Also, it would be better to gather feedback from users. Alternatively, you can ask to GNOME Developers Whiteboards to get their point of view on which approach would be better for users.

@rdbende
Copy link
Collaborator

rdbende commented Apr 19, 2024

Also, it would be better to gather feedback from users. Alternatively, you can ask to GNOME Developers Whiteboards to get their point of view on which approach would be better for users.

I think there's no need for discussion in this topic, and I'm pretty sure that most people don't want this behavior. As I've said, the current solution is more of a workaround for a technical difficulty, rather than an actual feature, and it should be solved instead of discussing the design. I'll look into it this afternoon.

@geigi
Copy link
Owner

geigi commented Apr 26, 2024

Sorry that some files where deleted which you didn't want to delete!
The current solution is indeed a workaround because of technical (and UX) reasons:

In the current state removing a file from Cozy would only work temporarily because during the next scan Cozy will have a look at all files in the configured directories and add every file which is not in the database yet.

A few years ago there was a blocklist implemented. If a file was removed from Cozy it was added to this blocklist. During import all files on this list where ignored. The user could also remove files again from this blocklist in the settings. This solution did cause it's own range of problems and misunderstandings which is why I removed it after some time. I think it was a UX pattern that was a bit unusual and complicated.

Using this mechanism as a basis I never had a good idea to solve this problem nicely. I think the current solution is how iTunes handled it back in the day. Maybe we could have a look at some other apps that also observe a directory and import everything in it and how they solve the problem, e.g. Jellyfin.

Another solution is, although it would change the UX quite a bit, to drop the observing of directories. The user would be able to import files or directories normally, we could even detect changes on imported files, but if new files get added to the directories we won't import them. That way the user can simply remove files from Cozy and keep them in the file system. But this solution also brings it's own challanges: what if the user renames a directory and issues a scan? The files won't be found anymore and the "new" ones won't be imported...

I'm not sure which solution we should go for.

@yakushabb
Copy link
Contributor Author

yakushabb commented Apr 26, 2024

IMHO, changing the wording of related strings might be the easiest way to handle this issue.

We can change the strings to something like the followings:

Suggestion 1

1
2

  • Missing ellipses.
  • Incompatible with GNOME HIG.
  • The text body is too long.

Suggestion 2

3
4

  • The title is obvious, the button is smilar with Nautilus

Suggestion 3

5
6

  • The title is obvious, the button is too long.

Nautilus

perm

yakushabb added a commit to yakushabb/cozy that referenced this issue Apr 26, 2024
Change the 'Remove from library' strings to 'Permanently delete'
to make the action clearer for the user, informing them that the
action will delete user files and cannot be undone.

Fixes geigi#912
@geigi
Copy link
Owner

geigi commented Apr 27, 2024

I think so too that this is the easiest way currently. Thanks for your mockups! I quite like suggestion 2.

yakushabb added a commit to yakushabb/cozy that referenced this issue Apr 27, 2024
Change the 'Remove from library' strings to 'Permanently delete'
to make the action clearer for the user, informing them that the
action will delete user files and cannot be undone.

Fixes geigi#912
yakushabb added a commit to yakushabb/cozy that referenced this issue Apr 27, 2024
Change the 'Remove from library' strings to 'Permanently delete'
to make the action clearer for the user, informing them that the
action will delete user files and cannot be recovered.

Fixes geigi#912
yakushabb added a commit to yakushabb/cozy that referenced this issue Apr 27, 2024
Change the 'Remove from library' strings to 'Permanently delete'
to make the action clearer for the user, informing them that the
action will delete user files and cannot be recovered.

Fixes geigi#912
@rdbende
Copy link
Collaborator

rdbende commented Jun 1, 2024

Reopening, as I think it would be still better to have a blacklist rather than deleting files.

@rdbende rdbende reopened this Jun 1, 2024
@Haaruun-I
Copy link
Contributor

Would it be possible to have the blacklist be a 'hide from library' instead of 'delete'

There could be a page or menu to manage the hidden books, and the 'delete' option could still be seperate

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 a pull request may close this issue.

4 participants