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

[Bug]: issues with rename file with invalid name dialog #6760

Open
4 of 8 tasks
Tracked by #6774
mgallien opened this issue May 16, 2024 · 1 comment
Open
4 of 8 tasks
Tracked by #6774

[Bug]: issues with rename file with invalid name dialog #6760

mgallien opened this issue May 16, 2024 · 1 comment

Comments

@mgallien
Copy link
Collaborator

⚠️ Before submitting, please verify the following: ⚠️

Bug description

The dialog has 2 cancel buttons.
When clicking on OK button to trigger a renaming, the user has no immediate feedback thinking that they did not click until the operation is done.
see
image

Steps to reproduce

  1. Create a group folder with trailing and leading space in the name
  2. Try to sync it via desktop client on windows
  3. This folder cannot be synced
  4. Clicking on the warning opens a dialog to rename (impossible for group folder root folder)
  5. Dialog has issues and rename cannot be done

Expected behavior

If a folder cannot rename, just tell it to user
A dialog should not have 2 cancel buttons
Upon clicking on a button, immediate feedback should be given

Which files are affected by this bug

n/a

Operating system

Windows

Which version of the operating system you are running.

windows 10

Package

Other

Nextcloud Server version

28.0.5

Nextcloud Desktop Client version

master branch

Is this bug present after an update or on a fresh install?

Updated from a minor version (ex. 3.4.2 to 3.4.4)

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

Are you using an external user-backend?

  • Default internal user-backend
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Nextcloud Server logs

No response

Additional info

No response

@mgallien
Copy link
Collaborator Author

issues from master
2 cancel buttons is a regression only on master
@tobiasKaminsky

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant