-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix/70 locale a11y #93
Conversation
4487024
to
016ec92
Compare
016ec92
to
a4d6d91
Compare
c277438
to
ab0ea19
Compare
when on desktop and tab into the autocomplete it displays fine, but when _clicking_ into it the `input` and `ul` are hidden. not sure if this will be needed after other changes though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cleaned up some of my WIP JS, added translations, etc. I'm running into a few CSS issues that I haven't found solutions for, though.
- Push content down when menus are open - Fix flex layout of nav menu on mobile
mobile doesn't need it b/c of the border from `#masthead`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, this is looking good in my tests 👍🏻
Fixes #70
Branches off the initial commits in #71 with styles to get the autocomplete as close as possible to the designs and the current switcher on prod
Screenshots
Desktop
Mobile (emulated)
Mobile
The autoselected input text isn't a very nice experience on my Chrome for Android, as the contextual options are displayed, but I guess that is expected, and enables the input to be cleared with one tap of delete: