Junior Job (Bug 245658)

[Boudewijn Rempt]

I've pushed this version of the patch in any case, since it's a really nice addition!

On Monday 24 March 2014 Mar 23:28:51 Koushik S wrote:
> Hi,
> 
>     I was working on and off on this bug, and here are the developments on
> this so far. I have been able to create a separate area for the FG colour
> to be displayed.
> 
>     Here's what I've come up with so far: http://i.imgur.com/UqyjdZl.png
> 
>     Any feedbacks are welcomed.
> 
> 
> On Sat, Mar 15, 2014 at 7:22 PM, Koushik S <skoushik333 at gmail.com> wrote:
> 
> >
> > On Sat, Mar 15, 2014 at 4:57 PM, Sven Langkamp <sven.langkamp at gmail.com>wrote:
> >
> >> It's hard to tell what you are trying to do. Can you attach the code and
> >> a screenshot?
> >
> >
> > Sure. Here's the code snippet I added in the paintEvent.
> >
> > <code>
> > //painting foreground color
> >     QPainterPath path2;
> >     path2.addEllipse(QPoint(180,180), colorInnerRadius-25,
> > colorInnerRadius - 25);
> >     painter.fillPath(path2, m_triangleColorSelector->color());
> >     painter.drawPath(path2);
> > </code>
> >
> > I've also added in some modifications to the masks. The size of the masks
> > and also the path are arbitrary and are just used for testing.
> >
> > <code>
> >     int side = qMin(width()/2, height()/2);
> >     int side2 = qMin(width()/2, height()/2);
> >     QRegion maskedRegion (width()/2 - side/2, height()/2 - side/2, side,
> > side, QRegion::Ellipse);
> >     QRegion maskedRegion2 (180 - side2/2, 180 - side2/2 , side2, side2,
> > QRegion::Ellipse);
> >     setMask(maskedRegion + maskedRegion2);
> > </code>
> >
> > These wouldn't work since, the QRect which houses only the popup palette.
> > I couldn't figure out the part of the code which changes the QRect size.
> >
> > But, as I've been going through it, changing the size in the sizeHint()
> > method seems to change the overall size of the widget i.e., it enlarges it.
> > So, I probably should change most of the code which calculates the relative
> > positions of the favourite brushes and so on. Is it the right way to do it?
> >
> > --
> > regards,
> > Koushik. S
> >
> 
> 
> 
> 

-- 
Boudewijn Rempt
http://www.valdyas.org, http://www.krita.org, http://www.boudewijnrempt.nl