Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

fix(inputs): ignoring input events in IE caused by placeholder changes o... #9264

Closed
wants to merge 1 commit into from

Conversation

jbedard
Copy link
Contributor

@jbedard jbedard commented Sep 25, 2014

...r focus/blur on inputs with placeholders

This is my attempt at fixing #8242 as well as the extra placeholder issues with focus/blur. I'm assuming I'm missing a weird test case that still breaks somewhere, but I still think this is an improvement (it definitely is for my use cases). This PR is for 1.2.x because this was a 1.2 regression.

@caitp
Copy link
Contributor

caitp commented Sep 25, 2014

Open the PR against the master branch, it will be cherry-picked into v1.2.x if it lands

@jbedard
Copy link
Contributor Author

jbedard commented Sep 25, 2014

Bah, sorry, just close this one then?

@caitp
Copy link
Contributor

caitp commented Sep 25, 2014

yeah

@jbedard
Copy link
Contributor Author

jbedard commented Sep 25, 2014

K, will reopen in a minute...

@jbedard jbedard closed this Sep 25, 2014
@jbedard
Copy link
Contributor Author

jbedard commented Sep 25, 2014

Created #9265. Inputs and validation sure have changed in 1.3, but the fix seems to be the same.

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

Successfully merging this pull request may close these issues.

3 participants