Closed
Bug 707917
Opened 13 years ago
Closed 4 years ago
Word cloud does not render correctly
Categories
(Firefox for Android Graveyard :: General, defect, P3)
Tracking
(blocking-fennec1.0 -, fennec-)
RESOLVED
INCOMPLETE
People
(Reporter: isandu, Unassigned)
References
()
Details
(Whiteboard: readability)
Attachments
(4 files)
The word cloud at the bottom of zoso.ro does not render correctly (screenshot attached). To reproduce, go to www.zoso.ro and scroll to the bottom of the page, to the word cloud.
Updated•13 years ago
|
Assignee: nobody → dbaron
Priority: -- → P2
Updated•13 years ago
|
tracking-fennec: --- → 11+
Updated•13 years ago
|
Keywords: fennecnative-releaseblocker
Updated•13 years ago
|
Blocks: font-inflation
Whiteboard: readability
Updated•13 years ago
|
blocking-fennec1.0: --- → +
Updated•13 years ago
|
Status: NEW → ASSIGNED
Comment 2•13 years ago
|
||
Scott, dbaron thinks this may be a dupe of bug 708187, can you confirm and mark it if it is.
Comment 3•13 years ago
|
||
So the word cloud doesn't seem to be on the page anymore. But if it were I think it probably would have been fixed by bug 708187. So I'm going to mark it as a duplicate, but if there's still some way to see that content on the blog, I'd be interested in testing.
I'll test on http://web.archive.org/web/20110725144805/http://www.zoso.ro/ though it doesn't have the same styles.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Comment 4•13 years ago
|
||
The word cloud is still available on the page and it still does not render correctly. Attaching screenshot from test on the latest nightly on Galaxy Nexus.
Reporter | ||
Comment 5•13 years ago
|
||
Reporter | ||
Comment 6•13 years ago
|
||
This is how it shows on the desktop.
Comment 7•13 years ago
|
||
If you look closely you will see an ever so slight variance in the font-sizes of the 'larger' words in the word cloud; but as Irina mentions it nowhere near looks like what you see on Desktop. This kind of defeats the purpose of using the word cloud if font-inflation can't get it right.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 8•13 years ago
|
||
So, just to give an update on this bug quickly:
The way the font inflation algorithm works, it takes a range of the font sizes on a given page, say 0px (we always start from 0px, even though there aren't any fonts that are actually rendered at 0px) to, for example, 45px. When we inflate the fonts to a minimum size (say we calculate the minimum to be 30px), then we squish the original range of 0px-45px into the range of 30px-45px, trying to maintain some separation between differing font sizes, while at the same time increasing lower font sizes. This means that we have a much finer gradiation than we originally have. In turn, this means that we have less range of difference between font sizes.
What I think is happening in this case is that we don't have enough difference between the font sizes after they are inflated to notice a difference. Unfortunately, this might just be an inherent flaw of the algorithm as it works right now. We can discuss it, and we might be able to come up with a fix, but I'd recommend that we remove this from the blocking list for fennec 1.0.
Comment 9•13 years ago
|
||
Also, some evidence to support my claim. If you adjust the font-inflation emPerLine property, starting at 60 emPerLine, and gradually move down to 15 emPerLine, you'll notice the fonts start to coalesce so they have a very small range of differences. This, I believe, is because as we move from 60 emPerLine down to 15 emPerLine, we're effectively increasing the minimum font size, and thus decreasing the possible range of differing font sizes available.
Comment 11•13 years ago
|
||
I think the original problem reported here was in fact a duplicate, as was marked. The new problem doesn't seem as serious... and is related to bug 707195.
Updated•13 years ago
|
blocking-fennec1.0: ? → -
Updated•13 years ago
|
blocking-fennec1.0: ? → -
Updated•12 years ago
|
tracking-fennec: 11+ → -
Comment 12•12 years ago
|
||
To evangelism to follow up. Will work with lmandel for proper tracking.
Assignee: dbaron → nobody
Component: General → Evangelism
Comment 13•12 years ago
|
||
Erin, can you comment on why you think this is an evangelism problem? From the comment stack this issue looks to be due to Fennec's use of font inflation.
Comment 14•12 years ago
|
||
Lawrence, text that shouldn't be inflated (such as this) can use the text-size-adjust CSS property
https://developer.mozilla.org/en-US/docs/CSS/text-size-adjust
Updated•12 years ago
|
Component: Evangelism → Mobile
Product: Firefox for Android → Tech Evangelism
Comment 15•12 years ago
|
||
(In reply to Brad Lassey [:blassey] from comment #14)
> Lawrence, text that shouldn't be inflated (such as this) can use the
> text-size-adjust CSS property
>
> https://developer.mozilla.org/en-US/docs/CSS/text-size-adjust
In this case, the site works as designed in the Android stock browser and Chrome. AFAIK, the site does not work in Firefox for Android due to our font inflation implementation. Given the state of the mobile Web and our requests for sites to accommodate Firefox by moving to standard practices, asking a site to add a non standard property in order to support Firefox for Android seems off message for me. I think we either need to decide to fix this use case in our browser or accept that this use case will be broken in our browser.
Component: Mobile → Readability
Product: Tech Evangelism → Firefox for Android
Comment 16•12 years ago
|
||
Comment 17•10 years ago
|
||
Mass change to move Firefox for Android::Readability bugs to block bug 1129033. Filter on readability-mass-move.
Blocks: 1129033
Component: Readability → General
Comment 19•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 4 years ago
Resolution: --- → INCOMPLETE
Assignee | ||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•