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

Feedback feature does not work on safari #3179

Closed
5 tasks done
garrettheel opened this issue Nov 4, 2021 · 2 comments
Closed
5 tasks done

Feedback feature does not work on safari #3179

garrettheel opened this issue Nov 4, 2021 · 2 comments
Labels
bug Issue reports a bug resolved Issue is resolved, yet unreleased if open

Comments

@garrettheel
Copy link
Contributor

Contribution guidelines

I've found a bug and checked that ...

  • ... the problem doesn't occur with the mkdocs or readthedocs themes
  • ... the problem persists when all overrides are removed, i.e. custom_dir, extra_javascript and extra_css
  • ... the documentation does not mention anything about my problem
  • ... there are no open or closed issues that are related to my problem

Description

The new "Was this page helpful" feature does not work on Safari.

When clicking a button, the following error is logged to the console:

TypeError: null is not an object (evaluating 'e.submitter.getAttribute')

Per https://developer.mozilla.org/en-US/docs/Web/API/SubmitEvent/submitter, SubmitEvent.submitter is not compatible with IE or Safari.

Expected behaviour

Expect the button to submit the feedback.

Actual behaviour

No feedback and a JS console error.

Steps to reproduce

Use the Safari browser to submit feedback.

Package versions

  • Material: 7.3.6+insiders.3.1.5

Configuration

n/a

System information

n/a

@squidfunk squidfunk added the bug Issue reports a bug label Nov 4, 2021
@squidfunk
Copy link
Owner

squidfunk commented Nov 4, 2021

Thanks for reporting! Fixed in 45b6801. I was naive to assume something as modern as FormEvent.submitter being available on all modern platforms 😉

@squidfunk squidfunk added the resolved Issue is resolved, yet unreleased if open label Nov 4, 2021
@squidfunk
Copy link
Owner

Released as part of 7.3.6-insiders-3.2.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue reports a bug resolved Issue is resolved, yet unreleased if open
Projects
None yet
Development

No branches or pull requests

2 participants