<html>
    <head>
      <base href="https://bugs.documentfoundation.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - button flashing - mouse wheel zooming breaks"
   href="https://bugs.documentfoundation.org/show_bug.cgi?id=121963#c49">Comment # 49</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - button flashing - mouse wheel zooming breaks"
   href="https://bugs.documentfoundation.org/show_bug.cgi?id=121963">bug 121963</a>
              from <span class="vcard"><a class="email" href="mailto:Armin.Le.Grand@me.com" title="Armin Le Grand <Armin.Le.Grand@me.com>"> <span class="fn">Armin Le Grand</span></a>
</span></b>
        <pre>The embedding to TransformPrimitive2D is indeed created by
ViewObjectContact::getPrimitive2DSequence and taking
supportsGridOffsets/getGridOffset into account - what is completely corect and
core element of the change.
What seems wrong OTOH is more that in
ViewObjectContactOfUnoControl_Impl::positionAndZoomControl the GridOffst is
*not* taken into account -> need to add that.
Made a 1st rough try in adding to _rViewTransformation, but this would add it
double in the non-EndRepaint stuff. Thus the logically correct thing to do is
probably to not use

                const tools::Rectangle aRect( pUnoObject->GetLogicRect() );

in ViewObjectContactOfUnoControl_Impl::positionAndZoomControl, but to get the
B2DRange from the SeqOfPrim which will then correctly include the offset...</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>