After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 317662 - Cell borders not printed in place on RTL sheets
Cell borders not printed in place on RTL sheets
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: Printing
git master
Other All
: High normal
: ---
Assigned To: Andreas J. Guelzow
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2005-10-01 02:30 UTC by Ido Ben-Barak
Modified: 2006-10-14 01:59 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Screenshot of Print Preview in RTL mode (15.68 KB, image/png)
2005-11-15 14:22 UTC, Phil Bull 		Details

Description Ido Ben-Barak 2005-10-01 02:30:33 UTC 
Distribution/Version: Ubuntu 5.10

1. Create a new sheet and set it RTL.
2. Insert text into a cell.
3. Add borders to the cell from all directions (right click->Format
Cells...->Border).
4. Request a print preview or print the page.
5. Zoom in as much as you can and notice that the right and left borders of the
cell are a little to the right of where they should be. This doesn't happen in
LTR sheets.
Comment 1 Phil Bull 2005-11-15 14:21:42 UTC 
Can reproduce this. Screenshot attached.

Gnumeric 1.5.90 on Ubuntu Breezy (gnumeric.1.5.90-1ubuntu2)
Comment 2 Phil Bull 2005-11-15 14:22:36 UTC 
Created attachment 54781 [details]
Screenshot of Print Preview in RTL mode
Comment 3 Jody Goldberg 2006-02-23 05:42:08 UTC 
There's a twisty maze of +/- offsets in the border code.  Couple that with some magic to align postscript style lines (centered on the coord) with display lines (pixel aligned) and we end up with a mess that will take some detailed thought to unravel for RTL.
Comment 4 Jody Goldberg 2006-10-14 01:59:07 UTC 
Ahh, I see the problem.

print_vline has as += .5 to round to pixel position and does not take rtl into account.

Patch will be in 1.7.2