<div>-------- Original message --------</div><div>From: Linus Torvalds 
<[email protected]> </div><div>Date:02/06/2015  08:20  (GMT-08:00) 
</div><div>To: Dirk Hohndel <[email protected]> </div><div>Cc: Subsurface 
Mailing List <[email protected]> </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 <[email protected]> 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
_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to