Graphite not fully working in Electronite

[birchamp]

The simple grahite test works but the complex scripts don't.
See: #6723 (comment)

[birchamp]

DoD: SIL graphite test page renders properly with Electronite.

[da1nerd]

Here's another example that works. #6723 (comment)

The Padauk font is not rendering correctly.

[da1nerd]

@jag3773 do you remember if we were able to render the Padauk font correctly when we first created Electronite? I though we had, but now I'm wondering if we only tested the Awami Nastaliq font.

[da1nerd]

Link to the graphite repo https://github.com/silnrsi/graphite

[da1nerd]

I reached out for some help on the graphite repo silnrsi/graphite#63.

NOTE: I confirmed graphite is not fully functional on linux or windows. I'm guessing others tried out macOS as well.

[da1nerd]

Are we missing something in /third_party/graphite/BUILD.gn?

[da1nerd]

I discovered we had been using a broken version of graphite silnrsi/graphite@4fb12b3.

Rebuilding with a patched versions now. Hopefully that solve the problem.

Update: It did not fix the problem.

[da1nerd]

Figured it out thanks to Martin.

The padauk font is loaded using the following css on the demo page.

.padauk_ttf {
  font-family: PadaukT, sans-serif;     
  font-size: 150%;
  -moz-font-feature-settings: "wtri=1";
  -moz-font-feature-settings: "wtri" 1;
}

Since electronite is chromium, the -moz tags don't work.
inserting font-feature-settings: "wtri" 1; fixed the display of the Padauk font.

[da1nerd]

Confirmed https://github.com/mannycolon/electronite-poc-app works on:

• Pop! OS 64bit (linux)
• Windows 10 64bit (virtual machine)
• macOS High Sierra 64bit
• Windows 32

[mannycolon]

Can confirm that after making the suggested CSS tweak the Padauk rendered correctly in my machine. However, the Arabic (Awami Nastaliq font) still isn't working for me.

Padauk Font

Awami Nastaliq Font

[da1nerd]

For posterity, we've confirmed that electronite can now pass all the tests on the SIL test page.

[cckozie]

electronite-poc-app
As Manny's comment referenced above, the Paduak font is being displayed correctly in the POC. The Awami Nastaliq Font is not. So it seems to me that this issue is not ready for QA.
Am I wrong or missing something?

[da1nerd]

@cckozie that is incorrect. The awami font is working correctly. The text is simply not wrapping at the same points because the browser is not the same width as when the SIL team took their screenshot.
Also, the line height is different. Neither of those are related to font rendering.

[birchamp]

@cckozie Please connect with @neutrinog to reproduce the steps to get it working.

On my machine I'm seeing what Joel is seeing

[cckozie]

Taking Birch's word that this is working as is should be. If problems still exist they will be written up in tC.