Issue 107965

Summary:	Selection behaviour broken with tables that contain spaces on Solaris/OpenSolaris
Product:	Impress	Reporter:	clippka
Component:	ui	Assignee:	clippka
Status:	CLOSED IRREPRODUCIBLE	QA Contact:	issues@graphics <issues>
Severity:	Trivial
Priority:	P3	CC:	hdu, issues, mdxonefour
Version:	OOo 1.0.0
Target Milestone:	OOo 3.2
Hardware:	All
OS:	All
Issue Type:	DEFECT	Latest Confirmation in:	---
Developer Difficulty:	---
Issue Depends on:
Issue Blocks:	99999

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.