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.

Sign up for GitHub

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

Jump to bottom

Textfield has no class 'is-focused' in Firefox #166

Closed
lesykos opened this issue Aug 11, 2016 · 3 comments
Closed

Textfield has no class 'is-focused' in Firefox #166

lesykos opened this issue Aug 11, 2016 · 3 comments
Assignees
@vipentti
Labels
bug

Comments

@lesykos
Copy link

lesykos commented Aug 11, 2016

textfield-focus-firefox
@debois debois added the bug label Aug 12, 2016
@debois
Copy link
Owner

debois commented Aug 12, 2016 •
edited
Loading

I can confirm this on Firefox/Mac OS. @vipentti, any ideas?

@debois debois mentioned this issue Aug 12, 2016
Closed
@vipentti
Copy link
Collaborator

I'll look into this

@vipentti
Copy link
Collaborator

Found out the culprit. Just need to figure out a workaround.

Issue is that Firefox does not support focusin and focusout events that are currently being used by the containing div to capture focus.

Focusin and Focusout as well as a reported Firefox issue

@vipentti vipentti self-assigned this Aug 12, 2016
@vipentti vipentti mentioned this issue Aug 13, 2016
Closed
@debois debois mentioned this issue Aug 17, 2016
Merged
vipentti added a commit to vipentti/elm-mdl that referenced this issue Aug 17, 2016
@vipentti
Temporary fix for debois#166
add2439 
This will break debois#87 `onBlur` and `onFocus` custom handlers
on Firefox temporarily.

Firefox does not support `focusin` and `focusout` events currently.
@vipentti vipentti mentioned this issue Aug 17, 2016
Merged
debois pushed a commit that referenced this issue Aug 17, 2016
Merge pull request #193 from vipentti/textfield-firefox-hotfix
aefe57d 
Temporary fix for #166
@debois debois mentioned this issue Aug 17, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vipentti vipentti

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@debois @lesykos @vipentti