Closed
Bug 1422408
Opened 7 years ago
Closed 7 years ago
Wrong Japanese font selected on jisho.org
Categories
(Core :: Graphics: WebRender, defect, P1)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox57 | --- | unaffected |
firefox58 | --- | unaffected |
firefox59 | --- | unaffected |
People
(Reporter: TD-Linux, Assigned: lsalzman)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [wr-reserve])
Attachments
(2 files)
A super pixely font is selected for large characters on jisho.org when Webrender is on.
Reporter | ||
Comment 1•7 years ago
|
||
Updated•7 years ago
|
status-firefox57:
--- → unaffected
status-firefox58:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Whiteboard: [wr-mvp] [triage]
Updated•7 years ago
|
Blocks: stage-wr-trains
Priority: -- → P3
Updated•7 years ago
|
Whiteboard: [wr-mvp] [triage] → [wr-reserve]
Assignee | ||
Comment 2•7 years ago
|
||
Which version of Firefox are you using? I am not able to reproduce this, as it might be fixed in nightly by bug 1413793.
Flags: needinfo?(tdaede)
Reporter | ||
Comment 3•7 years ago
|
||
Latest Nightly, 59.0a1 (2017-12-04), it still reproduces.
It might be Linux specific, as Bug 1408117 is.
Flags: needinfo?(tdaede)
Assignee | ||
Comment 4•7 years ago
|
||
(In reply to Thomas Daede [:TD-Linux] from comment #3)
> Latest Nightly, 59.0a1 (2017-12-04), it still reproduces.
>
> It might be Linux specific, as Bug 1408117 is.
The platform I tried to reproduce on was Linux.
Assignee | ||
Comment 5•7 years ago
|
||
(In reply to Lee Salzman [:lsalzman] from comment #4)
> (In reply to Thomas Daede [:TD-Linux] from comment #3)
> > Latest Nightly, 59.0a1 (2017-12-04), it still reproduces.
> >
> > It might be Linux specific, as Bug 1408117 is.
>
> The platform I tried to reproduce on was Linux.
Can you see what font it is trying to select, then use fc-match from console to see what it ends up routing to? Just a guess, but if it's a combination of a bitmap (non-scalable) font, and the webpage uses a content transform, it miiiight produce symptoms like that. But this would be rather odd and user specific, rather than just platform specific.
Reporter | ||
Comment 6•7 years ago
|
||
Using fc-match the first font that returns something other than "DejaVu Sans" is the "MS ゴシック" entry (MS Gothic) which is a font I have explicitly installed in my system, which is why it doesn't reproduce for you (presumably for you it hits the san-serif at the end of the font). MS Gothic is a really old font with both bitmaps and outlines (I have it installed because some programs depend on the exact pixel spacing).
Assignee | ||
Comment 7•7 years ago
|
||
(In reply to Thomas Daede [:TD-Linux] from comment #6)
> Using fc-match the first font that returns something other than "DejaVu
> Sans" is the "MS ゴシック" entry (MS Gothic) which is a font I have explicitly
> installed in my system, which is why it doesn't reproduce for you
> (presumably for you it hits the san-serif at the end of the font). MS Gothic
> is a really old font with both bitmaps and outlines (I have it installed
> because some programs depend on the exact pixel spacing).
Could you email me the font and the fontconfig file you are using to set up this font on your system, to ease repro for me?
Updated•7 years ago
|
Blocks: webrender-site-issues
Assignee | ||
Comment 8•7 years ago
|
||
Tracked this down to a problem dealing with fonts that provide both embedded bitmaps and scalable outlines. We're always selecting the embedded bitmaps in WebRender when available, which is not how Skia normally behaves. We have to choose between outline or bitmaps depending on whether the final resolved size actually many any available strikes.
Will take a couple days to properly engineer a fix.
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
Priority: P3 → P1
Assignee | ||
Comment 9•7 years ago
|
||
This will be resolved by WR PR https://github.com/servo/webrender/pull/2198
Assignee | ||
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•