Skip to content

Conversation

@microbit-mark
Copy link
Contributor

@microbit-carlos
Copy link
Collaborator

Can we restructure the docs to move this section to its own header that clearly targets the translators?

The current section does explain what they need to do, but it is kind of buried in a larger section that doesn't really apply the people clicking this link, so it can be a bit intimidating to find so much text.

https://github.com/bbcmicrobit/PythonEditor/blob/v2-beta/docs/translations.md#12-translating-the-editor

To be honest, it might be better if it was it's own file, so that it is very clear what they need to do and they don't have to worry about all the other stuff.


Also, I've added added a small CSS line to increase add the space between the language list and the link:

Before After
Screenshot 2019-10-14 at 13 28 35 image

@microbit-carlos
Copy link
Collaborator

Also, I take it this needs a roundtrip on Crowdin?

@microbit-mark
Copy link
Contributor Author

microbit-mark commented Oct 16, 2019

Thanks Carlos

To be honest, it might be better if it was it's own file, so that it is very clear what they need to do and they don't have to worry about all the other stuff.

In that case, I think it would be better as a KB article, 'Translating the micro:bit Python Editor' that has images/videos of the process and that links out to the Translations Release Strategy as that doc serves a different purpose. Sound ok?

@microbit-carlos
Copy link
Collaborator

Sure!

@microbit-mark
Copy link
Contributor Author

As discussed, updated the link to a placeholder KB article. If that's okay @microbit-carlos I will do the Crowdin round trip and then it will be ready to merge

Copy link
Collaborator

@microbit-carlos microbit-carlos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks Mark!

@microbit-carlos microbit-carlos merged commit 82429f5 into bbcmicrobit:v2-beta Oct 17, 2019
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