Ask confirmation before removing accounts
ClosedPublic

Authored by elvisangelaccio on Feb 22 2017, 6:01 PM.

Details

Summary

Removing accounts cannot be (easily) undone, so ask for confirmation
before doing it.

Test Plan

Messagebox pops up when clicking the delete button. Account is removed only if
Yes is pressed.

Diff Detail

Repository
R155 KAccounts Integration
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mck182 accepted this revision.Feb 23 2017, 3:57 AM
mck182 added inline comments.
src/kaccounts.cpp
125

I feel like the "from the list of your accounts" is sort of superfluous; I'd suggest something like "Are you sure that you want to remove the account '%1?' This cannot be undone."

This revision is now accepted and ready to land.Feb 23 2017, 3:57 AM
This revision was automatically updated to reflect the committed changes.