<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>