Bug 210340 - [Text] Expose Draw2d text API
Summary: [Text] Expose Draw2d text API
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy GEF (MVC) (show other bugs)
Version: 3.3.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-20 00:26 EST by Pratik Shah CLA
Modified: 2013-10-17 09:49 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pratik Shah CLA 2007-11-20 00:26:45 EST
I am making extensive use of the new text API in GEF, which hides quite a bit of the API.  I'm opening this bug to maintain a list of API that I think needs to be exposed.  Here are a few cases:

1) FlowBox#getLineRoot() should be made public.  In fact, all the package-protected methods there should be made protected, if not public.

2) CaretInfo's constructor should be exposed.  It's already marked as internal.

3) FlowAdapter's private fields should be made protected so that it can be subclassed as needed.
Comment 1 Pratik Shah CLA 2007-11-23 13:00:43 EST
4) FlowUtilities and its non-private methods
Comment 2 Randy Hudson CLA 2007-12-20 10:58:07 EST
> 1) FlowBox#getLineRoot() should be made public.  In fact, all the
> package-protected methods there should be made protected, if not public.

The fact that line roots persist after the layout could be something that is optimized away later. What is the data that is needed, and maybe it could be promoted to the TextFragment.
Comment 3 Anthony Hunter CLA 2008-02-08 15:32:51 EST
Missed M5, so moving to M6
Comment 4 Randy Hudson CLA 2008-03-27 13:04:22 EDT
Pratik, we haven't had a chance to look at superscript, subscript, or other line variations yet so it would be hard to nail down the API already. We have a workaround for 3.4 so I think this should be deferred until we have more experience with underlined superscripted text, etc.
Comment 5 Anthony Hunter CLA 2008-03-31 11:21:31 EDT
(In reply to comment #4)
> [...] this should be deferred [...]
Comment 6 Alexander Nyßen CLA 2013-10-17 09:49:46 EDT
Assigning back to gef-inbox (and state to new), as specified assignee is no current GEF committer.