supertuxkart-sourceforge-migration / stk-migration-test2

0 stars 0 forks source link

Get rid of CBillboardTextSceneNode #680

Open supertuxkart-sourceforge-migration opened 10 years ago

supertuxkart-sourceforge-migration commented 10 years ago

Author: kroartem

Stk r[11430] With Russian language enabled something strange happens with the word "Leader" shown above the first kart. Here are some thoughts from IRC:

<redskull> it seems to happen with every language not using latin alphabet
<Arthur_D> the headline font, currently in used to signify the leader, was hand-created by Auria and thus does not have many characters. So in many cases, we've used supplementary fonts, which IMO is ugly and we should get a proper headline font with enough characters

Migrated-From: https://sourceforge.net/apps/trac/supertuxkart/ticket/680

supertuxkart-sourceforge-migration commented 10 years ago

Author: wardje I think the issue is deeper than just this. After looking around the interface while in Russian for a bit, there's just too much font inconsistency.

What is needed:

Every font used for the Latin alphabet should also have the letters from different character sets (Cyrillic, hebrew, whatever other language). Currently it seems they use different font(styles) which then looks terrible because it's bound to get mixed up together (%s , and then %s becomes a player name in the Latin alphabet using that font).

supertuxkart-sourceforge-migration commented 10 years ago

Author: auria Screenshot : http://i.imgur.com/EWMwq.jpg

I think the problem may be that billboard text nodes don't use our font objects, which supports fallback fonts and all kinds of fancy stuff?

supertuxkart-sourceforge-migration commented 10 years ago

Author: auria I confirm that the string does contain russian characters. the title font is used so it should be fine in theory, maybe our font does not work for billboard nodes

supertuxkart-sourceforge-migration commented 10 years ago

Author: auria Somewhat hackish fix in r11431

supertuxkart-sourceforge-migration commented 10 years ago

Author: auria Changing this ticket into 'Get rid of CBillboardTextSceneNode'

Looking at the code of 'CBillboardTextSceneNode', it's just too tightly coupled with irrlicht's font object and it doesn't look like it will be easy to make it work with our font object, and it's already amazing it hasn't caused crashes

Proposed solution is to render text to a texture using RTT

supertuxkart-sourceforge-migration commented 10 years ago

Author: hikerstk Note that the original ticket is #263, which shows the look we are aiming for.

supertuxkart-sourceforge-migration commented 10 years ago

Author: auria The core issue was worked around in r11431, so I'm pushing the rest of this ticket back to later

supertuxkart-sourceforge-migration commented 10 years ago

Author: auria Current state acceptable