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

Linus Torvalds torvalds at linux-foundation.org
Fri Feb 6 08:20:04 PST 2015


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.

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

                          Linus


More information about the subsurface mailing list