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

Remove untranslated and/or low translation locales from the locale chooser #4196

Closed
zackkrida opened this issue Apr 24, 2024 · 2 comments · Fixed by #4224
Closed

Remove untranslated and/or low translation locales from the locale chooser #4196

zackkrida opened this issue Apr 24, 2024 · 2 comments · Fixed by #4224
Assignees
Labels
♿️ aspect: a11y Concerns related to the project's accessibility ✨ goal: improvement Improvement to an existing user-facing feature 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend

Comments

@zackkrida
Copy link
Member

zackkrida commented Apr 24, 2024

Problem

In our locale dropdown which allows users to choose a translated version of Openverse, we show all available languages, regardless of whether or not these languages have any translations.

For users, this means they may choose their language from the dropdown and see the site fully in English, with a notice that we are seeking translation contributions. This is not an ideal user experience.

Ideally, we either:

  • Only show languages with some translations, regardless of the % completion of traslation Edit: Let's start this way, by removing any language with 0% translation.
  • Choose a "minimum % completed" to show a language

For maintainers, the number of translated pages we generate is also likely related to vue-router performance problems which are blocking the #411 project.

Description

We should either remove untranslated locales and or remove untranslated locales and filter translated locales down by a particular % translated in:

const valid = [...locales.translated, ...locales.untranslated]

Alternatives

Additional context

@zackkrida zackkrida added 🟨 priority: medium Not blocking but should be addressed soon ✨ goal: improvement Improvement to an existing user-facing feature ♿️ aspect: a11y Concerns related to the project's accessibility 🧱 stack: frontend Related to the Nuxt frontend labels Apr 24, 2024
@zackkrida zackkrida mentioned this issue Apr 24, 2024
6 tasks
@sarayourfriend
Copy link
Contributor

@zackkrida I can't remember from our discussion what we decided either way, but should we create an issue to follow up on this and bring a new call-to-action for contributing translations to Openverse? When we looked at the interactions with the current link, they were really low, so it's probably a good idea to consider different ways to encourage and make folks aware of the opportunity to contribute translations for languages that are not currently in the app. @obulat for example suggested using the accept header to see when a user's system is configured to request pages in a language for which we don't have translations, as an indication for when to show some call to action for helping with translations.

Mostly asking: was that something we agreed we should do? And if so should we get an issue for it. Or, is there a way to fold this into or after something like #3559 where we'll potentially have better tools for calls to action?

@zackkrida
Copy link
Member Author

zackkrida commented Apr 29, 2024

I think initially, let's hide any language with 0% translation.

As for a call to action for translations @sarayourfriend, we should definitely do that. It might be most elegant to do it in conjunction with @obulat's accept header recommendation, because it would allow us to show users a targeted banner for their language. It would take a bit of work to store a list of untranslated locales in the app state in order to know which languages to show the banner for, but it's definitely worth considering.

Issue created here: #4237

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♿️ aspect: a11y Concerns related to the project's accessibility ✨ goal: improvement Improvement to an existing user-facing feature 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants