[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-kimageshop
Subject:    Re: backward compatibility
From:       Boudewijn Rempt <boud () valdyas ! org>
Date:       2010-06-22 14:09:00
Message-ID: 201006221609.00662.boud () valdyas ! org
[Download RAW message or body]

On Monday 21 June 2010, LukasT.dev@gmail.com wrote:
> > ...Or did I miss something?
> 
> There are two stages when loading the presets. One for the paintop and one
> for the GUI widgets.
> 
> I did the check when the paintop is loading the preset in
> KisDeformPaintOpSettings. But the GUI part of the paintop has also code for
> loading the preset. The airbrush option loads the preset at
> KisAirbrushOption::readOptionSetting(). That class is part of the library
> and does not know about deform brush (and it's presets strings).
> 
> I could hardcode DEFORM_USE_MOVEMENT_PAINT there so that the GUI is setup
> correctly in the deform brush case. But I don't think it's nice.

Hm, I have no real advice here. I can try to find time to look at the actual 
code tonight, if you remind me to :-)

-- 
Boudewijn Rempt | http://www.valdyas.org
_______________________________________________
kimageshop mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop
[prev in list] [next in list] [prev in thread] [next in thread]