<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>looks great <br><br>I would welcome this<br>I assume more colorse will be added interactively? is this possible ? Or will it be better to have a fixed RMB panel with empty orbs?<br><br><br><br><br>if I could suggest that this design could also work with this<br><br><a href="https://bugs.kde.org/show_bug.cgi?id=328482" target="_blank">https://bugs.kde.org/show_bug.cgi?id=328482</a><br><br>or something similar to complement that color history design.<br><br>Like the "settings orb" you have,this could be a permanent button that triggers that function.<br><br>maybe it just toggles to show the available RMB palettes by replacing color history orbs and showing the saved RMB palettes in their stead.<br><br><br>Ofcourse there could be a totally different solution for this as well,just though it sounded kinda similar to my idea,so I wanted to mention it.<br><br><br><br><br><div><hr id="stopSpelling">Date: Tue, 25 Mar 2014 13:23:12 +0100<br>From: animtim@gmail.com<br>To: kimageshop@kde.org<br>Subject: Re: Junior Job (Bug 245658)<br><br>
  
    
  
  
    <div class="ecxmoz-cite-prefix">Nice work,<br>
      And very good idea to expand this to color history. Also the
      config button can open the "select favorite preset" window, with
      maybe more options to further customize it like color selector
      type..<br>
      <br>
      +1 <br>
      <br>
      Le 25/03/2014 13:10, David Revoy a écrit :<br>
    </div>
    <blockquote cite="mid:CAPX1LSAshsUQ1tYjPxTrEupNwjs97CQzYxm8ODM5iDcjE6rqwQ@mail.gmail.com">
      <div dir="ltr">
        <div>
          <div>
            <div>
              <div>Really nice this way.<br>
              </div>
              I think it's done :-)<br>
              <br>
            </div>
            This new 'petal' design around the pop-up made me think of a
            design idea. <br>
          </div>
          To artist/user on mailing-list please give a review to this
          ideas mockup : <a href="http://i.imgur.com/46cYJTL.png" target="_blank">http://i.imgur.com/46cYJTL.png</a><br>
        </div>
        i don't know if you would find this attractive.<br>
        <div><br>
          <div>
            <div>
              <div><br>
              </div>
            </div>
          </div>
        </div>
      </div>
      <div class="ecxgmail_extra"><br clear="all">
        <div><font size="1"><span style="color:rgb(192,192,192);">_____________________</span><br>
            <a href="http://www.davidrevoy.com" target="_blank">http://www.davidrevoy.com</a></font><br>
        </div>
        <br>
        <br>
        <div class="ecxgmail_quote">On Tue, Mar 25, 2014 at 12:37 PM,
          Koushik S <span dir="ltr"><<a href="mailto:skoushik333@gmail.com" target="_blank">skoushik333@gmail.com</a>></span>
          wrote:<br>
          <blockquote class="ecxgmail_quote" style="border-left:1px #ccc solid;padding-left:1ex;">
            <div dir="ltr">
              <div>
                <div>
                  <div>Thanks for the feedback, David.<br>
                    <br>
                  </div>
                  Here's the updated look : <a href="http://i.imgur.com/84Q2n60.png" target="_blank">http://i.imgur.com/84Q2n60.png</a><br>
                  <br>
                </div>
                I'm not exactly sure if it was the right way to do it. I
                just decreased the color area, while keeping the mask
                region the same, which seems to be the same way the
                border around the palette is generated. <br>
              </div>
            </div>
            <div class="ecxgmail_extra">
              <div>
                <div class="h5"><br>
                  <br>
                  <div class="ecxgmail_quote">On Tue, Mar 25, 2014 at 2:21
                    PM, David Revoy <span dir="ltr"><<a href="mailto:davidrevoy@gmail.com" target="_blank">davidrevoy@gmail.com</a>></span>
                    wrote:<br>
                    <blockquote class="ecxgmail_quote" style="border-left:1px #ccc solid;padding-left:1ex;">
                      <div dir="ltr">
                        <div>Hi, it does look good to me. :-)<br>
                        </div>
                        ( only crit : maybe it needs also a black
                        outline around, as the main widget got one too )<br>
                      </div>
                      <div class="ecxgmail_extra"><br clear="all">
                        <div><font size="1"><span style="color:rgb(192,192,192);">_____________________</span><br>
                            <a href="http://www.davidrevoy.com" target="_blank">http://www.davidrevoy.com</a></font><br>
                        </div>
                        <br>
                        <br>
                        <div class="ecxgmail_quote">
                          <div>
                            <div>On Mon, Mar 24, 2014 at 6:58 PM,
                              Koushik S <span dir="ltr"><<a href="mailto:skoushik333@gmail.com" target="_blank">skoushik333@gmail.com</a>></span>
                              wrote:<br>
                            </div>
                          </div>
                          <blockquote class="ecxgmail_quote" style="border-left:1px #ccc solid;padding-left:1ex;">
                            <div>
                              <div>
                                <div dir="ltr">
                                  <div>
                                    <div>Hi,<br>
                                      <br>
                                    </div>
                                        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. <br>
                                    <br>
                                        Here's what I've come up with so
                                    far: <a href="http://i.imgur.com/UqyjdZl.png" target="_blank">http://i.imgur.com/UqyjdZl.png</a><br>
                                    <br>
                                  </div>
                                      Any feedbacks are welcomed. <br>
                                </div>
                                <div class="ecxgmail_extra">
                                  <div>
                                    <div><br>
                                      <br>
                                      <div class="ecxgmail_quote">On Sat,
                                        Mar 15, 2014 at 7:22 PM, Koushik
                                        S <span dir="ltr"><<a href="mailto:skoushik333@gmail.com" target="_blank">skoushik333@gmail.com</a>></span>
                                        wrote:<br>
                                        <blockquote class="ecxgmail_quote" style="border-left:1px #ccc solid;padding-left:1ex;">
                                          <div dir="ltr">
                                            <div>
                                              <div class="ecxgmail_extra"><br>
                                                <div class="ecxgmail_quote">On
                                                  Sat, Mar 15, 2014 at
                                                  4:57 PM, Sven Langkamp
                                                  <span dir="ltr"><<a href="mailto:sven.langkamp@gmail.com" target="_blank">sven.langkamp@gmail.com</a>></span>
                                                  wrote:<br>
                                                  <blockquote class="ecxgmail_quote" style="border-left:1px solid rgb(204,204,204);padding-left:1ex;">It's
                                                    hard to tell what
                                                    you are trying to
                                                    do. Can you attach
                                                    the code and a
                                                    screenshot?</blockquote>
                                                </div>
                                                <br>
                                              </div>
                                            </div>
                                            <div class="ecxgmail_extra">Sure.
                                              Here's the code snippet I
                                              added in the paintEvent.<br>
                                              <br>
                                            </div>
                                            <div class="ecxgmail_extra"><code><br>
                                              //painting foreground
                                              color<br>
                                                  QPainterPath path2;<br>
                                                 
                                              path2.addEllipse(QPoint(180,180),
                                              colorInnerRadius-25,
                                              colorInnerRadius - 25);<br>
                                                 
                                              painter.fillPath(path2,
                                              m_triangleColorSelector->color());<br>
                                                 
                                              painter.drawPath(path2);<br>
                                            </div>
                                            <div class="ecxgmail_extra"></code><br>
                                            </div>
                                            <div class="ecxgmail_extra"><br clear="all">
                                            </div>
                                            <div class="ecxgmail_extra">
                                              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. <br>
                                              <br>
                                            </div>
                                            <div class="ecxgmail_extra"><code><br>
                                            </div>
                                            <div class="ecxgmail_extra">
                                                  int side =
                                              qMin(width()/2,
                                              height()/2);<br>
                                            </div>
                                            <div class="ecxgmail_extra">   
                                              int side2 =
                                              qMin(width()/2,
                                              height()/2);<br>
                                                  QRegion maskedRegion
                                              (width()/2 - side/2,
                                              height()/2 - side/2, side,
                                              side, QRegion::Ellipse);<br>
                                                  QRegion maskedRegion2
                                              (180 - side2/2, 180 -
                                              side2/2 , side2, side2,
                                              QRegion::Ellipse);<br>
                                                  setMask(maskedRegion +
                                              maskedRegion2);<br>
                                            </div>
                                            <div class="ecxgmail_extra"></code><br>
                                            </div>
                                            <div class="ecxgmail_extra"><br>
                                            </div>
                                            <div class="ecxgmail_extra">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. <br>
                                              <br>
                                            </div>
                                            <div class="ecxgmail_extra">
                                              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? <br>
                                              <span><font color="#888888">
                                                </font></span></div>
                                            <span><font color="#888888">
                                                <div class="ecxgmail_extra"><br>
                                                  -- <br>
                                                  <div dir="ltr">
                                                    <div>regards,</div>
                                                    Koushik. S</div>
                                                </div>
                                              </font></span></div>
                                        </blockquote>
                                      </div>
                                      <br>
                                      <br clear="all">
                                      <br>
                                    </div>
                                  </div>
                                  <span><font color="#888888">-- <br>
                                      <div dir="ltr">
                                        <div>regards,</div>
                                        Koushik. S</div>
                                    </font></span></div>
                                <br>
                              </div>
                            </div>
                            <div>_______________________________________________<br>
                              Krita mailing list<br>
                              <a href="mailto:kimageshop@kde.org" target="_blank">kimageshop@kde.org</a><br>
                              <a href="https://mail.kde.org/mailman/listinfo/kimageshop" target="_blank">https://mail.kde.org/mailman/listinfo/kimageshop</a><br>
                              <br>
                            </div>
                          </blockquote>
                        </div>
                        <br>
                      </div>
                      <br>
                      _______________________________________________<br>
                      Krita mailing list<br>
                      <a href="mailto:kimageshop@kde.org" target="_blank">kimageshop@kde.org</a><br>
                      <a href="https://mail.kde.org/mailman/listinfo/kimageshop" target="_blank">https://mail.kde.org/mailman/listinfo/kimageshop</a><br>
                      <br>
                    </blockquote>
                  </div>
                  <br>
                  <br clear="all">
                  <br>
                </div>
              </div>
              <span class="ecxHOEnZb"><font color="#888888">-- <br>
                  <div dir="ltr">
                    <div>regards,</div>
                    Koushik. S</div>
                </font></span></div>
            <br>
            _______________________________________________<br>
            Krita mailing list<br>
            <a href="mailto:kimageshop@kde.org">kimageshop@kde.org</a><br>
            <a href="https://mail.kde.org/mailman/listinfo/kimageshop" target="_blank">https://mail.kde.org/mailman/listinfo/kimageshop</a><br>
            <br>
          </blockquote>
        </div>
        <br>
      </div>
      <br>
      <fieldset class="ecxmimeAttachmentHeader"></fieldset>
      <br>
      <pre>_______________________________________________
Krita mailing list
<a class="ecxmoz-txt-link-abbreviated" href="mailto:kimageshop@kde.org">kimageshop@kde.org</a>
<a class="ecxmoz-txt-link-freetext" href="https://mail.kde.org/mailman/listinfo/kimageshop" target="_blank">https://mail.kde.org/mailman/listinfo/kimageshop</a>
</pre>
    </blockquote>
    <br>
  

<br>_______________________________________________
Krita mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop</div>                                      </div></body>
</html>