[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdeextragear-2/kst/kst
From:       George Staikos <staikos () kde ! org>
Date:       2003-10-02 4:37:35
[Download RAW message or body]

CVS commit by staikos: 

This was the result of a misguided patch.  Unfortunately KstMouse will make it
overly complicated to add more mouse modes in the future due to its concept of
"plotNum".  It might be a good idea to store more stateful information, and
actually have the plotNum reflect the plot it's actually in.  I think this
could simplify KstView somewhat too.  Anyways, something to think about for the
future.

CCMAIL: 65278-done@bugs.kde.org


  M +3 -3      kstview.cpp   1.52


--- kdeextragear-2/kst/kst/kstview.cpp  #1.51:1.52
@@ -504,14 +504,14 @@ void KstView::mouseMoveEvent(QMouseEvent
   }
 
-  MouseInfo->setPlotNum(-1);
-
   for (i = i0; i < iN; i++) {
     QRect plot_rect = KST::plotList.at(i)->GetPlotRegion();
     if (plot_rect.contains(e->pos())) {
-      MouseInfo->setPlotNum(i);
       if (MouseInfo->getMode() == INACTIVE) {
+        MouseInfo->setPlotNum(i);
         if (i != plot_num) {
           updateTieBoxes(this);
         }
+      } else if (MouseInfo->getMode() == LABEL_TOOL) {
+        MouseInfo->setPlotNum(i);
       }
       in_plot = true;


[prev in list] [next in list] [prev in thread] [next in thread]