OSSP CVS Repository

ossp - Check-in [3913]
Not logged in
[Honeypot]  [Browse]  [Home]  [Login]  [Reports
[Search]  [Ticket]  [Timeline
  [Patchset]  [Tagging/Branching

Check-in Number: 3913
Date: 2002-Dec-20 19:19:12 (local)
2002-Dec-20 18:19:12 (UTC)
User:ms
Branch:
Comment: Tried to be more descriptive of sort policy on multiple selects, but functionality is out of member scope.
Tickets:
Inspections:
Files:
ossp-pkg/as/as-gui/TODO      1.58 -> 1.59     1 inserted, 1 deleted
ossp-pkg/as/as-gui/as_table.cpp      1.19 -> 1.20     5 inserted, 0 deleted

ossp-pkg/as/as-gui/TODO 1.58 -> 1.59

--- TODO 2002/12/20 15:13:52     1.58
+++ TODO 2002/12/20 18:19:12     1.59
@@ -25,7 +25,6 @@
 IDs in as_const.h much better choosing, so they make sense
 Be consistent with szQString vs QString var names
 After every result write a status bar message
-On sort, selection should move with swapped rows
 On multiple selection, no text should appear in edit ctrls
 Make consistent setEdition(), setDirty() default parameters
 
@@ -34,6 +33,7 @@
 Overidden paintCell() might not work when called asynchronously?
 Sometimes a click on add or delete toolbar button is ignored?
 When multiple new rows are added sometimes wrong row shading?
+When sorting rows, sometimes a false extra highlight appears
 Disabled actions still show enabled icons in menu popups
 Pressing escape on task data cells does not exit edit mode
 


ossp-pkg/as/as-gui/as_table.cpp 1.19 -> 1.20

--- as_table.cpp 2002/12/20 18:02:36     1.19
+++ as_table.cpp 2002/12/20 18:19:12     1.20
@@ -125,6 +125,11 @@
     this->setSortcol(nCol);
     QTable::sortColumn(nCol, this->getSortdir(), true);
 
+//    // Announce sorting policy with multiple selections
+//    QTableSelection Testsel = this->selection(this->currentSelection());
+//    if (Testsel.topRow() != Testsel.bottomRow())
+//        m_pStatbar->message(trUtf8("Multiple selections dropped when sorting"), 4000);
+
     // Move and display the selection highlight
     this->removeSelection(this->currentSelection());
     this->selectRow(this->currentRow());

CVSTrac 2.0.1