Zuu wrote:

Usage of font size modifiers was not considered when the StoryBook was implemented. All content will take a multiplier of the normal font size in height. One solution would be to try to block usage of font size modifiers.

This comment was imported from FlySpray: https://bugs.openttd.org/task/5617#comment12347

Rubidium wrote:

I think that the story page entries should get line wrapped as well. If that gets done, then the {BIG_FONT} support is essentially done as well.

This comment was imported from FlySpray: https://bugs.openttd.org/task/5617#comment12407

Zuu wrote:

From what I've understood the code that draw multi-line text, it do not work well with text that uses more than one font-size within the text block. The same goes for the computation of how many pixels of height that a a multi-line text needs.

Last time I had a look at this was before the new text layout changes. Thus it may have changed since then.

That being said, with line wrapping it is possible to support a paragraph consisting of just large, medium or small text. And when/if the text backend support multi-line text with varying text size, the story book will automatically also support that.

This comment was imported from FlySpray: https://bugs.openttd.org/task/5617#comment12522

Zuu wrote:

A fix that involves correctly calculating the content height and line warping has been produced.

Attachments

• 66-fix-5617-big_font.patch (2.16 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/5617#comment12607

Alberth wrote:

+ int rest = height % line_height;
+ return height / line_height + (rest > 0 ? 1 : 0);
can also be written as

return (height + line_height - 1) / line_height;

This comment was imported from FlySpray: https://bugs.openttd.org/task/5617#comment12608

Zuu closed the ticket.

Reason for closing: Fixed

In r25768

This comment was imported from FlySpray: https://bugs.openttd.org/task/5617