Open
Bug 786382
Opened 12 years ago
Updated 2 years ago
Font inflation not consistent for elements with similar positions like in forums and tables, e.g. on fark.com
Categories
(Core :: Layout: Text and Fonts, defect, P5)
Core
Layout: Text and Fonts
Tracking
()
People
(Reporter: nalexander, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
image/png
|
Details |
With today's nightly, on my Samsung Galaxy SII the headlines on fark.com are inflated to all different sizes. One and two line headlines are tiny; longer headlines are readable. Looks very strange.
I expect this to get duped to an existing bug.
Reporter | ||
Comment 1•12 years ago
|
||
Updated•12 years ago
|
tracking-fennec: --- → ?
Component: Graphics, Panning and Zooming → Layout
Product: Firefox for Android → Core
Updated•12 years ago
|
Blocks: font-inflation
Component: Layout → Layout: Text
Updated•12 years ago
|
Assignee: nobody → sjohnson
tracking-fennec: ? → +
Updated•12 years ago
|
Summary: Font inflation not consistent on fark.com → Font inflation not consistent for elements with similar positions like in forums and tables, e.g. on fark.com
Updated•11 years ago
|
Assignee: jaywir3 → nobody
Comment 6•10 years ago
|
||
Happening here also on firefox for android:
http://foroafeitado.com/foro/maquinillas-tradicionales-cuchillas-29/personna-platinum-mercadona-las-mismas-8720/index2.html
Comment 7•10 years ago
|
||
Is there any way I can add Linux as affected platform?
I originally filed a bug about FF for Linux, but it was marked as a duplicate of this report which is limited to Android.
Reporter | ||
Comment 8•10 years ago
|
||
(In reply to Clemens Eisserer from comment #7)
> Is there any way I can add Linux as affected platform?
> I originally filed a bug about FF for Linux, but it was marked as a
> duplicate of this report which is limited to Android.
The underlying renderer is the same; I don't think this is platform specific. The bug is filed in Core :: Layout, but I changed the platform to All just in case.
OS: Android → All
Comment 11•6 years ago
|
||
What we're missing is what Chrome calls "fingerprinting" in this document: https://docs.google.com/document/d/1PPcEwAhXJJ1TQShor29KWB17KJJq7UJOM34oHwYP3Zg/
I.e. translating this into our own terminology: For each font inflation flow root, we'd have to also compute a fingerprint based on a few relevant properties. Then, if for a given fingerprint there is at least one font inflation flow root with enough text to actually trigger font inflation, we'll enable font inflation for *all* flow roots sharing the same fingerprint. [1] The data that goes into the fingerprint needs to be carefully chosen so that the bits that are problematic today [2] share a common fingerprint and get inflated together, while at the same time correctly tagging things that should remain uninflated with a different fingerprint.
[1] Actually Chrome goes a step further and also ensures a common font size multiplier (presumably for things like pages with nested/indented comments), but an initial implementation might get away without doing that. This would also be one more argument to do what I suggested in bug 1493266 and completely move the MinimumFontSizeFor() calculations into the calculation of the FontInflationData instead of partially calculating it on the fly - I think this should make it easier to then ensure a common font size factor across all flow roots sharing a fingerprint.
[2] Forum threads, comment areas, Bugzilla bugs :-) and similar pages, where the content we care about for font inflation is divided into lots of separate bits (and therefore covered by separate font inflation flow roots) and where some of those bits are short enough to individually fall below the font inflation threshold.
Updated•6 years ago
|
Updated•4 years ago
|
See Also: → https://webcompat.com/issues/57624
Updated•4 years ago
|
See Also: → https://webcompat.com/issues/68051
Updated•3 years ago
|
Webcompat Priority: --- → ?
Updated•3 years ago
|
Webcompat Priority: ? → P2
Updated•2 years ago
|
Updated•2 years ago
|
Severity: normal → S3
Comment 12•2 years ago
|
||
The severity field for this bug is relatively low, S3. However, the bug has 5 duplicates.
:jfkthame, could you consider increasing the bug severity?
For more information, please visit auto_nag documentation.
Flags: needinfo?(jfkthame)
Comment 13•2 years ago
|
||
The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.
Flags: needinfo?(jfkthame)
You need to log in
before you can comment on or make changes to this bug.
Description
•