2.22 KB, patch
|Details | Diff | Splinter Review|
1.78 KB, text/css
347 bytes, text/css
3.80 KB, text/css
1.39 KB, text/css
On pages like http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-birch-android/ the text of the actual files is unreadably small on the default zoom level. However, there is still space, so this text could be displayed with a larger font.
OS: Linux → Android
Hardware: All → ARM
Created attachment 598268 [details] [diff] [review] hack This hack makes shrink-wrapped tables use all the available width rather than shrink-wrapping. It makes the file list table at the URL use the full width of the viewport, and its text slightly larger. It's an improvement IMO, but it seems like a brutal approach. Does anyone have other suggestions on how to improve a case like this? Also, does anyone know of "table based" sites I can test?
Should that patch be reviewed or given feedback?
I don't think that patch is quite valid because it ignores the min-width. I think the layout here was probably totally changed by bug 706193, such that font inflation isn't done at all. That said, I don't understand the description in comment 0 based on the behavior we had back then.
Now that we've added a bunch of fixes to text rendering on Mobile, this should look much better now. Please re-test.
Whiteboard: readability → readability [mentor=wesj]
Whiteboard: readability [mentor=wesj] → readability
If this is still a thing, can I ask a contributor to help carry it over the line?
This is still a thing. WesJ is fairly heads down in iOS land. Poking other people in #mobile, Margaret might be good as this is a look that we inherit from desktop.
Created attachment 8639524 [details] Base style Late reply, but yeah, someone could finish this up. Long ago I spent some time trying to make the page's responsive (just using dev-tools), so I figured I'd upload the CSS files for that if anyone was interested. This was a base style, mostly taken from whats in tree already.
Created attachment 8639525 [details] Small screen desktop css This was a style for small screens on Desktop, essentially removing the padding around the edges once the screen was small enough that it was taking up all the content. It also colored the header in this mode so that it would stand out.
Created attachment 8639526 [details] Touch screen styling (large screens) This was for large screen touch-enabled-devices. Basically looks like desktop right now, but makes the rows big enough to interact with reliably.
Created attachment 8639528 [details] Small touch screen tweaks This applies on small touch screen devices to remove a bunch of labels that don't really fit on screen anymore. The idea with these was that they'd all be contained in different media queries, so different ones would apply in different situations. i.e. anyone who's playing with this shouldn't expect to just apply one of these and have everything work.
remove the tracking fennec flag for now, because this bug happened 2 years ago , and please provide the reproducible way if still existing. Thank you !
tracking-fennec: + → ---
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195 Needinfo :susheel if you think this bug should be re-triaged.
Priority: P4 → P5
You need to log in before you can comment on or make changes to this bug.