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

[BUGFIX] Main Navigation dropdown hover flickering between dropdowns. #512

Merged
merged 2 commits into from
Apr 16, 2019

Conversation

pxamike
Copy link
Contributor

@pxamike pxamike commented Apr 11, 2019

See: #511

Removing z-index 3000 from .main-navigation__sub-item-list makes the dropdown inherit the parent one ( .main-navigation ) at 2000.

From my testing it doesn't seem to affect any nearby content, but it's possible it would affect content with a very high (>1999) z-index.

@pxamike
Copy link
Contributor Author

pxamike commented Apr 11, 2019

Greets to @mesosa for the solution

@pxamike
Copy link
Contributor Author

pxamike commented Apr 11, 2019

Seems however that this line was added as a bugfix earlier, see 8aa1704

@pxamike
Copy link
Contributor Author

pxamike commented Apr 11, 2019

Ah, was able to recreate the testcase of #395 , will see if I can find another solution.
testcase

@pxamike
Copy link
Contributor Author

pxamike commented Apr 11, 2019

Latest commit makes sure that 8aa1704 fix is intact but still fixes the flickering problem.

testcase_fixed

@dmh dmh merged commit ac34c19 into master Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants