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

Example 15/16 and RWD #151

Closed
domel opened this issue Feb 28, 2023 · 6 comments
Closed

Example 15/16 and RWD #151

domel opened this issue Feb 28, 2023 · 6 comments

Comments

@domel
Copy link
Collaborator

domel commented Feb 28, 2023

Example 15 and Example 16 in the spec display problematically in mobile devices. Maybe overflow-x:scroll in CSS will help?

@TallTed
Copy link
Member

TallTed commented Feb 28, 2023

Can you provide a screenshot of some problematic display?

Over time (not specific to these examples), I've found different issues in different browsers on different devices. Much is testable and improvable with web developer tools like the Web Developer Extension and Stylebot. The former can preview a page in variously sized DIVs to emulate various devices, though it is limited to the browser in which the extension is installed. Both can allow CSS tweaks to be shown in your local browser without changing the live page until your changes are well tested. Perhaps these will help you test whether overflow-x:scroll makes a difference, through testing in a desktop browser with DIVs sized to match your mobile devices....

@domel
Copy link
Collaborator Author

domel commented Feb 28, 2023

Here you are IMG_20230228_201704

To see the second example I should scroll to right, which is annoying.
The same issue is in Example 7 and Example 8.

@TallTed
Copy link
Member

TallTed commented Mar 1, 2023

Oh, how interesting... I think that will not be a CSS solution, but it shouldn't be too difficult to solve by editing the documentation HTML. I'll try to find the time to poke at it, but someone else may get there sooner.

@william-vw
Copy link
Collaborator

@domel @TallTed Please feel free to issue a PR to fix this problem.

@domel
Copy link
Collaborator Author

domel commented Mar 9, 2023

@william-vw see PR #159

@william-vw
Copy link
Collaborator

thanks - merged the PR.

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

3 participants