KisSelection vs. KisPixelSelection

Boudewijn Rempt boud at valdyas.org
Mon Jun 21 21:27:26 CEST 2010


On Sunday 20 June 2010, Sven Langkamp wrote:
> Both classes share only a few methods that are common to both. At the same
> time they have lots of stuff that is specific to one of them like updating
> the projection in KisSelection or manipulation methods in pixel selection.
> It would make more sense to put the methods that are the same in both
> classes into a shared base class.

Hm... I'm not completely following the issues Dmitry has with the current 
design, but reflecting on the current code, I think it's a bit ugly that 
KisSelection _is_ the projection. How about something like:

Make a KisSelection class that is not a paint device. Make it own:

* KisPixelSelection::KisPaintdevice
* KisVectorSelection
* and a KisPaintDevice for the projection.

similar to KisImage?

-- 
Boudewijn Rempt | http://www.valdyas.org


More information about the kimageshop mailing list