UI Mirroring Demo: ported to Godot 4.1

[Alex2782]

v4.2.dev5.official [e3e2528ba]

Screen-2023-10-06-012623.mp4

[Calinou]

Tested locally on 4.2.beta1. I can't see Japanese characters where they should appear in the translation demo:

This also applies to the UI mirroring demo (a label should appear below the focused dropdown with "Japanese (fa)"):

These issues also occur when testing with 4.1.2.

[Calinou]

I suggest checking what #930 does.

[Alex2782]

If I close Godot and open the project again then everything is ok, Japanese is also displayed.
v4.1.2.stable.official [399c9dc39]

[Calinou]

I still can't get it to work on my end, even after removing .import/ and .godot/ while the editor is closed 🙁
Note that I'm on Linux, so font loading behavior may differ compared to macOS.

PS: I noticed your PR adds two new files in ui_mirroring/fonts, noto_font.res and noto_font_arabic.res. Why are these files required when the existing TTF fonts could be reused?

[bruvzg]

Fonts are clearly not ported correctly:

I would suggest doing something like this (from the clean state before any changes):

• Delete all font *.tres files (droid_sans.tres, noto_font.tres).
• Reimport all font from the editor import tab (probably optional).
• Set fallbacks in the main font import settings and reimport it.

• Open the scene, and in the "Fix Dependencies" dialog, select the main font instead of missing font *.tres.

Also, do we need to ship all the font with the demos? For the Mirroring and Translation demos system font fallback should be fine, neither require specific font dimensions to work.

[Alex2782]

• Reimport all font from the editor import tab (probably optional).

I had opened Project Settings -> Import Defaults -> Importer "Font Data (Dynamic Font)" and under "Fallbacks" inserted and saved the 2 font files, after that I could not continue (Crash / Godot open Project Manager). Was that correct?

The file 'ui_mirroring.tscn' is now 7.3 MB. If I add font for Japanese, then the scene will be even bigger.

@smix8 says: #955 (comment)

If we rework and update the demo maybe we should save the NavigationPolygon resource to an external file as a binary .res file?

Right now it is stored embedded as plain text inside the navigation.tscn file, something that we should not teach to new users as it will bloat their scene files and make them super slow to load/save/update on large and complex navigation meshes.

@bruvzg does this PR work for you with binary 'res'-Fonts? Or is Japanese not displayed for you either?
The label for HELLO_TEXT has normal English font, but still Arabic is displayed correctly? @Calinou

I suggest checking what #930 does.
Add PO support in addition to CSV.

I think there is little point in trying to fix problems I can't reproduce.

[bruvzg]

does this PR work for you with binary 'res'-Fonts?

There should not be any binary (or text) font resources at all, font files should be used directly.

[bruvzg]

Something like this - bruvzg@0401f28

[Alex2782]

ok I'll check later, from some tutorials I know only first convert the fonts to "tres", also already with Godot 3.2

[Alex2782]

Also, do we need to ship all the font with the demos? For the Mirroring and Translation demos system font fallback should be fine, neither require specific font dimensions to work.

now without 'fonts', this looks and works now like with 'fonts' (on MacOS)

Should I remove HELLO_TEXT? does not work for @Calinou with a CSV file

[Calinou]

Something like this - bruvzg@0401f28

I've tested that commit on 4.2.beta2 but I still can't see the CJK text (even after removing .godot/ and reopening the editor to force a reimport of everything):

The UI mirroring demo update works well though.