[calligra] krita: only show filter dialog if the filter has a configuration widget
Cyrille Berger Skott
cberger at cberger.net
Fri Feb 25 08:39:47 CET 2011
On Wednesday 23 February 2011, Sven Langkamp wrote:
> Git commit 97b361b920d662b723f76f9a3a8a3065c502e91a by Sven Langkamp.
> Committed on 23/02/2011 at 18:00.
> Pushed by langkamp into branch 'master'.
>
> only show filter dialog if the filter has a configuration widget
Hum. The idea to show the dialog was to show a preview for the filter (even
allowing browsing if you are not satisfied with the current result), and also
that dialog is supposed to have a "create mask" button. With that change, it
would make it an unreliable way to create a filter mask. Also breaking the
consitency between filters.
So I feel this change deserves a wider discussion.
> +bool KisBaseProcessor::showConfigurationWidget()
> +{
> + return d->showConfigurationWidget;
> +}
> +
> +void KisBaseProcessor::setShowConfigurationWidget(bool v)
> +{
> + d->showConfigurationWidget = v;
> +}
Wouldn't it be easier and more reliable to actually check if the
KisBaseProcessor return a null configuration widget ?
--
Cyrille Berger Skott
More information about the kimageshop
mailing list