[zoom] fix text width - #3003

I just tweaked that a little, I think it's good now (despite failing CI build due to osuosl.org being to slow...).

@ben-wes can you confirm that it fixes #3002? thx!

@ben-wes can you confirm that it fixes #3002? thx!

thank you - this certainly fixes the wrapping issue for me!

there's one more minor issue now though that might be fixed here as well: pdlua sets the line width to 0 for shapes without border. the following line will add a border on zoom though:

do you think it would be valid to add a check for 0 < $newwidth here?

yes sure! Thanks for finding that. In fact I think the whole line is useless.

OTOH I just realized that my "fix text width" solution is actually buggy, because Tk stores the width value as integer...
You can see it on the pdlua "graphics" demo, if you zoom out to 6% then re-zoom in, back and forth, widths eventually get damaged.

I'll need to add a new tag I'm afraid.

Hope it's OK now.
@ben-wes plz test!

test!

thanks - lgtm!

Cool. @umlaeute maybe this can be merged to develop?

There was another bug, only visible with horizontal slider at moderate zoom out, here at 89%:

The solution is to round the width to nearest integer. Last commit implements that.

BTW I fixed a rounding bug with default_zoom when is less than 100%.
Sorry for being off-topic, but I discovered this when trying to configure default zoom to 89%, to help fixing the hslider bug...