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

Convert Attribution code block to tables #329

Merged
merged 6 commits into from
May 28, 2018

Conversation

wopian
Copy link
Collaborator

@wopian wopian commented Apr 5, 2018

Migrated the attribution section to tables for all languages to aid readability (particularly the approval list) as everything is ALL CAPS.

Unfortunately GFM doesn't allow header-less tables, so it's either an empty header:

or copyright as a header:

Before

After

@wopian wopian added Status: Review Needed Type: Meta Translations and other changes outside source code labels Apr 5, 2018
@wopian wopian requested a review from chrislgarry April 5, 2018 01:52
@wopian wopian requested a review from oldmud0 April 5, 2018 01:55
README.zh_cn.md Outdated
Norman E. Sears | Director, Mission Development<br>Apollo Guidance and Navigation Program | 1969年3月28日
Richard H. Battin | Director, Mission Development<br>Apollo Guidance and Navigation Program | 1969年3月28日
David G. Hoag | Director<br>Apollo Guidance and Navigation Program | 1969年3月28日
Ralph R. Ragan | Deputy Director<br>Instrumentation Laboratory | 1969年3月28日
Copy link
Collaborator

Choose a reason for hiding this comment

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

These fields were already localized in the target language, but they were pasted in English.

Copy link
Owner

Choose a reason for hiding this comment

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

Hmm looks so.

@chrislgarry
Copy link
Owner

Awesome work. Infinitely more readable. Lets have a look at @oldmud0 's comment and then we can merge.

@wopian
Copy link
Collaborator Author

wopian commented Apr 11, 2018

Roles did not keep their translations as the text was in all-caps and I don't know the capitalisation rules.

Comanche055 and Luminary099 are both derived from the Purpose in the original text and reworded, which without knowing the language I can't do without mutilating the translation. Which is better than a translation that no longer matches the source.

If this is 🆗'd the people who did the original translations can get mentioned to do the modifications correctly

@wopian wopian mentioned this pull request Apr 12, 2018
Added corrections and suggestions from PR #332.


제출됨: MARGARET H. HAMILTON 일시: 1969년 3월 28일
M.H.HAMILTON, 콜로서스 프로그래밍 리더
Copy link
Owner

Choose a reason for hiding this comment

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

We can copy and paste the roles from the original READMEs made by the translators, such as right here. I think "콜로서스 프로그래밍 리더" is the role. Or am I misunderstanding the issue.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The Korean readme already has the translated roles in this PR

@chrislgarry
Copy link
Owner

@wopian this still good?

@wopian
Copy link
Collaborator Author

wopian commented May 25, 2018

Good with me 👍

@chrislgarry chrislgarry merged commit be9b33e into master May 28, 2018
@wopian wopian deleted the feat/improve-attribution branch January 26, 2019 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Meta Translations and other changes outside source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants