[Kst] [Bug 130281] Ellipse border width can't be set to 0

George Staikos staikos at kde.org
Wed Jul 5 11:46:56 CEST 2006


------- 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=130281         
staikos kde org changed:

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



------- Additional Comments From staikos kde org  2006-07-05 11:46 -------
SVN commit 558375 by staikos:

Add support for ellipses without borders

FEATURE: 130281


 M  +3 -3      kstviewellipse.cpp  


--- trunk/extragear/graphics/kst/src/libkstapp/kstviewellipse.cpp #558374:558375
 @ -93,7 +93,7  @
   }
 
   const int bw(_borderWidth * p.lineWidthAdjustmentFactor());
-  QPen pen(_borderColor, bw);
+  QPen pen(bw > 0 ? _borderColor : _foregroundColor, bw);
   p.setPen(pen);
   if (_transparentFill) {
     p.setBrush(Qt::NoBrush);  
 @ -128,7 +128,7  @
 
 void KstViewEllipse::setBorderWidth(int width) {
   if (_borderWidth != width) {
-    _borderWidth = kMax(1, width);
+    _borderWidth = kMax(0, width);
     setDirty();
   }
 }
 @ -160,7 +160,7  @
   } else if (propertyName == "borderWidth") {
     map.insert(QString("_kst_widgetType"), QString("QSpinBox"));
     map.insert(QString("_kst_label"), i18n("Border width"));    
-    map.insert(QString("minValue"), 1);
+    map.insert(QString("minValue"), 0);
   } else if (propertyName == "fillColor") {
     map.insert(QString("_kst_widgetType"), QString("KColorButton"));
     map.insert(QString("_kst_label"), i18n("Fill Color"));


More information about the Kst mailing list