First patch - Delete dive(s) using the delete key

[Steve]

<div>-------- Original message --------</div><div>From: Linus Torvalds <torvalds at linux-foundation.org> </div><div>Date:02/06/2015  08:20  (GMT-08:00) </div><div>To: Dirk Hohndel <dirk at hohndel.org> </div><div>Cc: Subsurface Mailing List <subsurface at subsurface-divelog.org> </div><div>Subject: Re: First patch - Delete dive(s) using the delete key </div><div>
</div>On Fri, Feb 6, 2015 at 7:20 AM, Dirk Hohndel <dirk at hohndel.org> wrote:
> Hi Grace,
>
> welcome to the team!
>
> On Fri, Feb 06, 2015 at 02:19:04PM +0300, Grace Karanja wrote:
>>
>> This patch allows the user to delete dives using the delete key.
>
> Excellent.

Really? This seems like a recipe for mistakes and easy data loss.

If I read the patch right, there's no confirmation etc, so now just
mistakenly pressing the Del key will delete dives. That sounds like a
fairly easy way for somebody to fatfinger things and lose data.

Do we need an undo stack so the user can undo the last few actions?

--Steve
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150206/9b44450d/attachment-0001.html>