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

MaterialListBox does not get focus when clicked on #756

Closed
mtmmtm9 opened this issue Dec 22, 2017 · 2 comments
Closed

MaterialListBox does not get focus when clicked on #756

mtmmtm9 opened this issue Dec 22, 2017 · 2 comments
Assignees
@kevzlou7979
Labels
add unit test ready
Milestone
2.0.1

Comments

@mtmmtm9
Copy link

mtmmtm9 commented Dec 22, 2017

I would like to have a key-handler for some widgets. MaterialListBox and MaterialTextBox
when calling addDomHandler() for these widgets I have problem receiving any events for the MaterialListBox . It only works if I click on a MaterialTextBox and the uses to move the focus to the MaterialListBox . After that it is possible to receive the key-event. The problem might be that clicking on a MaterialListBox does not give it focus. Tested on 2.0.1-SNAPSHOT.
@kevzlou7979 kevzlou7979 self-assigned this Jan 16, 2018
@kevzlou7979 kevzlou7979 added this to the 2.0.1 milestone Jan 16, 2018
@kevzlou7979
Copy link
Contributor

kevzlou7979 commented Jan 17, 2018 •
edited
Loading

This was fixed via 6e2eec2 added test via fc6aefc, it's because of the hacked fixed done on IE 11 Dogfalo/materialize#901 which on the upcoming Materialize 1.0 we will receive a patch for the IE issue.

kevzlou7979 added a commit that referenced this issue Jan 17, 2018
@kevzlou7979
Fixed MaterialListBox does not get focus when clicked on #756
6e2eec2
kevzlou7979 added a commit that referenced this issue Jan 17, 2018
@kevzlou7979
Added testFocusAndBlurHandlers on bug #756
fc6aefc
@kevzlou7979
Copy link
Contributor

focus-blur

@kevzlou7979 kevzlou7979 mentioned this issue Jan 31, 2018
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kevzlou7979 kevzlou7979

Labels
add unit test ready
Projects
None yet
Milestone
2.0.1
Development

No branches or pull requests
2 participants
@kevzlou7979 @mtmmtm9