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

Mod: Easier cycling through translations #164

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

cadesalaberry
Copy link
Contributor

In our process to facilitate the use of this website by external translators, we came up with some changes that might help the overall feel on the l10ns web interface.

This is only part of our changes, but we thought it might interest you.

Let me know what you think !

Here are the proposed designs:
06
04
05
03
02
01

@tinganho
Copy link
Owner

I really appreciate these changes. I'm planning to add some simpler flow regarding switching languages and input multiple languages.

I think you can keep your fork, because I'm not planning on adding new features on the current version. As said, I really appreciate your PR.

I hope you don't mind, if I ask which company you are working for that is using L10ns?

@cadesalaberry
Copy link
Contributor Author

Thanks,

We use it at SPARTED to handle some translation in our application.

I understand you don't want to maintain an old version for you are working on another one right now.

If you don't mind, could you still have a look at the PR, and add some suggestions where you think we could have done otherwise ?

No worries if you don't have the time, I just thought you were the most skilled person to guide us on your project 😸

Cheers

@qborreda
Copy link

While you are at it, the search feature breaks (offers no results) trying to search for one of the proposed formats of string identifiers.

Searching for something with a _ makes it unable to show possible results .. in example GENERIC->ASYNC_ERROR

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.

6 participants