[PATCH] fdo#47305: set digit language, before OutputDevice draw text...

2013-05-04 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3773 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/73/3773/1 fdo#47305: set digit language, before OutputDevice draw text. Set ctl language as digit language when wrt

[PATCH] fix fdo#64085 : set proper alignment for justified align tex...

2013-05-01 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3704 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/04/3704/1 fix fdo#64085 : set proper alignment for justified align text. According to cell writing direction, set

[ABANDONED] make use of pOutWin, which is member of ImpEditView class.

2013-04-28 Thread navin patidar (via Code Review)
navin patidar has abandoned this change. Change subject: make use of pOutWin, which is member of ImpEditView class. .. Patch Set 2: Abandoned dynamic cast produces pOutWin only when pTargetDevice is NULL for

[PATCH] make use of pOutWin, which is member of ImpEditView class.

2013-04-24 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3592 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/92/3592/1 make use of pOutWin, which is member of ImpEditView class. Change-Id: I78218572abe06f189eb20ba341c4de3aced8df61

[PATCH] related to fdo#63546.

2013-04-22 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3549 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/49/3549/1 related to fdo#63546. set appropriate alignment when wrt direction of selected cells is changed. Change-Id:

[PATCH] fix fdo#63546 : set appropriate alignment when writing direc...

2013-04-17 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3424 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/24/3424/1 fix fdo#63546 : set appropriate alignment when writing direction is changed. Change-Id:

[PATCH] use ImpEditEngine::ImplInitDigitModel() to set digit languag...

2013-04-15 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3389 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/89/3389/1 use ImpEditEngine::ImplInitDigitModel() to set digit language. Change-Id:

[PATCH] fix fdo#60534 : use DefaultCharMapper::mapChar() to map RTL ...

2013-04-07 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3221 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/21/3221/1 fix fdo#60534 : use DefaultCharMapper::mapChar() to map RTL string unicodes. Use DefaultCharMapper::mapChar()

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

2013-04-03 Thread navin patidar (via Code Review)
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

[PATCH] remove unused code.

2013-04-02 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3167 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/67/3167/1 remove unused code. A request (rReq) having anyone of following SID SID_ATTR_CHAR_FONTHEIGHT,

[PATCH] fix 40496: don't reset RES_PARATR_ADJUST RES_FRAMEDIR attr...

2013-04-01 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3146 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/46/3146/1 fix 40496: don't reset RES_PARATR_ADJUST RES_FRAMEDIR attributes. Change-Id:

[PATCH] fix fdo#62050: for impress

2013-03-23 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2922 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/22/2922/1 fix fdo#62050: for impress Impress is also affected by fdo#62050. In RTL UI, comment window is RTL and

[PATCH] In writer’s RTL UI show author name and date at right side o...

2013-03-23 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2923 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/23/2923/1 In writer’s RTL UI show author name and date at right side of comment window Change-Id:

[PATCH] fix fdo#62143: set TEXT_LAYOUT_BIDI_STRONG flag.

2013-03-11 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2664 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/64/2664/1 fix fdo#62143: set TEXT_LAYOUT_BIDI_STRONG flag. Change-Id: I19ed1874ff73c3a8002934fa3456d718e383c5d6 --- M

[PATCH] fix fdo#61492: update sd:View with FONTHEIGHT attribute chan...

2013-02-26 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2433 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/33/2433/1 fix fdo#61492: update sd:View with FONTHEIGHT attribute changes. update sd:View with FONTHEIGHT attribute

[PATCH] fix fdo#60533, Set TEXT_LAYOUT_BIDI_STRONG flag.

2013-02-11 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/2076 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/76/2076/1 fix fdo#60533, Set TEXT_LAYOUT_BIDI_STRONG flag. Stop ImplLayoutArgs::ImplLayoutArgs() to perform ubidi algo on

[PATCH] fix logic to get L/R direction

2013-02-03 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/1972 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/72/1972/1 fix logic to get L/R direction Change-Id: I481f9ea57f8c5b71eb849df3f71c75f8b9b0f1fa --- M

[PATCH] fix fdo#59892 completely and fdo#33302 partially.

2013-02-02 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/1961 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/61/1961/1 fix fdo#59892 completely and fdo#33302 partially. fix fdo#33302 for rtl scripts in cal,impress and draw.

resolves:fdo#54629 vcl:gdi add recursive check

2013-01-25 Thread navin patidar (via Code Review)
navin patidar has abandoned this change. Change subject: resolves:fdo#54629 vcl:gdi add recursive check .. Patch Set 3: Abandoned I agree using MultiSalLayout::GetNextGlyphs is right way to fix this bug . I didn't remove

[PATCH] resolves:fdo#54629 vcl:gdi add recursive check

2013-01-12 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/1662 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/62/1662/1 resolves:fdo#54629 vcl:gdi add recursive check check all font fallback levels if rRect.IsEmpty() is true for

Change public variables of class ImplDevFontAttributes to pr...

2013-01-08 Thread navin patidar (via Code Review)
navin patidar has abandoned this change. Change subject: Change public variables of class ImplDevFontAttributes to private. .. Patch Set 2: Abandoned -- To view, visit https://gerrit.libreoffice.org/1565 To unsubscribe,

[PATCH] Change public variables of class ImplDevFontAttributes to pr...

2013-01-07 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/1565 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/65/1565/1 Change public variables of class ImplDevFontAttributes to private. Change-Id:

[PATCH] Change public variables of class ImplFontAttributes to priva...

2013-01-05 Thread navin patidar (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/1551 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/51/1551/1 Change public variables of class ImplFontAttributes to private. Change-Id: