Bug 147435

Summary:	Resize slider in sharpness dialog doesn't work correct.
Product:	[Applications] digikam	Reporter:	Juergen Flosbach <jforum>
Component:	Plugin-Editor-Sharpen	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED NOT A BUG
Severity:	normal	CC:	andi.clemens
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:	7.6.0
Sentry Crash Report:

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.