| Summary: | [patch] Add "delete transfer" confirmation | ||
|---|---|---|---|
| Product: | [Applications] kget | Reporter: | FiNeX <finex> |
| Component: | UI | Assignee: | KGet bugs <kget-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nekkar |
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: | patch | ||
Please ask the l10n team for an exception... for me the patch seems to be ok... Lukas So more comments: 1) This patch misses some KLocale-Plural stuff, for example when only one Transfer is selected... and 2) Please commit this to the patches/-Dir... Lukas AFAIK this has already been fixed. Correct me if I'm wrong |
Created attachment 28730 [details] patch Hi, I know that we are in string freeze time, but the current "delete selected" action, used to delete the selected transfers, is very dangerous without a confirmation dialog. I suggest to add a messagebox which ask the confirmation. I suggest to apply this patch, even if we are in string freeze. Bye.