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

[Mike Kaganski (via logerrit)]

 sc/source/ui/app/inputwin.cxx |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

New commits:
commit 68a8467723f69f5ab61d2bee5e83984c5584af41
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Mon Dec 2 12:39:56 2019 +0200
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Mon Dec 2 14:04:08 2019 +0100

    tdf#129141: paint all when in multiline mode
    
    ... otherwise it assumes that only part of first string to the right of the
    unchanged string part needs redrawing (which would be to the right of the
    control itself)
    
    Change-Id: I2833c1eb2026041b11d0b72431b164d2ee3befef
    Reviewed-on: https://gerrit.libreoffice.org/84207
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/sc/source/ui/app/inputwin.cxx b/sc/source/ui/app/inputwin.cxx
index dae95abdd797..b9ddf6219081 100644
--- a/sc/source/ui/app/inputwin.cxx
+++ b/sc/source/ui/app/inputwin.cxx
@@ -1796,10 +1796,8 @@ void ScTextWnd::SetTextString( const OUString& rNewString )
         // Find position of the change, only paint the rest
         if (!mpEditEngine)
         {
-            bool bPaintAll;
-            if ( bIsRTL )
-                bPaintAll = true;
-            else
+            bool bPaintAll = mnLines > 1 || bIsRTL;
+            if (!bPaintAll)
             {
                 //  test if CTL script type is involved
                 SvtScriptType nOldScript = SvtScriptType::NONE;
@@ -1817,7 +1815,7 @@ void ScTextWnd::SetTextString( const OUString& rNewString )
 
             if ( bPaintAll )
             {
-                // if CTL is involved, the whole text has to be redrawn
+                // In multiline mode, or if CTL is involved, the whole text has to be redrawn
                 Invalidate();
             }
             else