-
Notifications
You must be signed in to change notification settings - Fork 934
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
[IDP] Language selector component #4937
Conversation
…tes and more improvements
…ge selector dropdown
…English languages
…t ref to uswds working branch
…semantic text for screen readers
…cate English translation
|
@mitchmoccia When the languages button has its menu active, selecting the button again should dismiss the menu. As far as I can tell, it does not. |
…e button is selected repeatedly
@thisisdano This has been addressed. The language selector drop down now toggles when you select the language button over and over. Tested on all examples (three or more, header example, and unstyled) and also with the keyboard. |
packages/usa-header/_index.scss
Outdated
| @forward "usa-skipnav"; | ||
| @forward "usa-search"; |
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.
Why did you remove these?
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.
@thisisdano I show those as having been removed 8 months ago with the commit message "Add dependencies to component packages"
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'm going to restore them

Version one of the language selector component.