Needs better initial size for file listings

NEW
Unassigned

Status

()

P5
normal
7 years ago
5 months ago

People

(Reporter: micmon, Unassigned)

Tracking

unspecified
ARM
Android
Points:
---

Firefox Tracking Flags

(blocking-fennec1.0 -)

Details

(Whiteboard: readability , URL)

Attachments

(5 attachments)

(Reporter)

Description

7 years ago
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
Whiteboard: readability
Priority: -- → P4
tracking-fennec: --- → 11+
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?
Assignee: nobody → matspal
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.
blocking-fennec1.0: --- → -
tracking-fennec: 11+ → +
tracking-fennec: + → 17+
tracking-fennec: 17+ → +
Whiteboard: readability → readability [mentor=wesj]
Mentor: wjohnston
Whiteboard: readability [mentor=wesj] → readability
If this is still a thing, can I ask a contributor to help carry it over the line?
Flags: needinfo?(wjohnston)
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.
Flags: needinfo?(wjohnston)
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: + → ---
Mentor: wjohnston2000
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.