[BUG] #widthOfString: in addHorzScrollbar
Doug Way
dway at mailcan.com
Fri Mar 12 20:03:20 UTC 2004
Avi Bryant wrote:
> If a PluggableListMorph is given a styled item with the
> addHorzScrollbar changeset loaded, the Text object will eventually get
> given to AbstractFont>>widthOfString:, which doesn't know how to deal
> with it. I'm not sure who is at fault here (should #widthOfString: be
> able to deal with Text? Should it make sure to convert its argument
> to a string first? Should the PLM be converting its items to strings
> before asking about their widths? Should my code not be giving a Text
> item to a PLM?).
>
> Full debug log is attached.
For what it's worth, I ran into the same problem. If we could get this
resolved, it'd be great to get the addHorzScrollbar changeset
incorporated into 3.7. Other than this problem, it looked pretty good.
Offhand, it seems reasonable that AbstractFont should know how to
compute the width of some styled text. (Converting to a String ahead of
time wouldn't be ideal because bold text is often wider than non-bold
text, for example.) The only problem then is that the method should be
renamed to widthOfStringOrText:, or something like that.
- Doug
More information about the Squeak-dev
mailing list
|