<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>274321</bug_id>
          
          <creation_ts>2011-05-28 03:33:52 +0000</creation_ts>
          <short_desc>Plot objects ignore default fill</short_desc>
          <delta_ts>2011-06-28 21:07:54 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>kst</product>
          <component>view objects</component>
          <version>2.0.4</version>
          <rep_platform>Compiled Sources</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.0.4</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="D. V. Wiebe">dvw</reporter>
          <assigned_to>kst</assigned_to>
          <cc>netterfield</cc>
          
          <cf_commitlink></cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1124324</commentid>
    <comment_count>0</comment_count>
    <who name="D. V. Wiebe">dvw</who>
    <bug_when>2011-05-28 03:33:52 +0000</bug_when>
    <thetext>Plot objects (and perhaps other view objects?) don&apos;t respect the default fill set in the Kst Settings dialog box, but always end up being solid black on solid white.  (Using SVN r1233845).

Steps to reproduce:
1. Settings -&gt; Configure kst
2. Under &quot;Default Fill Properties&quot; change Color to, say, a rather urghsome shade of green.
3. OK.
4. Create -&gt; Plot

Result:
  A black plot on a white background is produced.

Expected result:
  A black plot on a rather-urghsome-shade-of-green background is produced.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1128891</commentid>
    <comment_count>1</comment_count>
    <who name="Netterfield">netterfield</who>
    <bug_when>2011-06-08 14:49:43 +0000</bug_when>
    <thetext>This is not formally a bug, but more... unclear UI.

The settings default fill properties currently only apply to the background of tabs and not anything else.  So If you had created a new tab, the background would have been a rather-urghsome-shade-of-green.

To set the defaults on plots, there is a check-box in the plot dialog 
[ ] Save as default.  

Applying after setting this will set all of the plot settings, including lines and backgrounds, to the new setting.  This does work.

There are 2+1 options as I see it:
1) Keep the current functionality, but change the text in the dialog to be more clear
2)Change the functionality so the settings dialog also changes the settings for view-tiems
1+2) optionally option 2, with a checkbox &quot;Apply to plots etc [ ]&quot;

Thoughts?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1128914</commentid>
    <comment_count>2</comment_count>
    <who name="Netterfield">netterfield</who>
    <bug_when>2011-06-08 15:24:05 +0000</bug_when>
    <thetext>Ok... here is what I propose:

i) fix the current tab to say &quot;page background settings&quot; and remove reference to fonts, etc

ii) add a dialog or page which is &quot;globally set defaults&quot; and put in it background, stroke, fonts, etc.  This will re-set all of the relevant defaults, which can then be re-reset individually in each of the view item dialogs, as now.

iii) add saving things (like label font color) which are not currently saved</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1129084</commentid>
    <comment_count>3</comment_count>
    <who name="D. V. Wiebe">dvw</who>
    <bug_when>2011-06-08 23:26:57 +0000</bug_when>
    <thetext>Ahhhh.... I totally overlooked that &quot;Save as default&quot;.  That seems
to do what I want.

&gt; i) fix the current tab to say &quot;page background settings&quot; and remove reference
&gt; to fonts, etc

Good.

&gt; ii) add a dialog or page which is &quot;globally set defaults&quot; and put in it
&gt; background, stroke, fonts, etc.  This will re-set all of the relevant defaults,
&gt; which can then be re-reset individually in each of the view item dialogs, as
&gt; now.

You mean it would set, say, the default fill of any type of object to, say,
red?  That could work, I guess.

What I&apos;d like to see, in the interest of full disclosure if nothing
else, is a central location where *all* the default object properties
which kst saves for *anything* are shown and can be modified, without having
to do it by proxy with an existing object first and then &quot;save as default&quot;.
To wit, a dialog box with panels called:

+ Default Plot Fill
+ Default Plot Stroke
+ Default Legend Fill
&amp;c.

&gt; iii) add saving things (like label font color) which are not currently saved

Sure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1131984</commentid>
    <comment_count>4</comment_count>
    <who name="Netterfield">netterfield</who>
    <bug_when>2011-06-16 18:51:36 +0000</bug_when>
    <thetext>SVN commit 1237013 by netterfield:

CCBUG: 274323 
CCBUG: 274321
All view objects for which fill and stroke mean anything can now save their default fill and stroke.
View objects for which these things don&apos;t make sense (eg, fill for lines) no longer have the tab in their
dialogs.



 M  +2 -0      arrowitem.cpp  
 M  +6 -0      arrowitem.h  
 M  +2 -0      boxitem.cpp  
 M  +6 -0      boxitem.h  
 M  +1 -0      cartesianrenderitem.h  
 M  +2 -0      circleitem.cpp  
 M  +6 -0      circleitem.h  
 M  +2 -0      ellipseitem.cpp  
 M  +6 -0      ellipseitem.h  
 M  +6 -6      labelitem.cpp  
 M  +6 -0      labelitem.h  
 M  +2 -0      layoutboxitem.h  
 M  +22 -50    legenditem.cpp  
 M  +6 -0      legenditem.h  
 M  +1 -0      lineitem.cpp  
 M  +6 -0      lineitem.h  
 M  +6 -0      pictureitem.h  
 M  +79 -139   plotitem.cpp  
 M  +6 -0      plotitem.h  
 M  +1 -0      sharedaxisboxitem.h  
 M  +6 -0      svgitem.h  
 M  +78 -0     viewitem.cpp  
 M  +12 -0     viewitem.h  
 M  +33 -5     viewitemdialog.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&amp;revision=1237013</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1132308</commentid>
    <comment_count>5</comment_count>
    <who name="Netterfield">netterfield</who>
    <bug_when>2011-06-17 18:28:25 +0000</bug_when>
    <thetext>SVN commit 1237296 by netterfield:

Make more of the settings code make sense.
Remove some dead things from the settings dialog.
Fix bug where entering and leaving a view item with a draw tool selected aborted the ability to draw.
CCBUG: 274321



 M  +0 -40     applicationsettings.cpp  
 M  +0 -13     applicationsettings.h  
 M  +4 -9      applicationsettingsdialog.cpp  
 M  +2 -2      arrowitem.cpp  
 M  +0 -2      datawizard.cpp  
 M  +0 -34     defaultlabelpropertiestab.cpp  
 M  +0 -9      defaultlabelpropertiestab.h  
 M  +8 -79     defaultlabelpropertiestab.ui  
 M  +1 -1      labelitem.cpp  
 M  +1 -1      legenditem.cpp  
 M  +1 -1      lineitem.cpp  
 M  +9 -12     plotitem.cpp  
 M  +7 -25     view.cpp  
 M  +1 -3      view.h  
 M  +2 -4      viewitem.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&amp;revision=1237296</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1136537</commentid>
    <comment_count>6</comment_count>
    <who name="Nicolas Brisset">nicolas.brisset</who>
    <bug_when>2011-06-28 21:07:54 +0000</bug_when>
    <thetext>Fixed by Barth in commit 1238324. For some reason the post-commit hook did not work...</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>