New brush dialog

[LukasT.dev@gmail.com]

On Monday, June 21, 2010 21:17:46 Boudewijn Rempt wrote:
> On Thursday 17 June 2010, LukasT.dev at gmail.com wrote:
> > Problem:
> > 1.find correct names - both Brush Tip and and Spray shape are Particle
> > types. How to call them?
> 
> Brush Tip and Spray Shape?

That's wondering question or suggestion?

> > 2. Small problem - hide spacing from GUI as spacing is not used by spray
> > (similary for the hairy brush), spacing is not part of the brush mask
> > definition used by particle based brush engines usually, spray has
> > defined area with own spacing.
> 
> You can get the widget through the tree by checking its name (it needs to
> be named, of course) and then hide it. It's a bit hacky, we could also add
> api for that. But I'm fine with the hacky approach for now.

I think adding API should not be a problem here. Something like 
setShowSpacing(). I will try it.

> > This is interesting: it is possible to define the area of spray by pixel
> > map - brush footprint - brush mask: every pixel means a place for
> > particle and opacity of pixel is saying what is the probability that
> > particle will hit that place (0% to 100%). So spray would use the brush
> > dialog twice - as particle type [spacing hidden] and as brush size [with
> > spacing]. Something similar would be possible for hairy brush - so far
> > Brush Tip define bristles, and every bristle could be some Brush Tip
> > (now it is pixel or anti-aliased pixel)
> 
> Eeek... Would even a very advanced user understand that?

I don't know -- enki, do you understand this concept?

> > Just a note to why it is good to be able to hide spacing :)
> > 
> > Solution for the future:
> > QPainterPath based type KisBrush for circle, rectangle, random shape -
> > can be used by any brush engine as it is brush type footprint
> > 
> > I don't know what to do with Pixel and Anti-aliased pixel (Wu particle)
> > and QImage in the Spray shape dialog
> 
> Just keep them apart. Maybe in a separate tab that we can add to the brush
> tip option sheet?

Separate tab would mean that either the tab would be hidden for other brush 
engines like Pixel Brush or the pixel type have to be KisBrush - performance 
might be problem here :) or some new API like new enum to enumBrushType 
(Pixel, WuPixel) and add code for rendering it to the engines (again KisBrush, 
but simpler).