UpsilonNumworks/Upsilon
1
0
Fork 0
mirror of https://github.com/UpsilonNumworks/Upsilon.git synced 2026-01-19 00:37:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

[escher] SelectableTableView: fix wrong assert

Browse Source
This commit is contained in:
Émilie Feral
2018-11-26 14:29:37 +01:00
committed by EmilieNumworks
parent ca2fdf8ffd
commit df45f800e1
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
escher/src/selectable_table_view.cpp
View File

@@ -158,7 +158,15 @@ void SelectableTableView::unhighlightSelectedCell() {
if (selectedColumn() >= 0 && selectedColumn() < dataSource()->numberOfColumns() &&
selectedRow() >= 0 && selectedRow() < dataSource()->numberOfRows()) {
HighlightCell * previousCell = cellAtLocation(selectedColumn(), selectedRow());
assert(previousCell);
previousCell->setHighlighted(false);
/* Previous cell does not always exist.
* For example, unhighlightSelectedCell can be called twice:
* - from selectCellAtLocation
* - and then from m_delegate->tableViewDidChangeSelection inside unhighlightSelectedCell
* The first call selects an invisible cell. At the time of the second call,
* the selected cell might be still invisible because scrolling happens
* after. */
if (previousCell) {
previousCell->setHighlighted(false);
}
}
}