Opened 8 years ago
Closed 2 years ago
#39598 closed defect (bug) (wontfix)
Twenty Seventeen: Can't scroll on iPhone 4
Reported by: | superpoincare | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 4.7.1 |
Component: | Bundled Theme | Keywords: | has-patch needs-testing needs-screenshots needs-refresh |
Focuses: | css | Cc: |
Description
If I visit wp-themes.com/twentyseventeen/ on iPhone 4 Safari with iOS 7.1, I can't swipe to scroll down and read/access the text.
It might be a general issue with touch devices, not just on iPhone 4 since the browser is high on HTML5 features and should work ideally.
Don't know how to show a demo of this. Anyone reading has to test it herself/himself to confirm.
Attachments (1)
Change History (15)
#2
@
8 years ago
- Summary changed from Can't scroll on Twenty Seventeen on iPhone 4 to Twenty Seventeen: Can't scroll on iPhone 4
This ticket was mentioned in Slack in #themereview by davidakennedy. View the logs.
8 years ago
This ticket was mentioned in Slack in #core-themes by davidakennedy. View the logs.
8 years ago
This ticket was mentioned in Slack in #forums by hardeepasrani. View the logs.
8 years ago
#8
@
8 years ago
- Keywords needs-patch added; reporter-feedback removed
- Milestone changed from Awaiting Review to 4.8
Just noting that this is indeed a bug and was verified by a few others on the Theme Review team. It needs to be investigated further.
#9
@
8 years ago
- Keywords has-patch needs-testing added; needs-patch removed
Here's what happens, it's not that you can't scroll, but the header is too big for the parallax effect to be noticed. This is because the theme uses viewport units to set the height, which is buggy in iOS7.
Attached patch adds media queries with specified heights in pixels for these screen sizes.
#10
@
8 years ago
- Keywords needs-screenshots added
@maguiar thanks for working on a patch! wow, that is a bunch of css :) does this have any effect on more modern devices running newer iOs versions?
Some screencasts before and after would be helpful here as a visual record and to demonstrate the fix.
@superpoincare Thanks for the report!
I wouldn't trust wp-themes.com for testing purposes. Can you reproduce this on the official demo? It's here: http://2017.wordpress.net/