[Kst] [Bug 118678] Ellipse drawn incorrectly when border width is zero

[George Staikos]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=118678         
staikos kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From staikos kde org  2005-12-20 01:48 -------
SVN commit 489872 by staikos:

QPainter::drawEllipse doesn't seem to work with 0-width pen 
BUG: 118678


 M  +3 -3      kstviewellipse.cpp  


--- trunk/extragear/graphics/kst/kst/kstviewellipse.cpp #489871:489872
 @ -27,7 +27,7  @
 #include <qvariant.h>
 
 KstViewEllipse::KstViewEllipse()
-: KstViewObject("Ellipse"), _borderWidth(0) {
+: KstViewObject("Ellipse"), _borderWidth(1) {
   setTransparent(true);
   _transparentFill = false;
 }
 @ -111,7 +111,7  @
 
 void KstViewEllipse::setBorderWidth(int width) {
   if (_borderWidth != width) {
-    _borderWidth = width;
+    _borderWidth = kMax(1, width);
     setDirty();
   }
 }
 @ -143,7 +143,7  @
   } else if (propertyName == "borderWidth") {
     map.insert(QString("_kst_widgetType"), QString("QSpinBox"));
     map.insert(QString("_kst_label"), i18n("Border width"));    
-    map.insert(QString("minValue"), 0);
+    map.insert(QString("minValue"), 1);
   } else if (propertyName == "fillColor") {
     map.insert(QString("_kst_widgetType"), QString("KColorButton"));
     map.insert(QString("_kst_label"), i18n("Fill Color"));