[Libreoffice-commits] core.git: sd/source

Armin Le Grand Armin.Le.Grand at cib.de
Thu May 17 18:06:32 UTC 2018


 sd/source/ui/slidesorter/view/SlsPageObjectLayouter.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 3a667101987ccbc75024942d779c8c62a4d0a296
Author: Armin Le Grand <Armin.Le.Grand at cib.de>
Date:   Thu May 17 19:58:17 2018 +0200

    Fixed PageObjectLayouter
    
    Someone removed maPageObjectSize in PageObjectLayouter
    and replaced it with the local var aPageObjectSize. This
    workms in principle, but was forgotten in one line where
    maFocusIndicatorBoundingBox gets set.
    Since the incoming rPageObjectWindowSize only has one
    value set (width or height) and the other one gets set
    in the call to CalculatePreviewBoundingBox, the whole
    layouting/visualization of SlideSorter and SlideView in
    Draw/Impress was broken.
    Fixed that - please be more careful with those changes
    in the future - take a look at the app where you do changes.
    
    Change-Id: I6bd9f8b11e7ebc7b880e465b80b813f5e2173c8c

diff --git a/sd/source/ui/slidesorter/view/SlsPageObjectLayouter.cxx b/sd/source/ui/slidesorter/view/SlsPageObjectLayouter.cxx
index 772506259d26..7b373be669dd 100644
--- a/sd/source/ui/slidesorter/view/SlsPageObjectLayouter.cxx
+++ b/sd/source/ui/slidesorter/view/SlsPageObjectLayouter.cxx
@@ -61,7 +61,7 @@ PageObjectLayouter::PageObjectLayouter (
         Size(rPageSize.Width(), rPageSize.Height()),
         aPageNumberAreaSize.Width(),
         nFocusIndicatorWidth);
-    maFocusIndicatorBoundingBox = ::tools::Rectangle(Point(0,0), rPageObjectWindowSize);
+    maFocusIndicatorBoundingBox = ::tools::Rectangle(Point(0,0), aPageObjectSize);
     maPageObjectBoundingBox = ::tools::Rectangle(
         Point(
             nFocusIndicatorWidth,


More information about the Libreoffice-commits mailing list