From 64f3533baa6463879500569f1b539864131451ee Mon Sep 17 00:00:00 2001 From: Timothy Pearson Date: Thu, 15 Nov 2012 15:33:01 -0600 Subject: Fix keyboard multi-select failure in ksysguard --- ksysguard/gui/SensorDisplayLib/ProcessList.cc | 133 ++++++++++++++++---------- ksysguard/gui/SensorDisplayLib/ProcessList.h | 6 ++ 2 files changed, 87 insertions(+), 52 deletions(-) (limited to 'ksysguard/gui') diff --git a/ksysguard/gui/SensorDisplayLib/ProcessList.cc b/ksysguard/gui/SensorDisplayLib/ProcessList.cc index e0b5f79c2..44ce491db 100644 --- a/ksysguard/gui/SensorDisplayLib/ProcessList.cc +++ b/ksysguard/gui/SensorDisplayLib/ProcessList.cc @@ -228,6 +228,8 @@ ProcessList::ProcessList(TQWidget *parent, const char* name) * settings. */ connect(header(), TQT_SIGNAL(clicked(int)), this, TQT_SLOT(sortingChanged(int))); + ctrlKeyDown = false; + shiftKeyDown = false; treeViewEnabled = false; openAll = true; @@ -294,73 +296,76 @@ ProcessList::getSelectedAsStrings() bool ProcessList::update(const TQString& list) { - /* Disable painting to avoid flickering effects, - * especially when in tree view mode. - * Ditto for the scrollbar. */ - setUpdatesEnabled(false); - viewport()->setUpdatesEnabled(false); - - pl.clear(); - - // Convert ps answer in a list of tokenized lines - KSGRD::SensorTokenizer procs(list, '\n'); - for (unsigned int i = 0; i < procs.count(); i++) + if ((!shiftKeyDown) && (!ctrlKeyDown)) { - KSGRD::SensorPSLine* line = new KSGRD::SensorPSLine(procs[i]); - if (line->count() != (uint) columns()) + /* Disable painting to avoid flickering effects, + * especially when in tree view mode. + * Ditto for the scrollbar. */ + setUpdatesEnabled(false); + viewport()->setUpdatesEnabled(false); + + pl.clear(); + + // Convert ps answer in a list of tokenized lines + KSGRD::SensorTokenizer procs(list, '\n'); + for (unsigned int i = 0; i < procs.count(); i++) { + KSGRD::SensorPSLine* line = new KSGRD::SensorPSLine(procs[i]); + if (line->count() != (uint) columns()) + { #if 0 - // This is needed for debugging only. - kdDebug(1215) << list << endl; - TQString l; - for (uint j = 0; j < line->count(); j++) - l += (*line)[j] + "|"; - kdDebug(1215) << "Incomplete ps line:" << l << endl; + // This is needed for debugging only. + kdDebug(1215) << list << endl; + TQString l; + for (uint j = 0; j < line->count(); j++) + l += (*line)[j] + "|"; + kdDebug(1215) << "Incomplete ps line:" << l << endl; #endif - return (false); + return (false); + } + else + pl.append(line); } + + int currItemPos = itemPos(currentItem()); + int vpos = verticalScrollBar()->value(); + int hpos = horizontalScrollBar()->value(); + + updateMetaInfo(); + + clear(); + + if (treeViewEnabled) + buildTree(); else - pl.append(line); - } - - int currItemPos = itemPos(currentItem()); - int vpos = verticalScrollBar()->value(); - int hpos = horizontalScrollBar()->value(); - - updateMetaInfo(); - - clear(); - - if (treeViewEnabled) - buildTree(); - else - buildList(); - - TQListViewItemIterator it( this ); - while ( it.current() ) { - if ( itemPos( it.current() ) == currItemPos ) { - setCurrentItem( it.current() ); - break; + buildList(); + + TQListViewItemIterator it( this ); + while ( it.current() ) { + if ( itemPos( it.current() ) == currItemPos ) { + setCurrentItem( it.current() ); + break; + } + ++it; } - ++it; + + verticalScrollBar()->setValue(vpos); + horizontalScrollBar()->setValue(hpos); + + // Re-enable painting, and force an update. + setUpdatesEnabled(true); + viewport()->setUpdatesEnabled(true); + + triggerUpdate(); } - verticalScrollBar()->setValue(vpos); - horizontalScrollBar()->setValue(hpos); - - // Re-enable painting, and force an update. - setUpdatesEnabled(true); - viewport()->setUpdatesEnabled(true); - - triggerUpdate(); - return (true); } void ProcessList::setTreeView(bool tv) { - if (treeViewEnabled = tv) + if ((treeViewEnabled = tv)) { savedWidth[0] = columnWidth(0); openAll = true; @@ -938,4 +943,28 @@ ProcessList::selectAllChilds(int pid, bool select) } } +void +ProcessList::keyPressEvent(TQKeyEvent *e) +{ + if (e->key() == Key_Shift) { + shiftKeyDown = true; + } + if (e->key() == Key_Control) { + ctrlKeyDown = true; + } + KListView::keyPressEvent(e); +} + +void +ProcessList::keyReleaseEvent(TQKeyEvent *e) +{ + if (e->key() == Key_Shift) { + shiftKeyDown = false; + } + if (e->key() == Key_Control) { + ctrlKeyDown = false; + } + KListView::keyReleaseEvent(e); +} + #include "ProcessList.moc" diff --git a/ksysguard/gui/SensorDisplayLib/ProcessList.h b/ksysguard/gui/SensorDisplayLib/ProcessList.h index 370fe23fd..fc668cf15 100644 --- a/ksysguard/gui/SensorDisplayLib/ProcessList.h +++ b/ksysguard/gui/SensorDisplayLib/ProcessList.h @@ -170,6 +170,10 @@ signals: void listModified(bool); +protected: + void keyPressEvent(TQKeyEvent *e); + void keyReleaseEvent(TQKeyEvent *e); + private: // items of table header RMB popup menu enum @@ -244,6 +248,8 @@ private: bool killSupported; bool treeViewEnabled; bool openAll; + bool ctrlKeyDown; + bool shiftKeyDown; /* The following lists are primarily used to store table specs between * load() and the actual table creation in addColumn(). */ -- cgit v1.2.1