Summary: | [Win] Material icons font aren't shown because ligature doesn't work | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Fujii Hironori <Hironori.Fujii> | ||||||||||||
Component: | Text | Assignee: | Nobody <webkit-unassigned> | ||||||||||||
Status: | RESOLVED DUPLICATE | ||||||||||||||
Severity: | Normal | CC: | don.olmstead, mmaxfield | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=247285 | ||||||||||||||
Bug Depends on: | |||||||||||||||
Bug Blocks: | 201214 | ||||||||||||||
Attachments: |
|
Description
Fujii Hironori
2019-08-27 22:08:09 PDT
Created attachment 377425 [details]
test case
Created attachment 377426 [details]
test case
It seems that ligature doesn't work.
complex text code path can show ligature glyphs properly. And, DumpRenderTree.exe has --complex-text option. .\WebKitBuild\Debug\bin64\DumpRenderTree.exe --show-webview --complex-text https://bug-201213-attachments.webkit.org/attachment.cgi?id=377426 Created attachment 377432 [details]
WIP patch
I found another issue of Windows port, and filed: Bug 201214 – [Win][UniscribeController] Material icons containing underscore or numbers aren't shown because ScriptItemize splits them apart This looks like a good start. Comment on attachment 377432 [details] WIP patch View in context: https://bugs.webkit.org/attachment.cgi?id=377432&action=review > Source/WebCore/platform/graphics/FontCascade.h:281 > +#if PLATFORM(COCOA) || USE(FREETYPE) || PLATFORM(WIN) With PLATFORM(WIN) in there that’s everyone so might as well just return true and drop this check. (In reply to Fujii Hironori from comment #3) > complex text code path can show ligature glyphs properly. > > And, DumpRenderTree.exe has --complex-text option. > > .\WebKitBuild\Debug\bin64\DumpRenderTree.exe --show-webview --complex-text > https://bug-201213-attachments.webkit.org/attachment.cgi?id=377426 I’m reminded of https://bugs.webkit.org/show_bug.cgi?id=179102 Myles (In reply to Don Olmstead from comment #8) > > I’m reminded of https://bugs.webkit.org/show_bug.cgi?id=179102 Myles What I've got in mind is specific to CoreText, but probably wouldn't be too difficult to hook the architecture I've got in mind up to HarfBuzz. I don't know about DirectWrite, though. Possibly. GTK port is always using complex text code path as default. Bug 183285 – [GTK] Switch to use always complex text code path Bug 194614 – [WPE][GTK] Clean up handling of WEBKIT_FORCE_COMPLEX_TEXT I confirmed GTK port doen't show the Material icons if I set the env ver WEBKIT_FORCE_COMPLEX_TEXT=0. Created attachment 402978 [details] [screenshot] layout issues I found layout issues for this patch. https://b.hatena.ne.jp/hotentry/it * unexpected word breaks * narrow character spaces (In reply to Fujii Hironori from comment #12) > * unexpected word breaks Filed: Bug 213691 – [WinCairo][Uniscribe] Katakana texts are unexpectedly wrapped because they don't fit into its own width (In reply to Fujii Hironori from comment #12) > * narrow character spaces Filed: Bug 213754 – [WinCairo][Uniscribe] synthetic bold of embedded bitmap font glyphs are placed narrowly in complex text code path Created attachment 403148 [details]
WIP patch
Myles is working on it in Bug 215318. Closed as duplicate. *** This bug has been marked as a duplicate of bug 215318 *** |