Issue 107965 - Selection behaviour broken with tables that contain spaces on Solaris/OpenSolaris
Summary: Selection behaviour broken with tables that contain spaces on Solaris/OpenSol...
Status: CLOSED IRREPRODUCIBLE
Alias: None
Product: Impress
Classification: Application
Component: ui (show other issues)
Version: OOo 1.0.0
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.2
Assignee: clippka
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks: 99999
  Show dependency tree
 
Reported: 2009-12-28 12:37 UTC by clippka
Modified: 2010-01-04 14:09 UTC (History)
3 users (show)

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

Attachments
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 clippka 2009-12-28 12:37:19 UTC 
There is a bug with embedded tables in presentations on Solaris/OpenSolaris.

How to recreate:
Create a new empty presentation.
Insert -> Table
  choose 1 row and 1 column
Insert the following text into the single cell:
 abcde
i.e. a the string abcde beginning with a space.

Effect:
- the area above the table will be considered as part of the table
  i.e. when the table is deselected and you click into the area   the 4-arrow
cursor will appear and the table will be selected
  The side effect is that other frames in that upper area will be non-
  selectable
- when you change the page format and increase the width e.g.
  from 11.69" to 12" the table will disappear from the slide
- when you change back the format (Edit->Undo) the table will
  be placed at the bottom of the slide
Comment 1 clippka 2009-12-28 12:41:48 UTC 
The behaviour is the same that is already fixed by issue 105856. But this case
looks like a problem with boundary rectangles for spaces which are not reported
as empty on solaris only.
Comment 2 clippka 2009-12-28 15:11:32 UTC 
fixed in vcl/source/gdi/outdev3.cxx, Committed revision 277836.
Comment 3 clippka 2010-01-04 12:52:20 UTC 
after discussion with hdu I reject this patch
Comment 4 clippka 2010-01-04 13:01:38 UTC 
cl->aw: changing the base functionality of OutputDevice::GetTextBoundRect seems
to risky for me after discussion with hdu. Removing the position may break
functionality at other places. Hdu suggest the better solution would be to check
for a rectangle with an empty size at the place where you actually call
GetTextBoundRect. This would also avoid the risk of regressions in other text
layouting features.

I will remove the previous fix from cws impress185. feel free to use this cws
for your new fix
Comment 5 Armin Le Grand 2010-01-04 14:06:14 UTC 
AW->CL: I disagree here. The problem is that ONLY on SOLARIS we get that
Rechtangle with Size (0,0), all other systems already give an Empty rectangle
(where ::IsEmpty() says TRUE). This implies that there are no places to break,
these would have been broken on Solaris all the time.
This task is about system-independent behaviour in the system-independent part
of VCL. The error is the system-dependent behaviour for the Solaris implementation.
Comment 6 wolframgarten 2010-01-04 14:09:26 UTC 
Reproducible with 3.1.1 but not with OOO320_m8 anymore.
Comment 7 wolframgarten 2010-01-04 14:09:56 UTC 
Closed.