[Kst] [Bug 87988] Curves sometimes cover up axes ticks

Andrew Walker arwalker at sumusltd.com
Wed Aug 25 19:48:26 CEST 2004


------- 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=87988      
arwalker sumusltd com changed:

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



------- Additional Comments From arwalker sumusltd com  2004-08-25 19:48 -------
CVS commit by arwalker: 

Plot the axes after the curves, so that the tickmarks are not covered by any curve.

CCMAIL: 87988-done bugs kde org


  M +5 -7      kst2dplot.cpp   1.252


--- kdeextragear-2/kst/kst/kst2dplot.cpp  #1.251:1.252
 @ -1575,11 +1575,4  @ void Kst2DPlot::draw(QPainter &p, KstPai
                   YTick, ytop_bdr_px, ybot_bdr_px, y_orig_px, ytick_px, ytick_len_px, y_px);
 
-    //return original pen to painter
-    p.setPen(QPen(_foregroundColor, penWidth));
-
-    plotAxes(p, RelPlotRegion, x_max, y_max, x_min, y_min,
-                        XTick, Xorg, xleft_bdr_px, xright_bdr_px, x_orig_px, xtick_px, xtick_len_px, x_px,
-                        YTick, Yorg, ytop_bdr_px, ybot_bdr_px, y_orig_px, ytick_px, ytick_len_px, y_px);
-
     /*** plot the legend now if its in the background **/
     if (!Legend->getFront()) {
 @ -1590,4 +1583,9  @ void Kst2DPlot::draw(QPainter &p, KstPai
     plotCurves(p, Lx, Hx, Ly, Hy, m_X, m_Y, b_X, b_Y, penWidth);
 
+    p.setPen(QPen(_foregroundColor, penWidth));
+    plotAxes(p, RelPlotRegion, x_max, y_max, x_min, y_min,
+                        XTick, Xorg, xleft_bdr_px, xright_bdr_px, x_orig_px, xtick_px, xtick_len_px, x_px,
+                        YTick, Yorg, ytop_bdr_px, ybot_bdr_px, y_orig_px, ytick_px, ytick_len_px, y_px);
+
     plotPlotMarkers(p, m_X, b_X, x_max, x_min, y_px, ytop_bdr_px, ybot_bdr_px);



More information about the Kst mailing list