Bug 175769 - [patch] Add "delete transfer" confirmation
Summary: [patch] Add "delete transfer" confirmation
Status: RESOLVED FIXED
Alias: None
Product: kget
Classification: Applications
Component: UI (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: KGet authors
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-21 15:52 UTC by FiNeX
Modified: 2009-08-11 12:28 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
patch (883 bytes, patch)
2008-11-21 15:52 UTC, FiNeX
Details

Note You need to log in before you can comment on or make changes to this bug.
Description FiNeX 2008-11-21 15:52:58 UTC
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.
Comment 1 Lukas Appelhans 2008-11-21 18:55:51 UTC
Please ask the l10n team for an exception... for me the patch seems to be ok...

Lukas
Comment 2 Lukas Appelhans 2008-12-16 20:07:51 UTC
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
Comment 3 Dario Massarin 2009-08-11 12:28:27 UTC
AFAIK this has already been fixed. Correct me if I'm wrong