Summary: | new setting: tied zoom through all windows | ||
---|---|---|---|
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 |
Description
Nicolas Brisset
2005-02-04 18:02:46 UTC
Barth had said he wanted it to only work per-window. Barth? I think there are pretty different use cases for kst out there, and that's why I believe a kst setting would be appropriate. I know some people who would definitely want it to work globally. Now, I have no idea how difficult it would be to implement it. But if it's easy, I don't see any reason to not do it, as long as it remains a _configurable option_. CVS commit by arwalker: Added ability to define a cursor position within a plot by hitting the 'C' button (Hit SHIFT+'C" to remove the cursor). Once a cursor is set the status bar will show the offset from the cursor position to the current mouse position. This functionality also works in data mode. Do we want to add an item to the context menu for this? CCMAIL: 98564@bugs.kde.org M +249 -90 kst2dplot.cpp 1.393 M +12 -2 kst2dplot.h 1.153 Please ingore previous comment. Attached to wrong bug report. Enzo says: "This feature would be very useful." I agree. Should be fixed for 1.2.1 release I would propose adding a "Apply tied-zoom globally" checkbox to the General tab of the Kst Settings dialog. If checked the tied-zoom feature would work across all windows, otherwise it would continue to work as it does now. Created attachment 14859 [details]
Proposed patch
Makes global/window tied zoom configurable from Kst Settings dialog.
Also fixes several bugs where previously different zoom modes worked either
only on themselves, only across the window, or across all windows (though for
the last the updates for plots in other windows were not occurring).
Seems to work fine, but for some reason, minor grid lines are now default from the wizard... This is either from this patch, or a recent one from George. Also, I see that this fixes the log range 'bug' which is good. Go ahead and commit, but lets find the minor y grid bug. cbn On February 24, 2006 04:52 pm, Andrew Walker wrote: [bugs.kde.org quoted mail] On Sunday 26 February 2006 18:32, Barth Netterfield wrote: > Seems to work fine, but for some reason, minor grid lines are now default > from the wizard... This is either from this patch, or a recent one from > George. > > Also, I see that this fixes the log range 'bug' which is good. > > Go ahead and commit, but lets find the minor y grid bug. Conceptually it's fine with me but why does it have to touch KstApp and KstTopLevelView? We have everything we need in Kst2DPlot. It seems unnecessarily intrusive. And beyond that, KstApp and KstTopLevelView really shouldn't know about Kst2DPlot. I've been working to remove references to it. SVN commit 514309 by arwalker: BUG:98564 Better handle tied zoom locally and globally M +4 -0 libkst/kstsettings.cpp M +1 -0 libkst/kstsettings.h M +57 -0 libkstapp/kst.cpp M +4 -1 libkstapp/kst.h M +116 -108 libkstapp/kst2dplot.cpp M +4 -37 libkstapp/kst2dplot.h M +23 -0 libkstapp/kstsettingsdlg.ui M +4 -1 libkstapp/kstsettingsdlg.ui.h M +44 -0 libkstapp/ksttoplevelview.cpp M +5 -1 libkstapp/ksttoplevelview.h SVN commit 516363 by arwalker: CCBUG:98564 Replaced void* with const QString& M +9 -9 kst.cpp M +3 -3 kst.h M +24 -24 kst2dplot.cpp M +6 -6 ksttoplevelview.cpp M +3 -3 ksttoplevelview.h |