mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Correctly handle empty dive list after delete

Browse source 
Things need to be cleared out and disabled. But fixing this bug leaves
another issue - while the newly selected dive after a delete acts as if it
was selected, it doesn't show up as selected in the dive list.

Reported-by: Henrik Brautaset Aronsen <subsurface@henrik.synth.no>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2013-09-22 22:24:28 -07:00
parent 6ce5704435
commit dba16202c4
3 changed files with 28 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

15
qt-ui/divelistview.cpp
View file

@ -7,6 +7,7 @@
#include "divelistview.h"
#include "models.h"
#include "modeldelegates.h"
#include "mainwindow.h"
#include "../display.h"
#include <QApplication>
#include <QHeaderView>
@ -72,7 +73,7 @@ void DiveListView::selectDive(struct dive *dive, bool scrollto, bool toggle)
expand(parent);
flags = toggle ? QItemSelectionModel::Toggle : QItemSelectionModel::Select;
flags |= QItemSelectionModel::Rows;
selectionModel()->select( idx, flags);
selectionModel()->select(idx, flags);
if (scrollto)
scrollTo(idx, PositionAtCenter);
}
@ -330,9 +331,19 @@ void DiveListView::removeFromTrip()
void DiveListView::deleteDive()
{
int nr;
struct dive *d = (struct dive *) contextMenuIndex.data(DiveTripModel::DIVE_ROLE).value<void*>();
if (d)
if (d) {
nr = get_divenr(d);
delete_single_dive(get_index_for_dive(d));
if (amount_selected == 0) {
if (nr > 0)
select_dive(nr - 1);
else
mainWindow()->cleanUpEmpty();
}
}
mainWindow()->refreshDisplay();
reload(currentLayout, false);
}