Bug 147435 - Resize slider in sharpness dialog doesn't work correct.
Summary: Resize slider in sharpness dialog doesn't work correct.
Status: RESOLVED NOT A BUG
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-Editor-Sharpen (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-02 00:02 UTC by Juergen Flosbach
Modified: 2022-01-17 20:58 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 7.6.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Juergen Flosbach 2007-07-02 00:02:03 UTC
Version:           Unbekannt (using KDE 3.5.7 "release 47.1" , openSUSE )
Compiler:          Target: x86_64-suse-linux
OS:                Linux (x86_64) release 2.6.13-15.16-default

My words might be incorrect because I use the german version.

Go to the "sharpen" menu and try to resize the preview window
by pointing at the preview-size slider and use the mouse wheel
to move the slider.
The slider moves but the shown percentage doesn't change as well
as the preview window.

You can change the preview size if you click and drag the slider
with the left mouse button. But you don't get any information
about the preview size at the current slider position.
You get information about the preview size first if you release
the mouse button.

Juergen Flosbach
Comment 1 Dik Takken 2007-07-26 22:20:08 UTC
Confirmed. There seem to be some signals that need to be connected here. Only grabbing the slider with the mouse works at the moment.
Comment 2 Andi Clemens 2008-07-23 17:20:31 UTC
I can confirm this, right now we have the slider connected with a signal that is emitted on releasing the mouse button (or if the zoom buttons are pressed).
This was necessary for the old filter style because every new zoom step would have rerun the calculating of the filter.
But now sharpen has a "Try" button, so we can change this behavior. In the future we might add a "preview" mode to all those filter dialogs, so we can have both signals at one time. In preview mode, we zoom on mouse release and in "normal" mode we also zoom if slider is moved.
Comment 3 Andi Clemens 2008-09-01 14:05:05 UTC
This bugreport should be "invalid" now with the port to new image editor tools API.

Gilles,

agree?
Comment 4 caulier.gilles 2008-09-01 14:13:04 UTC
Andi,

yes, and no. We need to implement a common class named EditorToolPreview for all Editor Tool to render preview with zoom/pnaing, with a better support of zomming capabilities. There is already a file open here about this subject, where i have posted an uncomplete patch.

In fact, this file is a duplicate

Gilles Caulier
Comment 5 Andi Clemens 2008-09-01 14:18:25 UTC
Sure, but this bugreport is only concerning that the zoom toolbar in the dialog is not working correctly when using the mouse wheel. Since we have removed this tool completely, we should close this one?
Comment 6 caulier.gilles 2008-09-01 14:23:27 UTC
Yes, right, this file can be closed.

Gilles
Comment 7 Andi Clemens 2008-09-01 14:25:57 UTC
The zoom slider has been removed now since all image plugins will be / are converted to the new editortool API. Zooming is handled directly in the editor window toolbar now. So closing this one.