Skip to content

Missing viewport meta tag #510

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

Closed
Philonous opened this issue Jun 15, 2016 · 6 comments
Closed

Missing viewport meta tag #510

Philonous opened this issue Jun 15, 2016 · 6 comments

Comments

@Philonous
Copy link

Philonous commented Jun 15, 2016

The hackage site is currently nigh unusable on high-dpi mobile devices because of a missing viewport meta tag.

Adding

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

improves it markedly

@gbaz
Copy link
Contributor

gbaz commented Jun 15, 2016

Without a high-dpi screen myself (yes I feel old and behind the times) its hard to see what's going on.

Can you provide screenshots or e.g. a way to use a tool like http://www.infobyip.com/testwebsiteresolution.php so I can understand the issue more clearly?

@Philonous
Copy link
Author

I should have made it clearer that I meant high-dpi mobile devices, sorry.

I've added some screenshots from my mobile phone, illustrating the problem. I've down-sized the images from the native 1440x2560

Please note that the screen of the device is 69mm by 113mm and at least on my (desktop) screen the images are displayed much larger than that. So in real life the text is tiny

This is before applying the meta tag

On the real device the pages barely legible, hitting links is hard

Hackage home page

hackage_before

Servant package page

servant_before

This is after adding the meta tag:

The pages are legible, links are easier to hit. It's not the best mobile experience in the world, but it's usable.

Hackage homepage

hackage_after

Servant page

servant_after

@gbaz
Copy link
Contributor

gbaz commented Oct 14, 2016

Thanks! So you'd want this on every page then, not just some subset?

@Philonous
Copy link
Author

Yes, it would be nice to have on every page

@gbaz
Copy link
Contributor

gbaz commented Oct 23, 2016

should be addressed by 2c3e31c

(rolled into PR #551)

@Philonous
Copy link
Author

Thanks! This should fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants