Plot objects (and perhaps other view objects?) don'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 -> Configure kst 2. Under "Default Fill Properties" change Color to, say, a rather urghsome shade of green. 3. OK. 4. Create -> 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.
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...