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

List:       kde-kimageshop
Subject:    Re: Junior Job (Bug 245658)
From:       Dmitry Kazakov <dimula73 () gmail ! com>
Date:       2014-02-25 4:32:17
Message-ID: CAEkBSfW3j0e8moJ38Kmh5Q7MjfPrnEy9hY88Us9Y6PRb-LYr2A () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hi, Koushik!

If I understood you right, you already have a KisColorSelectorComponent in
the pop-up box in the instance of 'm_triangleColorSelector', so yes, you
can access this method.

Talking about preview of the currently selected color, you might want to
look at the code in KisColorPreviewPopup class, which is in
kis_color_selector_base.cpp file. Probably, you would be able to share the
code between those two classes.



On Mon, Feb 24, 2014 at 10:07 PM, Koushik S <skoushik333@gmail.com> wrote:

> Hello,
>
>     I was working on and off on Bug 245658 (JJ: Ability to see currently
> selected color in a pop-up palette), and went through the code in
> kis_popup_palette (thanks to boud for directing me to it!).
>
>     I tried to add code to get the required feature, but to no avail. In
> fact, the code actually replicated the already built-in feature of
> displaying the recently used color.
>
>     I examined the code which updates the color selector, and have a vague
> idea of solving this. Would it be correct if I initialize a
> KisColorSelectorComponent and use it's currentColor() method to get the
> currently selected color? If not, could someone point me in the right
> direction. Thanks.
>
>
> --
> regards,
> Koushik. S
>
> _______________________________________________
> Krita mailing list
> kimageshop@kde.org
> https://mail.kde.org/mailman/listinfo/kimageshop
>
>


-- 
Dmitry Kazakov

[Attachment #5 (text/html)]

<div dir="ltr"><div><div>Hi, Koushik!<br><br></div>If I understood you right, you \
already have a KisColorSelectorComponent in the pop-up box in the instance of \
&#39;m_triangleColorSelector&#39;, so yes, you can access this method.<br> \
<br></div>Talking about preview of the currently selected color, you might want to \
look at the code in KisColorPreviewPopup class, which is in \
kis_color_selector_base.cpp file. Probably, you would be able to share the code \
between those two classes.<br> <br></div><div class="gmail_extra"><br><br><div \
class="gmail_quote">On Mon, Feb 24, 2014 at 10:07 PM, Koushik S <span \
dir="ltr">&lt;<a href="mailto:skoushik333@gmail.com" \
target="_blank">skoushik333@gmail.com</a>&gt;</span> wrote:<br> <blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><div dir="ltr">Hello,<div><br></div><div>      I was working \
on and off on  Bug 245658 (JJ:  Ability to see currently selected color in a pop-up \
palette), and went through the code in kis_popup_palette (thanks to boud for \
directing me to it!).  </div>

<div><br></div><div>      I tried to add code to get the required feature, but to no \
avail. In fact, the code actually replicated the already built-in feature of \
displaying the recently used color.</div><div><br></div><div>

      I examined the code which updates the color selector, and have a vague idea of \
solving this. Would it be correct if I initialize a KisColorSelectorComponent and use \
it&#39;s currentColor() method to get the currently selected color? If not, could \
someone point me in the right direction. Thanks.</div> <span class="HOEnZb"><font \
color="#888888"> <div><br><div><br></div>-- <br><div \
dir="ltr"><div>regards,</div>Koushik. S</div> </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><br clear="all"><br>-- <br>Dmitry Kazakov </div>



_______________________________________________
Krita mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop


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

Configure | About | News | Add a list | Sponsored by KoreLogic