-
Notifications
You must be signed in to change notification settings - Fork 203
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
Fixes #3505 - Remove animations/scrolling for prefers-reduced-motion #3747
Conversation
@@ -20,9 +18,6 @@ const handleMessage = (message) => { | |||
|
|||
sendAnalyticsCS(utm_campaign, utm_source); | |||
if (url) { | |||
showStep("category", { | |||
url: extractPrettyUrl(url), | |||
}); | |||
updateUrl(url); |
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've also removed the "skip the url and scroll to the category step" when reporting through "Report site issue". It was part of the experiment to try and improve the form drop off. Thinking it might be confusing for new users as it happens immediately, skipping the part that we're looking for web compatibility issues (and "Learn more about web compatibility" pop up) .
r? @denschub |
@@ -8,4 +8,3 @@ | |||
<p>You can return to this web page at any time to track the progress of this report.</p> | |||
</section> | |||
</script> | |||
˚ |
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.
😆
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.
🙈
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.
Thanks for working on this!
Thank you! |
Fixes #3505 (and #3735)