--===============0589288674==
Content-Type: multipart/alternative; boundary=001485f772a2cc7239047936f817

--001485f772a2cc7239047936f817
Content-Type: text/plain; charset=UTF-8

>
> > paintDevice stores selection of the mask instead of the result image. Is
> it
> > expected behavior?
> Yes, that is what gimp and photoshop show too. And with boudewijn we
> concluded
> that it is what makes most sense. If we want to show a preview of the
> applied
> result, we could show it in the tooltip.
>

Ok, thanks. I was a bit of confused because of copy-paste naming =)

Maybe:
s/originalDevice/selection



>
> > Could you explain why is thumbnail created from paintDevice() instead of
> > original()?
> Isn't the original of a mask null ? Because my first attempt was to use
> "original()" (actually moving the thumbnail code from KisLayer to
> KisBaseNode), since I expected the data of the mask to be in "original()",
> but
> I got a null "original()" for the transparency mask.
>

Sorry, i got remembered that KisMask does not inherit KisLayer too late to
write =)

For Boud:
Boud, could you revert this misleading part of r1030899 and move original()
and projection() back to KisLayer as pure virtual methods? I've been writing
to you about that already... original() and projection() is nothing for
masks so they shouldn't be present in KisNode and return bogus values...




-- 
Dmitry Kazakov

--001485f772a2cc7239047936f817
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"borde=
r-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-le=
ft: 1ex;"><div class=3D"im">
&gt; paintDevice stores selection of the mask instead of the result image. =
Is it<br>
&gt; expected behavior?<br>
</div>Yes, that is what gimp and photoshop show too. And with boudewijn we =
concluded<br>
that it is what makes most sense. If we want to show a preview of the appli=
ed<br>
result, we could show it in the tooltip.<br></blockquote><div><br>Ok, thank=
s. I was a bit of confused because of copy-paste naming =3D)<br><br>Maybe:<=
br>s/originalDevice/selection<br><br>=C2=A0</div><blockquote class=3D"gmail=
_quote" style=3D"border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt=
 0pt 0.8ex; padding-left: 1ex;">

<div class=3D"im"><br>
&gt; Could you explain why is thumbnail created from paintDevice() instead =
of<br>
&gt; original()?<br>
</div>Isn&#39;t the original of a mask null ? Because my first attempt was =
to use<br>
&quot;original()&quot; (actually moving the thumbnail code from KisLayer to=
<br>
KisBaseNode), since I expected the data of the mask to be in &quot;original=
()&quot;, but<br>
I got a null &quot;original()&quot; for the transparency mask.<font color=
=3D"#888888"><br>
</font></blockquote><div><br>Sorry, i got remembered that KisMask does not =
inherit KisLayer too late to write =3D)<br><br>For Boud:<br>Boud, could you=
 revert this misleading part of r1030899 and move original() and projection=
() back to KisLayer as pure virtual methods? I&#39;ve been writing to you a=
bout that already... original() and projection() is nothing for masks so th=
ey shouldn&#39;t be present in KisNode and return bogus values...<br>
<br></div></div><br><br clear=3D"all"><br>-- <br>Dmitry Kazakov<br>

--001485f772a2cc7239047936f817--

--===============0589288674==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

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

--===============0589288674==--