<div dir="ltr">Hi, <div><br></div><div>I knew you were sick, hope you are better now. I've noticed that you applied as a mentor for the project, thanks!<br><br>As for the button, I was thinking about some kind of enable button in the Tools Option Docker.<br>I was talking to Dmitry about whether to extend to Ellipse Tool or Ellipse Assistant Tool, he suggested that I extend the Ellipse Assistant Tool. Actually the ellipse in the Ellipse Assistant Tool is transformable contrary to the ellipse in the Ellipse Tool which is not transformable. Also, I went through some of the code in <span style="color:rgb(0,0,0);font-family:Arial;white-space:pre-wrap">EllipseAssista</span><span style="color:rgb(0,0,0);font-family:Arial;white-space:pre-wrap">nt.cc and it looked like some of the functions, particularly related to transformation, have already been implemented there. </span></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 19, 2022 at 3:43 PM Halla Rempt <<a href="mailto:halla@valdyas.org">halla@valdyas.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
Sorry, I've been pretty sick, so I'm now trying to slowly catch up... I've applied as a mentor for your project, so that's settled.<br>
<br>
On woensdag 12 januari 2022 12:28:41 CET Srirupa Datta wrote:<br>
> Hey all,<br>
> <br>
> Since Dmitry is on vacation, I am writing here because I needed to clarify<br>
> a few things. I wanted to add support for Perspective Ellipse in Krita as a<br>
> part of Season of KDE.<br>
> The link for the feature request: Circle in a Square<br>
> <<a href="https://bugs.kde.org/show_bug.cgi?id=405643" rel="noreferrer" target="_blank">https://bugs.kde.org/show_bug.cgi?id=405643</a>><br>
> <br>
> Now, I wanted to clarify if this is roughly what I would be doing.<br>
> <br>
> <br>
> 1. Create a seperate button on Tools Option Docker for Ellipse Tool.<br>
> <br>
<br>
Ah, we need to be careful here: do you mean a new button in the toolbox, or a toggle in the option widget?<br>
<br>
> 2. When the perspective ellipse button is clicked, a circle in a square<br>
> figure appears on the canvas, the corners of the squares being adjustable.<br>
> The circle in a square can be moved to the desired location and corners can<br>
> be adjusted. 3. Now when the corners are adjusted, I need to use the<br>
> KisPerspectiveTransformStrategy::continuePrimaryAction() function to get<br>
> the transformation matrix from the four points and use that to transform<br>
> the circle to the corresponding ellipse.<br>
<br>
Yes.<br>
<br>
> 4. As Halla suggested, this transformation should not transform the pixels,<br>
> so I probably need to look into what other methods are there for<br>
> transforming.<br>
<br>
Yes, the stroke should be drawn using the current brush following the (qpainter)path created by the user when manipulating the perspective box. <br>
<br>
> 5. As for the mathematical part, I can use the transformation matrix on the<br>
> old circle/ellipse to convert it to the new ellipse, but the KisAlgebra2d::<br>
> transformEllipse() function uses the semimajor and semiminor axes lengths,<br>
> so would it possible to reuse that function by finding the major and minor<br>
> axes?<br>
<br>
I don't know -- maths is not my forte.<br>
<br>
> <br>
> 6. Also, just to be clear, I am extending the Ellipse Tool, not creating<br>
> another Assistant Tool(there is one Ellipse option in the Assistant Tool as<br>
> well), right?<br>
<br>
Yes, that is correct.<br>
<br>
> <br>
> <br>
> I have created a draft proposal which was reviewed by Halla earlier. The<br>
> details have been mentioned there. Link: Draft Proposal<br>
> <<a href="https://docs.google.com/document/d/1Bgi0og-YMkyU8rmVr9RuXyzi6f4YLretYk-HFSkG-d4/edit?usp=sharing" rel="noreferrer" target="_blank">https://docs.google.com/document/d/1Bgi0og-YMkyU8rmVr9RuXyzi6f4YLretYk-HFSkG-d4/edit?usp=sharing</a>><br>
> <br>
> <br>
> With regards,<br>
> <br>
> Srirupa Datta<br>
> <br>
> IRC: sriru:<a href="http://matrix.org" rel="noreferrer" target="_blank">matrix.org</a><br>
> <br>
<br>
<br>
<br>
<br>
</blockquote></div>