Merge remote-tracking branch 'origin/bugfix/6980_curve_colour_choosing'

mantidproject · Oct 25, 2013 · a58b3ce · a58b3ce
2 parents 77b2f87 + e8a67c6
commit a58b3ce
Showing 1 changed file with 12 additions and 14 deletions.
diff --git a/Code/Mantid/MantidPlot/src/Graph.cpp b/Code/Mantid/MantidPlot/src/Graph.cpp
@@ -5015,27 +5015,25 @@ void Graph::guessUniqueCurveLayout(int& colorIndex, int& symbolIndex)
     }
   }
 
-  for (int i=0; i<n_curves; i++){
+  for (int i=0; i<n_curves; ++i)
+  {
     const PlotCurve *c = dynamic_cast<PlotCurve *>(curve(i));
-    if (c){
-      int index = ColorBox::colorIndex(c->pen().color());
-      if (index > colorIndex)
-        colorIndex = index;
+    if (c)
+    {
+      colorIndex = std::max(ColorBox::colorIndex(c->pen().color()), colorIndex);
 
       QwtSymbol symb = c->symbol();
-      index = SymbolBox::symbolIndex(symb.style());
-      if (index > symbolIndex)
-        symbolIndex = index;
+      symbolIndex = std::max(SymbolBox::symbolIndex(symb.style()), symbolIndex);
     }
   }
   if (n_curves > 1)
-    colorIndex = (colorIndex+1)%16;
-  if (colorIndex == 15) //avoid white invisible curves
-    colorIndex = 0;
+    colorIndex = (++colorIndex)%ColorBox::numPredefinedColors();
+  if (ColorBox::color(colorIndex) == Qt::white) //avoid white invisible curves
+    ++colorIndex;
 
-  symbolIndex = (symbolIndex+1)%15;
-  if (!symbolIndex)
-    symbolIndex = 1;
+  symbolIndex = (++symbolIndex)%15;
+  if (symbolIndex == 0)
+    ++symbolIndex;
 }
 
 void Graph::addFitCurve(QwtPlotCurve *c)