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

Navbar Language Switch #380

Merged
merged 5 commits into from
Jul 13, 2023
Merged

Navbar Language Switch #380

merged 5 commits into from
Jul 13, 2023

Conversation

Prid13
Copy link
Collaborator

@Prid13 Prid13 commented Jul 7, 2023

Opted for "English" (instead of "EN").

It's a dropdown :)

image

image

Mobile

image

image

@jonny-wg2
Copy link
Contributor

Any chance you can make this flag icons? I find that it's quite large and doesn't looks the best.

@Prid13
Copy link
Collaborator Author

Prid13 commented Jul 11, 2023

Any chance you can make this flag icons? I find that it's quite large and doesn't looks the best.

Sure, but what flag should I use for English language? I actually thought of this idea, but then I got stuck on whether the UK or the US flag would fit best 😅

Also, should I use emojis (🇬🇧 | 🇯🇵) (they show up as "GB" and "JP" on Windows) or actual icons?

@torotimes
Copy link
Contributor

Hey both, why don't we just use the (🇬🇧 | 🇯🇵) flags for now and just deploy this? We can always amend later.

Wanna make that edit, @Prid13, and issue the PR?

I guess I kind of agree with Zain on the usability issues by selecting language automatically based on location.

Old icons had MIT license.

New icons are taken from Wikipedia, which uses Creative Commons license, and the flags are public domain as it says on Wikipedia.
Wasn't possible to add images/icons, so had to opt for emoji flags 😅 Non-existent in Windows though
@Prid13
Copy link
Collaborator Author

Prid13 commented Jul 12, 2023

Done :) On Windows the flags won't show up:

image

But they will on mobile. Something like this:

Screenshot_9823

@torotimes torotimes merged commit a40e5de into main Jul 13, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants