[PATCH 2/3] Removed the delete option by right clicking on "Dive Info"

Dirk Hohndel dirk at hohndel.org
Wed Oct 24 10:36:36 PDT 2012


"Lubomir I. Ivanov" <neolit123 at gmail.com> writes:

> On 24 October 2012 06:55, Dirk Hohndel <dirk at hohndel.org> wrote:
>> This doesn't apply on my tree. Is that relative to the latest master?
>>
>
> just tried the 3 patches they apply in order on the current master
> (48e757574190)
> please, let me know if you need something re-worked in there.

You are of course right. No idea why this failed for me last
night. I'm still not a huge fan of these confirmation dialogs.

Since these are clean patches that fix a real issue I pushed them on
master, but I really want to hear from more people.

Please try this and delete dives and/or trips from the divelist. I'm
sure there must be a better way to do this, even if we decide to go with
these dreaded confirmation dialogs. Maybe show the dive numbers or dive
dates of the dives that would be deleted? Something a little more
informative? 


/D


More information about the subsurface mailing list