Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add max-height to workaround Chrome on Android's font-boost making pages unreadable #1698

Closed
cesy opened this issue Feb 21, 2016 · 14 comments
Closed

Comments

@cesy
Copy link
Contributor

cesy commented Feb 21, 2016

There are certain pages that are unreadable in parts in Chrome on Android, including anything in format=light or more importantly site style. Mobile Chrome has a "feature" called font boost, where any paragraph over a certain length gets enlarged, leaving the rest of the site too small to read. This frequently applies, for example, to a journal post with comments. This is a problem. It applies both to any link from www.dreamwidth.org/mobile/read, and also to following a link from an email notification to a journal post where someone has chosen to display journal entries in site style, which is something I like to do quite often on my phone. Setting a large max-height should fix it. Discussed on http://dw-dev.dreamwidth.org/182633.html

@rahaeli
Copy link
Contributor

rahaeli commented Feb 21, 2016

Can you get a screenshot of your phone showing one of the unreadable pages, btw? (Just to give a sense of what's going on!)

@cesy
Copy link
Contributor Author

cesy commented Feb 21, 2016

screenshot_2016-02-21-16-58-01

@cesy
Copy link
Contributor Author

cesy commented Feb 21, 2016

That one just has subject and author tiny. I'll watch out for a public one with comments as that's even clearer.

@kaberett
Copy link
Contributor

As far as I'm concerned you're welcome to use any of my public posts!
On 21 Feb 2016 17:20, "Cesy" notifications@github.com wrote:

That one just has subject and author tiny. I'll watch out for a public one
with comments as that's even clearer.


Reply to this email directly or view it on GitHub
#1698 (comment)
.

@azurelunatic
Copy link
Contributor

Mine as well!

On Sun, Feb 21, 2016 at 9:21 AM, kaberett notifications@github.com wrote:

As far as I'm concerned you're welcome to use any of my public posts!
On 21 Feb 2016 17:20, "Cesy" notifications@github.com wrote:

That one just has subject and author tiny. I'll watch out for a public
one
with comments as that's even clearer.


Reply to this email directly or view it on GitHub
<
https://github.com/dreamwidth/dw-free/issues/1698#issuecomment-186862393>
.


Reply to this email directly or view it on GitHub
#1698 (comment)
.

@cesy
Copy link
Contributor Author

cesy commented Feb 22, 2016

Example where names are unreadable and reply/expand links are too small to hit without zooming:
screenshot_2016-02-22-07-06-03

@rahaeli
Copy link
Contributor

rahaeli commented Feb 22, 2016

Thanks! The screenshots make it a lot easier for me to picture what the problem is. :)

@kaberett
Copy link
Contributor

Have a particularly egregious example:

screenshot_2016-03-16-08-40-40

@chrisboyle
Copy link
Contributor

claim

@chrisboyle
Copy link
Contributor

Unfortunately what was asked for doesn't achieve what was wanted. The max-height hack does disable font boost, but that just leaves all the text at the "unreadable" size. It's consistent, and you can zoom in, but you then need to scroll side to side to read a line of text. Furthermore, the hack makes the browser's Accessibility / Text Size setting do nothing. I don't think that's an improvement.

screenshot_20160514-140818

screenshot_20160514-140825

screenshot_20160514-140901

There is a semi-standardised way to disable this sort of thing: [-webkit-]text-size-adjust, but Chrome ignores it.

CSS does not have "min-font-size". One could imagine doing something like @media (max-width: 600px) { html * { font-size: 48px; } } and relying on the apparently established accessibility policy that styles should only adjust font sizes relatively (using ems). An initial unscientific test of that did not go well:

screenshot_20160514-155535

Seems to me the most correct way to make all the text bigger without the side-scrolling problem is to do the "stop pretending there's a huge viewport" thing that many pages on the site do already (but not the style=site / format=light ones that are the subject of this bug):

<meta name="viewport" content="width=device-width, initial-scale=1.0"/>

@chrisboyle
Copy link
Contributor

Viewport device-width does fix all of these issues, but something would need doing about the top of the page:

screenshot_20160514-162040

screenshot_20160514-162047

@rahaeli
Copy link
Contributor

rahaeli commented May 14, 2016

Ugh. I'm not sure what the best solution to this is. Paging some people who might know a workaround: @afuna @deborahgu

@nfagerlund
Copy link
Contributor

I think the s2foundation beta fixes this whole family of disgusting mobile Chrome issues.

@kareila
Copy link
Member

kareila commented Jul 21, 2020

In that case I will consider this closed! We can always come back to it if needed.

@kareila kareila closed this as completed Jul 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

8 participants