Summary: | Auto-resize for labels: improve usability | ||
---|---|---|---|
Product: | [Applications] kst | Reporter: | Nicolas Brisset <nicolas.brisset> |
Component: | general | Assignee: | kst |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | ||
Priority: | HI | ||
Version: | 1.x | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Solaris | ||
Latest Commit: | Version Fixed In: | ||
Attachments: |
Proposed patch
Proposed patch |
Description
Nicolas Brisset
2006-02-01 17:20:22 UTC
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 |