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

feat(language-selector): add vuestic-language-selector component #94

Merged
merged 1 commit into from
Feb 15, 2018

Conversation

yevdyko
Copy link
Contributor

@yevdyko yevdyko commented Jan 31, 2018

This PR adds language-selector component, includes it to navbar and removes Change Language section from User Profile dropdown in navbar. It uses flag-icon-css library for country flags in SVG.

@yevdyko yevdyko force-pushed the add-language-selector branch 5 times, most recently from 1791acf to 5f0c978 Compare February 6, 2018 01:21
@smartapant
Copy link
Contributor

@yevdyko language selection is empty on initial page load. Also, selected language shall be green when dd is shown

@yevdyko yevdyko force-pushed the add-language-selector branch 2 times, most recently from 7e96a7b to e12fdcf Compare February 7, 2018 00:53
@yevdyko
Copy link
Contributor Author

yevdyko commented Feb 7, 2018

@smartapant What about mobile screens? Should dropdown look like the desktop one or like other dropdowns in navbar?

@yevdyko
Copy link
Contributor Author

yevdyko commented Feb 7, 2018

@smartapant Should we use a fixed dropdown width or relative depending on the item text?

@smartapant
Copy link
Contributor

@yevdyko you could use min/max widths. As for mobile view - asked the designer. Will update you

@yevdyko yevdyko force-pushed the add-language-selector branch 4 times, most recently from 3544581 to baede39 Compare February 14, 2018 07:11
- Add vuestic-language-selector component
- Remove Change Language section from User Profile dropdown in navbar
- Add vuestic-language-selector to navbar.
- Add flag-icon-css library for country flags in SVG
@smartapant smartapant merged commit 81e200b into epicmaxco:master Feb 15, 2018
@yevdyko yevdyko deleted the add-language-selector branch February 17, 2018 14:17
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