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

Luboš Luňák l.lunak at collabora.com
Fri Aug 8 07:44:59 PDT 2014


 sw/source/core/objectpositioning/tocntntanchoredobjectposition.cxx |    6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

New commits:
commit b4f775daf031d7d28505825ed0e7b4e1d19f0e1b
Author: Luboš Luňák <l.lunak at collabora.com>
Date:   Fri Aug 8 16:39:24 2014 +0200

    fix image dragging inside a document
    
    Testcase is a document with a table and a wrap-through anchored-to-character
    image below it. Dragging the image up to be anchored to a place in a cell
    and positioned partially on top of the table does not actually change
    the image position if the image was below the table before the drag,
    but changes when the table was above. In the faulty case,
    SwToCntntAnchoredObjectPosition::CalcPosition() returns prematurely
    because of paragraph portion information not existing at that point
    (interestingly, the problem is also fixed simply by commenting out the return).
    
    This basically reverts 9b87346bf4a1d32abfa781aa66d4d5b627133edf . I'm unable
    to check the original problem as the bugreport references a web page that
    no longer exists. So if that one still exists, it'll need to be handled
    whenever somebody runs into a way of reproducing it.
    
    Change-Id: Id354237dace3179b94768146ec1a599cc3d49146

diff --git a/sw/source/core/objectpositioning/tocntntanchoredobjectposition.cxx b/sw/source/core/objectpositioning/tocntntanchoredobjectposition.cxx
index 96643ba..be59ead 100644
--- a/sw/source/core/objectpositioning/tocntntanchoredobjectposition.cxx
+++ b/sw/source/core/objectpositioning/tocntntanchoredobjectposition.cxx
@@ -204,11 +204,7 @@ void SwToCntntAnchoredObjectPosition::CalcPosition()
                    !GetAnchoredObj().GetLastCharRect().Width() ) ||
                  !GetAnchoredObj().GetLastTopOfLine() )
             {
-                // #i111886#
-                // Check existence of paragraph portion information in order
-                // to avoid formatting which could cause deletion of follow frames.
-                GetAnchoredObj().CheckCharRectAndTopOfLine();
-
+                GetAnchoredObj().CheckCharRectAndTopOfLine( false );
                 // Due to table break algorithm the character
                 // rectangle can have no height. Thus, check also the width
                 if ( ( !GetAnchoredObj().GetLastCharRect().Height() &&


More information about the Libreoffice-commits mailing list