Version: 1.2.0_devel (using KDE 3.4.0, compiled sources) Compiler: gcc version 3.4.3 OS: SunOS (sun4u) release 5.8 Currently, newly created labels by default do not auto-resize for text. What's more, the "Margin" spinbox is not disabled when the auto-resize feature is disabled, whereas it doesn't make sense in that case. I suggest the following improvements (please don't tell me they should be separate reports!): 1) move the "Margin" spinbox closer to the "Auto-resize for text" checkbox 2) disable the "Margin" spinbox when it does not make sense 3) change the "Auto-resize for text" checkbox to radiobuttons "Fit box to text", "Fit text to box", "No fit" 4) (less important) remember the "fit" settings used when creating the last label Barth, I think point 3) solves the concern you expressed in the discussion on the mailing list in an elegant way.
Counter proposal: -Get rid of the Auto Resize check box -Make labels always Auto resized If you want a label in a box which is not autoresized, put a borderless transparent label in a box. Rational: a non-auto-resized label doesn't resize fonts on resize, which completely breaks What You See Is Sort Of What You Get (WYSISOWYG). This is what we did with legends already.
That sounds good to me. Less hacky code to keep running.
OK, sounds acceptable and simpler.
Created attachment 16723 [details] Proposed patch As per Barth's suggestion
You cannot remove public API from KstScript anymore. It will break existing scripts. They'll throw an exception and halt. I think this needs to be rethought in terms of implementation at least.
So we put the stuff back into scripting but the function does nothing.
Created attachment 16727 [details] Proposed patch
please don't remove the commented code about tab/cr expansion breaking latex support. I don't want that regression to creep in. Also the JavaScript docs need to be updated (in the .h file) to mark the function as obsolete. The rest looks fine now. I guess there's not much we can do.
SVN commit 553369 by arwalker: BUG:121167 Remove auto-resize option from view label as it breaks desired WYSIWYG behaviour. M +3 -17 extensions/js/bind_label.cpp M +1 -4 extensions/js/bind_label.h M +0 -1 libkstapp/kst2dplot.cpp M +0 -1 libkstapp/kstfitdialog_i.cpp M +0 -1 libkstapp/kstgfxtextmousehandler.cpp M +28 -65 libkstapp/kstviewlabel.cpp M +0 -5 libkstapp/kstviewlabel.h M +36 -42 libkstapp/viewlabelwidget.ui