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

No confirmation dialog for moving files to the trash #5974

Closed
tbsbdr opened this issue Nov 3, 2021 · 7 comments · Fixed by #6566 or #9527
Closed

No confirmation dialog for moving files to the trash #5974

tbsbdr opened this issue Nov 3, 2021 · 7 comments · Fixed by #6566 or #9527
Assignees

Comments

@tbsbdr
Copy link
Contributor

tbsbdr commented Nov 3, 2021

Userstory

As a user I want to permanently delete files efficiently without having to worry about irreversible damage.

Acceptance Criteria

  • Move to trash: Don't show a confirmation dialog if a file* gets moved to the trash (reversible action)
    • Show a success notification
  • Empty trash Show a confirmation dialog if a file* gets deleted permanently (eg. when the bin gets emptied) (irreversible action)
    • Show confirmation dialog: "Empty trash" "Are you sure you want to permanently delete your items in the trash? You can’t undo this action. [Cancel] [Okay]"

More background on:

* one or many files or folders

@tbsbdr tbsbdr changed the title [Bin & delte files] Confirmation Dialogs [Trash + delte files] Confirmation Dialogs Nov 3, 2021
@tbsbdr tbsbdr assigned tbsbdr and unassigned tbsbdr Nov 3, 2021
@pascalwengerter
Copy link
Contributor

Not sure about the Show a success message with an undo-option and dismiss-x "XX Item(s) successfully moved to trash. [undo] [X]" part. IIRC we moved to auto-closing notifications due to A11y reasons, and putting an undo option in there may be a screenreader offense since it doesn't get focus

How about relying on a success message (Resource X has been moved to the Trashbin) and the confirmation upon emptying trash/ultimately deleting a single file there? Otherwise I think we need to re-think this ticket again @kulmann ?

@pascalwengerter pascalwengerter changed the title [Trash + delte files] Confirmation Dialogs Confirmation dialogue for "moving to trash" vs "deleting" resources in different views Nov 4, 2021
@tbsbdr
Copy link
Contributor Author

tbsbdr commented Nov 4, 2021

forgot to mention, that the message notification should auto-close. added it above. undo would be nice to have, no must-have.
so this means, that we should have no clickable areas in notifications because then we would need to send the focus there and focus management is expensive, correct?

@AlexAndBear
Copy link
Contributor

Empty trash Show a confirmation dialog if a file* gets deleted permanently (eg. when the bin gets emptied) (irreversible action)
Show confirmation dialog: "Empty trash" "Are you sure you want to permanently delete your items in the trash? You can’t undo this action. [Cancel] [Okay]"

Will be handled in PR #6544

@kulmann
Copy link
Member

kulmann commented Mar 17, 2022

Reopening, because only the confirmation for empty trash was done in the linked PR (as being stated in all the messages... ticket was closed accidentally).

@kulmann kulmann reopened this Mar 17, 2022
@tbsbdr tbsbdr changed the title Confirmation dialogue for "moving to trash" vs "deleting" resources in different views No confirmation dialog for moving files to the trash Apr 4, 2022
@settings settings bot removed the p3-medium label Apr 6, 2022
@AlexAndBear
Copy link
Contributor

@tbsbdr can you please provide me the notifications text for move to trashbin, single and multiple file actions

e.G File x was moved to trashbin, 2 files were moved to trashbin?

@tbsbdr
Copy link
Contributor Author

tbsbdr commented Aug 2, 2023

how about:

  • [filename] was moved to trash bin
  • 2 items were moved to trash bin

@AlexAndBear
Copy link
Contributor

👍

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