Issue 19430 - Rotated content in spreadsheet cells are cut off
Summary: Rotated content in spreadsheet cells are cut off
Status: CLOSED FIXED
Alias: None
Product: ui
Classification: Code
Component: code (show other issues)
Version: current
Hardware: PC Linux, all
: P3 Trivial (vote)
Target Milestone: OOo 2.2
Assignee: kla
QA Contact: issues@ui
URL:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2003-09-11 09:22 UTC by ru
Modified: 2007-02-14 15:22 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Output of JUAB with XAccessibleText logging enabled (3.29 KB, text/plain)
2003-09-11 09:32 UTC, nospam4obr 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description ru 2003-09-11 09:22:01 UTC 
Content that is rotated in a spreadsheet cell is truncated. Gnopernicus only
reports the part that is visible when the content would have alligned horizontally.
Comment 1 nospam4obr 2003-09-11 09:32:22 UTC 
Created attachment 9190 [details]
Output of JUAB with XAccessibleText logging enabled
Comment 2 nospam4obr 2003-09-11 09:35:06 UTC 
The code that calculates the text segment for getTextAtIndex() with
text type LINE seems not to take rotation into account (see log file).
Comment 3 thb 2003-09-12 19:22:13 UTC 
Sascha, please have a look. Rotated text is not an issue for
draw/impress, since we don't make text accessible in non-edit mode.
Comment 4 ooo 2003-09-29 10:54:36 UTC 
KA: I don't consider this a relevant task for OOo 2.0
Comment 5 ooo 2003-09-29 10:55:05 UTC 
.
Comment 6 malte_timmermann 2003-10-06 14:28:14 UTC 
We don't want to make too many changes for Accessibility in OOo 2.0,
but maybe this can be important if many spread sheets contain rotated
cells.
Other problem: UAA/JAA/ATK+ don't have information about rotated text,
so this can not become fixed correctly, only thing that we can fix is
the bounding rectangle, and only for 90/270°.

Rotation is not an information in the EditEngine model, it's only
passed to the drawing methods, so maybe the Accessibility
implementation currently doesn't have that information?
Comment 7 nospam4obr 2003-10-07 07:56:44 UTC 
Actually this is not about providing information about the rotation
itself. The point is, the text is cut off on XAccessibleText interface.

Stating from the log file:

[label] Cell A1: getTextAtIndex(LINE,0) returns (0,12,Abcdefghijkl)
[label] Cell A1: getTextAtIndex(WORD,0) returns
(0,23,Abcdefghijklmnopqrstuvw)

So somewhere in the office code, the wrong coordinates are used to
calculate the LINE bounderies.
Comment 8 thb 2003-10-08 19:57:52 UTC 
Yeah. Of course, for line calculations, formatting has to be
performed, which is done by the EditEngine in this case. If the
EditEngine has no clue about the rotation, the result does not
surprise me. But before we invest time and energy into this bug, maybe
we should discuss what is needed here. Is it only the correct part of
the visible content, or should rotated text in general work? If the
latter, IMHO lots more is lacking, first of all the fact that bounding
boxes cannot really be rotated (and therefore, character bounds will
grossly overlap for multiples of pi/4).
Comment 9 ru 2003-10-15 17:29:40 UTC 
As discussed with TZ target goes to OOo2.0.
Comment 10 frank 2003-10-20 15:26:11 UTC 
removed SO9 Flag according to latest comment from RU.
Comment 11 sascha.ballach 2003-10-28 15:07:46 UTC 
I take it.
Comment 12 frank 2004-05-04 15:51:14 UTC 
Due to limited time ressources I re-Target this one to OOo Later
Comment 13 frank 2004-05-04 15:51:41 UTC 
Due to limited time ressources I re-Target this one to OOo Later
Comment 14 mdxonefour 2005-09-07 10:17:14 UTC 
re-targeted to OOo 3.0
Comment 15 mdxonefour 2005-09-29 11:04:16 UTC 
re-targeted to OOo 3.0
Comment 16 niklas.nebel 2005-09-30 18:10:21 UTC 
reassigning
Comment 17 daniel.rentz 2005-10-04 12:33:07 UTC 
started
Comment 18 mdxonefour 2006-10-16 11:15:04 UTC 
MD: re-targeting to 2.2 since it's an important a11y issue
Comment 19 pavel 2007-01-15 19:19:44 UTC 
dr: can you please update the status of this issue?

Can you make it into 2.2?
Comment 20 daniel.rentz 2007-01-18 11:54:58 UTC 
this issue depends on http://bugzilla.gnome.org/buglist.cgi?query=306902
Comment 21 daniel.rentz 2007-01-24 08:56:24 UTC 
fixed in CWS src680/dr53
Comment 22 daniel.rentz 2007-01-24 10:01:02 UTC 
back to QA
Comment 23 kla 2007-01-24 14:20:58 UTC 
seen ok into CWS dr53
Comment 24 kla 2007-02-14 15:22:05 UTC 
Seen ok into the master -> closed