[PATCH] more fix for fdo#62399

Faisal al-otaibi (via Code Review) gerrit at gerrit.libreoffice.org
Sun Mar 31 23:54:09 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3145

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/45/3145/1

more fix for fdo#62399

this will fix the border position, when the borders from the same
side are different in cells format dialog, when the sheet RTL.

Change-Id: I9fa8a665dfa0b0dcb5230bafa1e3537f450afb9c
---
M sc/source/ui/view/tabvwsha.cxx
1 file changed, 6 insertions(+), 5 deletions(-)



diff --git a/sc/source/ui/view/tabvwsha.cxx b/sc/source/ui/view/tabvwsha.cxx
index 43a04a4..20b3d39 100644
--- a/sc/source/ui/view/tabvwsha.cxx
+++ b/sc/source/ui/view/tabvwsha.cxx
@@ -485,15 +485,16 @@
     //Fix border incorrect for RTL fdo#62399
     if( pDoc->IsLayoutRTL( GetViewData()->GetTabNo() ) )
     {
-        SvxBoxItem aNewFrame( aLineOuter );
+        SvxBoxItem     aNewFrame( aLineOuter );
+        SvxBoxInfoItem aTempInfo( aLineInner );
+
         if ( aLineInner.IsValid(VALID_LEFT) )
             aNewFrame.SetLine( aLineOuter.GetLeft(), BOX_LINE_RIGHT );
         if ( aLineInner.IsValid(VALID_RIGHT) )
             aNewFrame.SetLine( aLineOuter.GetRight(), BOX_LINE_LEFT );
-        if ( aLineInner.IsValid(VALID_TOP) )
-            aNewFrame.SetLine( aLineOuter.GetTop(), BOX_LINE_TOP );
-        if ( aLineInner.IsValid(VALID_BOTTOM) )
-            aNewFrame.SetLine( aLineOuter.GetBottom() , BOX_LINE_BOTTOM );
+
+        aLineInner.SetValid( VALID_LEFT, aTempInfo.IsValid(VALID_RIGHT));
+        aLineInner.SetValid( VALID_RIGHT, aTempInfo.IsValid(VALID_LEFT));
 
         pOldSet->Put( aNewFrame );
     }

-- 
To view, visit https://gerrit.libreoffice.org/3145
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9fa8a665dfa0b0dcb5230bafa1e3537f450afb9c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Faisal al-otaibi <fmalotaibi at kacst.edu.sa>



More information about the LibreOffice mailing list