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

List:       kde-commits
Subject:    branches/koffice/1.5/koffice/kspread trunk/koffice/kspread
From:       Stefan Nikolaus <stefan.nikolaus () kdemail ! net>
Date:       2006-05-22 16:19:06
Message-ID: 1148314746.927339.13221.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 543704 by nikolaus:

Selection	Paint the selection border around all ranges (again).
		Otherwise, one can't see which ranges are selected, if the
		selection color is too light/the same as the background. Also,
		one can't see which cells are merged.
CCBUG: 125774


 M  +5 -4      branches/koffice/1.5/koffice/kspread/kspread_canvas.cc  
 M  +5 -4      trunk/koffice/kspread/kspread_canvas.cc  


--- branches/koffice/1.5/koffice/kspread/kspread_canvas.cc #543703:543704
@@ -4369,11 +4369,11 @@
   if (d->cellEditor)
 	return;
 
-	if (!selectionInfo()->activeElement())
-			return;
+  Region::ConstIterator end(selectionInfo()->constEnd());
+  for (Region::ConstIterator it(selectionInfo()->constBegin()); it != end; ++it)
+  {
+    QRect range = (*it)->rect().normalize();
 
-	QRect range=selectionInfo()->activeElement()->rect().normalize();
-
   	double positions[4];
   	bool paintSides[4];
 
@@ -4468,6 +4468,7 @@
         }
       }
     }
+  }
 }
 
 void Canvas::sheetAreaToRect(const QRect& sheetArea, KoRect& rect)
--- trunk/koffice/kspread/kspread_canvas.cc #543703:543704
@@ -4353,11 +4353,11 @@
   if (d->cellEditor)
 	return;
 
-	if (!selectionInfo()->activeElement())
-			return;
+  Region::ConstIterator end(selectionInfo()->constEnd());
+  for (Region::ConstIterator it(selectionInfo()->constBegin()); it != end; ++it)
+  {
+    QRect range = (*it)->rect().normalize();
 
-	QRect range=selectionInfo()->activeElement()->rect();
-
   	double positions[4];
   	bool paintSides[4];
 
@@ -4452,6 +4452,7 @@
         }
       }
     }
+  }
 }
 
 void Canvas::sheetAreaToRect(const QRect& sheetArea, KoRect& rect)
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic