dive merge

Lubomir I. Ivanov neolit123 at gmail.com
Tue Sep 24 03:09:30 UTC 2013


On 24 September 2013 13:05, Dirk Hohndel <dirk at hohndel.org> wrote:
> Hi Lubomir,
>
> How about this one?
>
> diff --git a/qt-ui/divelistview.cpp b/qt-ui/divelistview.cpp
> index a1436bf..ddd10e2 100644
> --- a/qt-ui/divelistview.cpp
> +++ b/qt-ui/divelistview.cpp
> @@ -400,7 +400,7 @@ void DiveListView::contextMenuEvent(QContextMenuEvent *event)
>         }
>         if (d)
>                 popup.addAction(tr("delete dive"), this, SLOT(deleteDive()));
> -       if (selectionModel()->selection().indexes().count() > 14)
> +       if (amount_selected > 1)
>                 popup.addAction(tr("merge selected dives"), this, SLOT(mergeDives()));
>         // "collapse all" really closes all trips,
>         // "collapse" keeps the trip with the selected dive open
>
>
> That seems to work reliably for me and feels much more robust and
> straight forward ;-)
>

yes, this is much better.

lubomir
--


More information about the subsurface mailing list