Bug 274321

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 "Apply to plots etc [ ]"

Thoughts?

Ok... here is what I propose:

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

ii) add a dialog or page which is "globally set defaults" 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

Ahhhh.... I totally overlooked that "Save as default".  That seems
to do what I want.

> i) fix the current tab to say "page background settings" and remove reference
> to fonts, etc

Good.

> ii) add a dialog or page which is "globally set defaults" 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.

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

What I'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 "save as default".
To wit, a dialog box with panels called:

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

> iii) add saving things (like label font color) which are not currently saved

Sure.

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'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&revision=1237013

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&revision=1237296

Fixed by Barth in commit 1238324. For some reason the post-commit hook did not work...