You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@poi.apache.org by bu...@apache.org on 2006/05/08 18:42:24 UTC

DO NOT REPLY [Bug 39395] - PATCH: Placeholder shape and other improvements

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=39395>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=39395


nick@torchbox.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO




------- Additional Comments From nick@torchbox.com  2006-05-08 16:42 -------
Cheers for this, I've applied it. Couple of things though:

Do you think have a nice helper for RGB for the text colour (and presumably also
background colour, shape colour etc), splitting into the tripplets, or should we
just stick with using the java.awt one?

Looking at TextBox, I'm not happy with how much of RichTextRun it seems to be
duplicating. Can we just provide a RichTextRun over the top of it? Otherwise, we
should refactor both of them to remove the duplication.

Also, why did you change TextCharsAtom to no longer use the common StringUtil
stuff? I didn't commit that, I didn't see the point!

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
Mailing List:    http://jakarta.apache.org/site/mail2.html#poi
The Apache Jakarta POI Project: http://jakarta.apache.org/poi/