Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 210337

Summary:

TextFlow doesn't take border fragments into account

Product:

[Tools] GEF

Reporter:

Pratik Shah <ppshah>

Component:

GEF-Legacy Draw2d

Assignee:

Pratik Shah <ppshah>

Status:

RESOLVED FIXED

QA Contact:

Severity:

major

Priority:

Version:

3.3.2

Target Milestone:

3.4.0 (Ganymede) M5

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Patch for TextFlow	none

Description Pratik Shah

2007-11-20 00:14:23 EST

If you set a flow border on a TextFlow, it adds an extra fragment on each end.  The offset for these fragments is -1 by default, which is not valid.  TextFlow does not always account for them and throws exceptions or returns invalid values.   getBidiSubstring() and getPreviousVisibleOffset() are two examples, but this bug is present in several methods in this class.

Comment 1 Pratik Shah

2008-01-14 16:20:48 EST

Created attachment 86872 [details]
Patch for TextFlow

Comment 2 Pratik Shah

2008-01-14 16:24:08 EST

The patch was delivered to head.  Marking as fixed.