#17872 closed bug (fixed)

Appearance font selector text wrapping misbehaves

Reported by: waddlesplash Owned by: jscipione
Priority: normal Milestone: R1/beta4
Component: Preferences/Appearance Version: R1/beta3
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

At least on larger font sizes (e.g. 18pt), starting Appearance has the text wrapped at only about 2/3 of the actual width of the text box. This seems unnecessarily wasteful and does not look very appealing.

Attachments (2)

Appearance.png (62.0 KB ) - added by waddlesplash 21 months ago.
screenshot1.png (40.1 KB ) - added by Starcrasher 21 months ago.
font preview readability

Download all attachments as: .zip

Change History (8)

comment:1 by jscipione, 21 months ago

There’s probably an adjustment that could be made here but that’s how wrapping works, it wraps the whole word. Do we have justified text support? Screenshot please.

by waddlesplash, 21 months ago

Attachment: Appearance.png added

comment:2 by waddlesplash, 21 months ago

There's clearly space for the remaining words on the same line, it doesn't need to wrap in this case.

comment:3 by jscipione, 21 months ago

Ok that’s not what I was thinking it was, there’s enough room there but it’s clearly messed up. The initial text width calculation is done in a different way because the text wrapping isn’t know at that point. The wrapping should fix itself if you bump the font size up and down but I’m not sure why the initial width calculation is not working in this case.

comment:4 by waddlesplash, 21 months ago

Yet another BTextView rect calculationg bug?

comment:5 by Starcrasher, 21 months ago

While at it, use document background colour for the preview background or panel text colour for the text preview. Otherwise we may have readability problems.

by Starcrasher, 21 months ago

Attachment: screenshot1.png added

font preview readability

comment:6 by waddlesplash, 20 months ago

Milestone: UnscheduledR1/beta4
Resolution: fixed
Status: assignedclosed

Fixed in hrev56434.

Note: See TracTickets for help on using tickets.