experimental patch: allow alt text to wrap
Just tried this, and I like the results quite a bit. - "simple" insofar as it doesn't bother with the letter-spacing and font-variant code. - it's a separate path from drawText() because this many-argument version of fl_draw() in fl_draw.cxx is rather a long function so that it can deal with wrapping and images and symbols and all kinds of alignment in general. - if the wrapped text fits into the box, we do some unnecessary clipping. Would it be worth the trouble to add an interface to get fl_measure() information?
On Tue, Jul 19, 2011 at 09:39:32PM +0000, corvid wrote:
Just tried this, and I like the results quite a bit.
After some testing, it looks like a worthy addition. Sites w/o images become more "readable" with it. BTW, using SHADING_DARK for drawSimpleWrappedText() helps too.
- "simple" insofar as it doesn't bother with the letter-spacing and font-variant code.
- it's a separate path from drawText() because this many-argument version of fl_draw() in fl_draw.cxx is rather a long function so that it can deal with wrapping and images and symbols and all kinds of alignment in general.
- if the wrapped text fits into the box, we do some unnecessary clipping. Would it be worth the trouble to add an interface to get fl_measure() information?
It doesn't look worth the effort. If you see no further problems, please commit. -- Cheers Jorge.-
On Wed, Jul 27, 2011 at 07:51:53PM -0400, Jorge Arellano Cid wrote:
On Tue, Jul 19, 2011 at 09:39:32PM +0000, corvid wrote:
Just tried this, and I like the results quite a bit.
After some testing, it looks like a worthy addition. Sites w/o images become more "readable" with it.
I also like it. I would prefer to use our own methods for word wrapping (i.e. Textblock), but couldn't come up with anything really working until now, so I'm happy with the drawSimpleWrappedText(). We could still change it in the future. Cheers, Johannes
participants (3)
-
corvid＠lavabit.com -
jcid＠dillo.org -
johannes.hofmann＠gmx.de