Bug 142699

Summary:	New symbol type is desirable
Product:	[Applications] kst	Reporter:	Andrew Walker <arwalker>
Component:	general	Assignee:	kst
Status:	RESOLVED FIXED
Severity:	wishlist
Priority:	NOR
Version First Reported In:	1.x
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Proposed patch

Description Andrew Walker 2007-03-08 20:32:49 UTC

Version:           HEAD (using KDE KDE 3.5.1)
Installed from:    Compiled From Sources
OS:                Linux

The LFI team would like to see a new symbol type for data points that is a simple point of 1 pixel in size.

This is useful when displaying large amounts of data where the x-axis is not monotonically increasing.

Comment 1 Andrew Walker 2007-03-08 20:37:02 UTC

Created attachment 19913 [details]
Proposed patch

Comment 2 Netterfield 2007-03-08 20:51:28 UTC

Seems ok.  Should it respect line width or something like that?


On Thursday 08 March 2007 2:37:03 pm Andrew Walker wrote:
[bugs.kde.org quoted mail]

Comment 3 Andrew Walker 2007-03-08 21:30:24 UTC

SVN commit 640694 by arwalker:

BUG:142699 Add point to symbol type where the point size respects the line width.

 M  +8 -0      kstcurvepointsymbol.cpp  
 M  +1 -1      kstcurvepointsymbol.h  


--- trunk/extragear/graphics/kst/src/libkstmath/kstcurvepointsymbol.cpp #640693:640694
@@ -128,6 +128,14 @@
         p->drawPolygon(pts);
       }
       break;
+    case 13:
+      if (s < 4) {
+        p->drawPoint(x,y);
+      } else {
+        p->setBrush(QBrush::SolidPattern);
+        p->drawPie(x-(s/4), y-(s/4), s/2, s/2, 0, 5760);
+      }
+      break;
   }
 
   p->setBrush(QBrush::NoBrush);
--- trunk/extragear/graphics/kst/src/libkstmath/kstcurvepointsymbol.h #640693:640694
@@ -21,7 +21,7 @@
 #include "kst_export.h"
 #include <qpainter.h>
 
-#define KSTPOINT_MAXTYPE 13
+#define KSTPOINT_MAXTYPE 14
 
 namespace  KstCurvePointSymbol {
     /** draw the point on a painter the scale of the point is based on size */