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

i18n: change selection style #41

Merged
merged 2 commits into from
Sep 25, 2024
Merged

i18n: change selection style #41

merged 2 commits into from
Sep 25, 2024

Conversation

mwarning
Copy link
Collaborator

@mwarning mwarning commented Sep 23, 2024

  • Do not show english name for selected language to keep selected item shorter.
  • Remove select width hack with data-width attribute.
  • Also replace drop down arrow with language symbol.
  • Handle small screen sizes slightly better.

@mwarning mwarning changed the title i18n: change selection style [draft] i18n: change selection style Sep 23, 2024
@mwarning mwarning force-pushed the language_selection branch 4 times, most recently from 0c98c93 to 1bfd9d1 Compare September 24, 2024 18:26
@mwarning mwarning changed the title [draft] i18n: change selection style i18n: change selection style Sep 24, 2024
@mwarning
Copy link
Collaborator Author

mwarning commented Sep 24, 2024

gscreenshot_2024-09-24-213033

@aparcar
Copy link
Member

aparcar commented Sep 24, 2024

Fine with me

* Remove english name for selected language to keep selected language shorter.
* Remove workaround using data-width attribute.
* Replace drop down arrow with language symbol.
* Handle small screen sizes slightly better.

Signed-off-by: Moritz Warning <[email protected]>
Signed-off-by: Moritz Warning <[email protected]>
@mwarning mwarning merged commit 9fd29f9 into master Sep 25, 2024
2 checks passed
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