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

Options in select element still highlighted after unselecting it. #133

Closed
Singha360 opened this issue May 6, 2021 · 6 comments · Fixed by #134
Closed

Options in select element still highlighted after unselecting it. #133

Singha360 opened this issue May 6, 2021 · 6 comments · Fixed by #134
Labels

Comments

@Singha360
Copy link

Peek 2021-05-05 20-58

@Smankusors Smankusors added bug Something isn't working component: select css/sass For something that related to styling/view labels May 6, 2021
@DanielRuf
Copy link

Can be reproduced on https://materializecss.github.io/materialize/select.html

It seems the active class is not removed.

Bildschirmfoto 2021-05-06 um 19 44 59

The issue does not exist on https://materializecss.com/select.html
One of the commits that is not in the used version there and that might be relevant is c0da340#diff-f23639ab50ee01de087486418e0e652284a0a5e6d05cc0698a6e85521bb46a38

At least I think so.

Maybe someone can do a git bisect:

git bisect start
git bisect good v1.0.0
git bisect bad v1.1.0-alpha
...
git bisect bad/good

@DanielRuf DanielRuf added javascript and removed css/sass For something that related to styling/view labels May 6, 2021
@DanielRuf
Copy link

@materializecss/members

@bugy
Copy link

bugy commented May 6, 2021

I think it was broken here:
#98

After I reverted the changes in dropdown.js locally, it worked fine for me

@ChildishGiant

Another problem which I see now, is that the list elements are not focused anymore. document.activeElement always returns <ul> element, instead of <li>

@DanielRuf
Copy link

Ah, that makes sense as it uses the dropdown component.

I think we should try to do at least one review per PR in the future =)

@ChildishGiant
Copy link
Member

I'm on holiday right now but will take a look at patching it 😅 Does look like I broke it, but this is what alphas are for 😉

@DanielRuf
Copy link

Does look like I broke it, but this is what alphas are for 😉

Exactly =)

I'm on holiday right now but will take a look at patching it 😅

No hurry, enjoy your holiday =)

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.

5 participants