[PUSHED] fdo#64011 fix the position and the drawing from left to righ...

2013-05-21 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3912 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3912 To unsubscribe,

[PATCH] fdo#35320 A fix for Arabic justification problem caused by k...

2013-04-28 Thread Ahmad Harthi (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/3644 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/44/3644/1 fdo#35320 A fix for Arabic justification problem caused by kashidas Change-Id:

[PUSHED] fdo#35320 A fix for Arabic justification problem caused by k...

2013-04-28 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3644 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3644 To unsubscribe,

[PUSHED] more fix to fdo#62470

2013-04-14 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3305 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3305 To unsubscribe,

[PUSHED] fdo#62470 fix the input change when the table RTL

2013-04-01 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3133 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3133 To unsubscribe,

[PUSHED] fdo#62470 fix the table format dialog reading

2013-04-01 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3134 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3134 To unsubscribe,

[PUSHED] more fix for fdo#62399

2013-04-01 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/3145 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/3145 To unsubscribe,

[PUSHED] fix fdo#62050: for impress

2013-03-23 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2922 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2922 To unsubscribe,

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

2013-03-23 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2923 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2923 To unsubscribe,

[PUSHED] fix fdo#62050 : Initially disable RTL for comment window.

2013-03-20 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2831 Approvals: Ahmad Harthi: Verified; Looks good to me, approved abdulmajeed ahmed: Looks good to me, but someone else must approve -- To

[PUSHED] Resolves fdo#60554

2013-03-04 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/2375 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/2375 To unsubscribe,

[PUSHED] Fixes fdo#46186, The Table Remains in the defined writing mo...

2013-02-09 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/1964 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/1964 To unsubscribe,

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

2013-02-02 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/1961 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/1961 To unsubscribe,

Change in core[master]: Resolves the bug fdo#59117 missing borders of last column

2013-01-28 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/1834 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/1834 To unsubscribe,

Change in core[master]: More fixes to fdo#57422

2013-01-28 Thread Ahmad Harthi (via Code Review)
Hi, Thank you for your patch! It has been merged to LibreOffice. If you are interested in details, please visit https://gerrit.libreoffice.org/1521 Approvals: Ahmad Harthi: Verified; Looks good to me, approved -- To view, visit https://gerrit.libreoffice.org/1521 To unsubscribe,

[PATCH] More fixes to fdo#57422

2012-12-30 Thread Ahmad Harthi (via Code Review)
Hi, I have submitted a patch for review: https://gerrit.libreoffice.org/1521 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/21/1521/1 More fixes to fdo#57422 * Code cleanup + replacing drawText() with drawTextLayout(). This solves mirrored LTR