Bug 124985 - Legend drawing artefacts between text and border
Summary: Legend drawing artefacts between text and border
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: unspecified Solaris
: NOR normal
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-04-05 17:33 UTC by Nicolas Brisset
Modified: 2006-06-01 19:47 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
Example (8.50 KB, image/png)
2006-04-05 17:34 UTC, Nicolas Brisset
Details
kst file to use with test.dat just attached to bug #128436 (5.81 KB, text/plain)
2006-06-01 12:55 UTC, Nicolas Brisset
Details
The way the "new" kst stores the same thing (5.55 KB, text/plain)
2006-06-01 12:56 UTC, Nicolas Brisset
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Brisset 2006-04-05 17:33:14 UTC
Version:           1.3.0_devel (using KDE 3.4.0, compiled sources)
Compiler:          gcc version 3.4.3
OS:                SunOS (sun4u) release 5.8

In my setup (today's svn), legends exhibit drawing artefacts in the area between text and border. The rectangle around the text has the solid background color, but it seems is does not extend far enough to reach the border. I will attach a screenshot to make it clearer. As soon as another window or a menu covers the legend, it leaves traces behind. And if grids are used, they can be seen in the interval as well.
Comment 1 Nicolas Brisset 2006-04-05 17:34:35 UTC
Created attachment 15479 [details]
Example
Comment 2 Andrew Walker 2006-05-08 19:39:20 UTC
I'm unable to reproduce this. Nicolas, would you mind checking this is still a problem with the latest version of the code. Thanks.
Comment 3 Nicolas Brisset 2006-05-11 18:01:36 UTC
Well, I can't reproduce it. It must have been fixed somehow, I'll close the bug for now.
Comment 4 Nicolas Brisset 2006-06-01 12:48:00 UTC
Well, actually it seems the problem appears when loading "old" kst files with a newer kst version. I'll attach a test case shortly :-)
Comment 5 Nicolas Brisset 2006-06-01 12:55:21 UTC
Created attachment 16399 [details]
kst file to use with test.dat just attached to bug #128436

Try to reload the .kst file with kst from svn: the artefacts are visible. This
.kst was generated with an older version (1.2.0_devel) of kst. 
By the way, it would be cool if the "About kst..." dialog showed the svn
revision number along with the human-readable one, unless there is a way to
find it elsewhere when you don't remember from which of the many extragear
copies you have lying around the installed version comes :-)
Comment 6 Nicolas Brisset 2006-06-01 12:56:40 UTC
Created attachment 16400 [details]
The way the "new" kst stores the same thing

Looking at the .kst files, there are differences indeed. So, basically, this is
a lack of upward compatibility...
Comment 7 Andrew Walker 2006-06-01 19:47:48 UTC
SVN commit 547320 by arwalker:

BUG:124985 In the old kst file the padding was set to 5. The padding should always be zero for this object.

 M  +2 -0      kstviewlegend.cpp  


--- trunk/extragear/graphics/kst/src/libkstapp/kstviewlegend.cpp #547319:547320
@@ -124,6 +124,8 @@
       addCurve(*it);
     }
   }
+  
+  setPadding(0);
 }