[PATCH] fix fdo#40496: don't reset RES_FRAMEDIR attribute.

navin patidar (via Code Review) gerrit at gerrit.libreoffice.org
Wed Apr 3 02:03:35 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3182

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/82/3182/1

fix fdo#40496: don't reset  RES_FRAMEDIR attribute.

fix regression introduced by bb920862c02047471f0ac5e78358e8d8fd9b93e7 commit

discussed here :
https://bugs.freedesktop.org/show_bug.cgi?id=40496#c13

choosing correct default alignment for RTL text need some discussion.

Change-Id: I2aec9391d076af6d5fbf00ba5bdad030ac0286ed
---
M sw/source/ui/shells/textsh1.cxx
1 file changed, 1 insertion(+), 2 deletions(-)



diff --git a/sw/source/ui/shells/textsh1.cxx b/sw/source/ui/shells/textsh1.cxx
index 395c7d8..e92cde6 100644
--- a/sw/source/ui/shells/textsh1.cxx
+++ b/sw/source/ui/shells/textsh1.cxx
@@ -499,9 +499,8 @@
                     aAttribs.insert( aAttribs.end(), nE++ );
                 pUShorts += 2;
             }
-            // we don't want to change writing direction and text alignment.
+            // we don't want to change writing direction.
             aAttribs.erase( RES_FRAMEDIR );
-            aAttribs.erase( RES_PARATR_ADJUST );
             rWrtSh.ResetAttr( aAttribs );
             rReq.Done();
             break;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2aec9391d076af6d5fbf00ba5bdad030ac0286ed
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: navin patidar <patidar at kacst.edu.sa>



More information about the LibreOffice mailing list