Nabble - Squeak - Balloon

Re: [Balloon] [Bug+Fix] in BalloonCanvas>>drawString: ...

2006-11-27T14:33:37Z

Hi,

I added drawing of a red rectangle, to show that there is something that
can not be drawn
in the second change set. It would be better to actually fix this
method, but I think, for that we now have Rome...

Jens Lincke schrieb:

> here is the changeset
>
> Andreas Raab schrieb:
>> Can you email me the patch?
>>
>> Thanks,
>> - Andreas
>>
>> Jens Lincke wrote:
>>> Hi,
>>>
>>> I found a bug in the BalloonCanvas when drawing a String with an
>>> offset transformation
>>>
>>> the method
>>> drawString: s from: firstIndex to: lastIndex in: boundsRect font:
>>> fontOrNil color: c
>>> was marked TODO and did only draw when it can fall back to FormCanvas
>>>
>>> But the offset in the Transformation was ignored, so I changed it into:
>>>
>>> drawString: s from: firstIndex to: lastIndex in: boundsRect font:
>>> fontOrNil color: c
>>> (self ifNoTransformWithIn: boundsRect)
>>> ifTrue:[ | offset |
>>> offset := transform a13 @ transform a23.
>>>
>>> ^super drawString: s from: firstIndex to: lastIndex in:
>>> boundsRect + offset font: fontOrNil color: c]
>>>
>>>
>>> and uploaded a iSqueak3.8 package Balloon-jl.4 to the
>>> source.impara.de/Stuff
>>>
>>> - Jens Lincke -
>>>
>>> _______________________________________________
>>> Balloon mailing list
>>> Balloon@...
>>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon
>>>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Balloon mailing list
> Balloon@...
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon
>

_______________________________________________
Balloon mailing list
Balloon@...
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon

Balloon-Fix-drawString-offset.2.cs.gz (570 bytes) Download Attachment

Re: [Balloon] [Bug+Fix] in BalloonCanvas>>drawString: ...

2006-11-27T13:49:21Z

here is the changeset

Andreas Raab schrieb:

> Can you email me the patch?
>
> Thanks,
> - Andreas
>
> Jens Lincke wrote:
>> Hi,
>>
>> I found a bug in the BalloonCanvas when drawing a String with an
>> offset transformation
>>
>> the method
>> drawString: s from: firstIndex to: lastIndex in: boundsRect font:
>> fontOrNil color: c
>> was marked TODO and did only draw when it can fall back to FormCanvas
>>
>> But the offset in the Transformation was ignored, so I changed it into:
>>
>> drawString: s from: firstIndex to: lastIndex in: boundsRect font:
>> fontOrNil color: c
>> (self ifNoTransformWithIn: boundsRect)
>> ifTrue:[ | offset |
>> offset := transform a13 @ transform a23.
>>
>> ^super drawString: s from: firstIndex to: lastIndex in:
>> boundsRect + offset font: fontOrNil color: c]
>>
>>
>> and uploaded a iSqueak3.8 package Balloon-jl.4 to the
>> source.impara.de/Stuff
>>
>> - Jens Lincke -
>>
>> _______________________________________________
>> Balloon mailing list
>> Balloon@...
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon
>>

_______________________________________________
Balloon mailing list
Balloon@...
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon

Balloon-Fix-drawString-offset.1.cs.gz (422 bytes) Download Attachment

Re: [Balloon] [Bug+Fix] in BalloonCanvas>>drawString: ...

2006-11-27T13:11:39Z

Can you email me the patch?

Thanks,
- Andreas

Jens Lincke wrote:

> Hi,
>
> I found a bug in the BalloonCanvas when drawing a String with an offset
> transformation
>
> the method
> drawString: s from: firstIndex to: lastIndex in: boundsRect font:
> fontOrNil color: c
> was marked TODO and did only draw when it can fall back to FormCanvas
>
> But the offset in the Transformation was ignored, so I changed it into:
>
> drawString: s from: firstIndex to: lastIndex in: boundsRect font:
> fontOrNil color: c
> (self ifNoTransformWithIn: boundsRect)
> ifTrue:[ | offset |
> offset := transform a13 @ transform a23.
>
> ^super drawString: s from: firstIndex to: lastIndex in: boundsRect
> + offset font: fontOrNil color: c]
>
>
> and uploaded a iSqueak3.8 package Balloon-jl.4 to the
> source.impara.de/Stuff
>
> - Jens Lincke -
>
> _______________________________________________
> Balloon mailing list
> Balloon@...
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon
>

_______________________________________________
Balloon mailing list
Balloon@...
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon

[Balloon] [Bug+Fix] in BalloonCanvas>>drawString: ...

2006-11-27T11:20:05Z

Hi,

I found a bug in the BalloonCanvas when drawing a String with an offset
transformation

the method
drawString: s from: firstIndex to: lastIndex in: boundsRect font:
fontOrNil color: c
was marked TODO and did only draw when it can fall back to FormCanvas

But the offset in the Transformation was ignored, so I changed it into:

drawString: s from: firstIndex to: lastIndex in: boundsRect font:
fontOrNil color: c
(self ifNoTransformWithIn: boundsRect)
ifTrue:[ | offset |
offset := transform a13 @ transform a23.

^super drawString: s from: firstIndex to: lastIndex in:
boundsRect + offset font: fontOrNil color: c]

and uploaded a iSqueak3.8 package Balloon-jl.4 to the
source.impara.de/Stuff

- Jens Lincke -

_______________________________________________
Balloon mailing list
Balloon@...
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/balloon