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

File manager's upload file overwriting feature is labelled in a confusing way, and never prompts the user #2113

Open
nekohayo opened this issue Apr 1, 2024 · 3 comments

Comments

@nekohayo
Copy link

nekohayo commented Apr 1, 2024

In the File Manager's "Advanced options", there is a setting called "Do not overwrite existing files upon operations", where the default is "No" and "Always ask".

There are a couple of issues with this:

  • It is a double-negation, so it's hard to understand, it's confusing.
    It would better to flip everything around and name this something like "Overwrite files when uploading", and the "No" would become either become "Yes" or "Keep the same name and overwrite", and the previously-named "Yes" choice would become "Name the uploaded file differently (add a number at the end)"… or something like that.

  • The "Always ask" checkbox is unclear, and does not seem to work.
    Presumably it means that if enabled, it would prompt the user to ask them what to do or to ask for confirmation before overwriting... however as you can see in the attached video here, it does not work:

Webmin.file.manager.upload.behavior.inconsistent.no.overwriting.prompts.webm

Tested with Firefox 124 on Webmin 2.105 / Virtualmin 7.10.0 running on Debian 12.

@jcameron
Copy link
Collaborator

jcameron commented Apr 1, 2024

@iliajie I think this is a theme-controlled option?

@nekohayo
Copy link
Author

nekohayo commented Apr 1, 2024

Alternate possibility, which would be a cleaner/simpler UX: don't make this a setting/option, just prompt me to make a choice between "Keep the same names and overwrite files" and "Rename uploaded files to avoid conflicting" everytime I start an upload that has conflicting filenames; I believe this approach would make more sense, because this type of choice is probably highly context-dependent, more than a "global preference" kind of thing…

iliajie added a commit to webmin/authentic-theme that referenced this issue Apr 3, 2024
@iliajie
Copy link
Collaborator

iliajie commented Apr 3, 2024

  • It is a double-negation, so it's hard to understand, it's confusing.

Agreed, I will call it Preserve existing files upon operations instead.

  • name this something like "Overwrite files when uploading"

This feature ins't about uploading files only.

Presumably it means that if enabled, it would prompt the user to ask them what to do or to ask for confirmation before overwriting

Yes, but not for uploads. For other operations, like when extracting a file:

image

Alternate possibility, which would be a cleaner/simpler UX: don't make this a setting/option, just prompt me to make a choice between

Sorry, that wouldn't work for a lot of other folks.

Everytime I start an upload that has conflicting filenames; I believe this approach would make more sense

In particular, in application to uploads, it isn't easy to implement; so, it's much simpler to ask before the process.

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

No branches or pull requests

3 participants