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.

The context menu requires a lot more actual effort to get to the dive delete..

                          Linus
_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to