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

Select onInput doesn't function in Microsoft Edge #2331

Open
kamilgrzegorczyk opened this issue Feb 11, 2020 · 5 comments
Open

Select onInput doesn't function in Microsoft Edge #2331

kamilgrzegorczyk opened this issue Feb 11, 2020 · 5 comments
Labels

Comments

@kamilgrzegorczyk
Copy link

Hello,

Browser: Microsoft Edge 44
Preact 10.3.1

Way to reproduce:

Expected:
The value is updated properl and onInput callback is getting fired.

@marvinhagemeister
Copy link
Member

Can't reproduce the issue in the chromium based versions of edge. Can confirm that the issue occurs on older versions with the Chakra engine.

@kamilgrzegorczyk
Copy link
Author

Yeah, this is also my observation - new Edge engine does not have that issue. Unfortunately Microsoft Edge 44 was released just few months ago so it's hard to treat it as an unsupported browser.

@marvinhagemeister
Copy link
Member

marvinhagemeister commented Feb 11, 2020

We never said that we don't support Edge ;) In fact the oldest version we officially support is Internet Explorer 11. This is definitely a bug in Preact.

@kamilgrzegorczyk
Copy link
Author

kamilgrzegorczyk commented Feb 11, 2020

I believe that this might be due to the fact that Microsoft Edge does not support the onInput event on the <select> tags. I have replaced onInput with onChange there.

Source:
https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/input_event

Microsoft Edge - partial:
Not supported on select, checkbox, or radio inputs.

@developit
Copy link
Member

I think we can find a way to patch this since we're already special-casing onChange.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants