Font character spacing settings (GlyphExtraSpacing)

[bagobor]

You are using hardcoded character spacing constant:
const int character_spacing_x = 1;
and then, when building font glyphs:
glyph.XAdvance = (signed short)(pc.xadvance + character_spacing_x);

That makes text blocks looks too spare.
So please move it to config 👍

[ocornut]

Out of curiosity with what sort of font 1 isn't a good value?

I can move that number to the font atlas settings but it'll still be baked into glyphs (so as to maximize the speed of text size calculation which is a bottleneck for big, unclipped interfaces). So if there is a call to change it that call will have to go through glyphs.

[bagobor]

for me 0 looks much better with default embeded font. win32/opengl
(but still suck with lack of hinting. )

[ocornut]

The AA branch now support this with the new ImFontConfig* settings.
All the AddFontxxx() functions now have a ImFontConfig* pointer. If you pass one you can alter settings.

Incidentally you were correct about the spacing. The extra +1 spacing was more of a bug on my side that intentional, so the AA branch now include the default font with that +1 spacing.

Alter with:

ImFontConfig font_cfg;
font_cfg.GlyphExtraSpacing.x = 2.0f
io.AddFontFromFileTTF("...tf", 20.0f, &font_cfg);

[ocornut]

This is now merged in master along with the entire new AA-branch. Note that it is a rather big update and you'll need to update your render function to use indexed vertex rendering.

[bagobor]

Nice work!