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

scrollTo unsupported on older android devices #604

Closed
1 task
sarayourfriend opened this issue Mar 8, 2022 · 3 comments
Closed
1 task

scrollTo unsupported on older android devices #604

sarayourfriend opened this issue Mar 8, 2022 · 3 comments
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: frontend Related to the Nuxt frontend 🐛 tooling: sentry Sentry issue

Comments

@sarayourfriend
Copy link
Collaborator

sarayourfriend commented Mar 8, 2022

Description

https://sentry.io/share/issue/336fc229d5fb4428bbdc73ded9c0b581/

Some folks browsing Openverse do not have the scrollTo function.

Do we want to include a conditional async polyfill for this feature? On mobile it's a pretty useful button.

Reproduction

  1. Try the scroll to top button on an older android device with an older chrome version.

Environment

  • OS: Android 6
  • Browser: Chrome
  • Version: 58

Additional context

May be related to how committed we are to supporting older Android devices which are more prevalent outside of North America and Europe.

Related to global & economic inclusiveness.

Resolution

  • 🙋 I would be interested in resolving this bug.
@sarayourfriend sarayourfriend added 🟩 priority: low Low priority and doesn't need to be rushed 🛠 goal: fix Bug fix 💻 aspect: code Concerns the software code in the repository labels Mar 8, 2022
@AetherUnbound
Copy link
Collaborator

The original Sentry alert seems to have disappeared, but a new alert was made here: https://sentry.io/share/issue/336fc229d5fb4428bbdc73ded9c0b581/

@obulat obulat added the 🐛 tooling: sentry Sentry issue label Sep 7, 2022
@obulat obulat transferred this issue from WordPress/openverse-frontend Feb 22, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Openverse Backlog Feb 23, 2023
@obulat obulat added the 🧱 stack: frontend Related to the Nuxt frontend label Feb 24, 2023
dhruvkb pushed a commit that referenced this issue Apr 14, 2023
🔄 Synced local '.github/ISSUE_TEMPLATE/' with remote '.github/ISSUE_TEMPLATE/'

Co-authored-by: openverse-bot <null>
@sarayourfriend
Copy link
Collaborator Author

The sentry issue does not exist any more. @obulat any idea if this is still a problem? Could we fix it with a polyfill, and if so, would it be a good first issue (just adding a polyfill should be straightforward and we have examples of it already).

@obulat
Copy link
Contributor

obulat commented Apr 29, 2024

Sorry for late reply, @sarayourfriend. I'm going to go ahead and close this issue. If we do get any more exceptions in Sentry, we could open a good first issue to add a polyfill.

@obulat obulat closed this as completed Apr 29, 2024
@openverse-bot openverse-bot moved this from 📋 Backlog to ✅ Done in Openverse Backlog Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟩 priority: low Low priority and doesn't need to be rushed 🧱 stack: frontend Related to the Nuxt frontend 🐛 tooling: sentry Sentry issue
Projects
Archived in project
Development

No branches or pull requests

3 participants