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

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1846940

Title:
  [upstream] Loop in libreoffice-calc when scrolling to top of
  spreadsheet

To manage notifications about this bug go to:
https://bugs.launchpad.net/df-libreoffice/+bug/1846940/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to