OilfieldToolsNet/ResInsight
4
0
Fork 0
mirror of https://github.com/OPM/ResInsight.git synced 2025-01-21 22:13:25 -06:00
Code Issues Packages Projects Releases Wiki Activity

#879 AppFwk : Workaround for navigation with arrow keys during single selection in multi selection lists

Browse Source
This commit is contained in:
Magne Sjaastad 2016-11-10 11:57:51 +01:00
parent c486a7b855
commit c830afbdc1
1 changed files with 17 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
Fwk/AppFwk/cafUserInterface/cafPdmUiListEditor.cpp
View File

@ -272,8 +272,6 @@ void PdmUiListEditor::slotSelectionChanged(const QItemSelection & selected, cons
// An invalid call to setSelection() from QAbstractItemView::keyPressEvent() causes the stepping using arrow keys
// in a single selection list to not work as expected.
//
// NOTE: Multi select in selection lists do still have issues similar to the above description.
//
// See also https://github.com/OPM/ResInsight/issues/879
//
// WORKAROUND : The list view is in single selection mode, and the selection is set based on current index
@ -290,9 +288,24 @@ void PdmUiListEditor::slotSelectionChanged(const QItemSelection & selected, cons
}
else if (fieldValue.type() == QVariant::List)
{
QList<QVariant> valuesToSetInField;
QModelIndexList idxList = m_listView->selectionModel()->selectedIndexes();
if (idxList.size() <= 1)
{
// NOTE : Workaround for update issue seen on RHEL6 with Qt 4.6.2
// An invalid call to setSelection() from QAbstractItemView::keyPressEvent() causes the stepping using arrow keys
// in a multi selection list to not work as expected.
//
// See also https://github.com/OPM/ResInsight/issues/879
//
// WORKAROUND : The list view has one or none items selected, manually set the selection from current index
m_listView->selectionModel()->select(m_listView->currentIndex(), QItemSelectionModel::SelectCurrent);
// Update the list of indexes after selection has been modified
idxList = m_listView->selectionModel()->selectedIndexes();
}
QList<QVariant> valuesToSetInField;
for (int i = 0; i < idxList.size(); ++i)
{
if (idxList[i].row() < m_options.size())