<div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div bgcolor="#ffffff" text="#000000"><div class="im"><blockquote type="cite">

      <div class="gmail_quote">
        <div>Well, if some tests fail, it means that some other
          subsystem may fail because of changed guarantees. I think you
          need to write what coordinate systems you have now and how
          your QTransform objects reflect relations between these
          coordinate systems.</div>
      </div>
    </blockquote></div>
    OK i maybe was a bit rash.<br>
    I reverted all changes and applied the commits again in another
    arrangement and I changed a few things.<br>
    Now the tests really do pass, yay :D.</div></blockquote><div><br>That is good. Actually, the tests were my biggest concern. So now you can merge freely =) And the renaming of that QTransform has made the system more clear.<br>
</div><div class="im"><br>Btw, could you take a look into the bug <a href="https://bugs.kde.org/show_bug.cgi?id=265528">https://bugs.kde.org/show_bug.cgi?id=265528</a> ? There is some problem with autoscrolling in KoCanvasControllerWidget. I think it gets non-rotated coordinates. This is not a regression from your patch, so you can do it after the merge, if you want =)<br>
 <br></div></div><br>-- <br>Dmitry Kazakov<br>