-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Remove the flash on story pages transitions. #13070
Conversation
b1077ab
to
e30e06a
Compare
|
||
// Remove the cursor: pointer; style set by the runtime, to avoid wrong | ||
// touch feedback on mobile, like a flashing overlay on page transitions. | ||
resetStyles(this.win.document.documentElement, ['cursor']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be sufficient to do this in the "standalone" case (where amp-story is the only tag in the document), where we add the CSS class .i-amphtml-story-standalone
to the document element anyway. So, maybe in the CSS we can just add:
.i-amphtml-story-standalone {
cursor: default !important;
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
435dc21
to
e2f7c84
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PTAL
|
||
// Remove the cursor: pointer; style set by the runtime, to avoid wrong | ||
// touch feedback on mobile, like a flashing overlay on page transitions. | ||
resetStyles(this.win.document.documentElement, ['cursor']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
e2f7c84
to
453dc3e
Compare
* Remove the flash on story page transitions. * Removing the iOS cursor: pointer on standalone stories only.
* Remove the flash on story page transitions. * Removing the iOS cursor: pointer on standalone stories only.
Removes the flash on story pages transitions.
From this, to this.
There's another known hack to get rid of this overlay, but it'd break tap feedback on all the elements, including links or forms if we ever support it.