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

doc: Translate some part into English #31

Merged
merged 2 commits into from
Nov 30, 2018

Conversation

DanSnow
Copy link
Contributor

@DanSnow DanSnow commented Nov 29, 2018

No description provided.

Copy link
Contributor

@JamesIves JamesIves left a comment

Choose a reason for hiding this comment

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

Some suggested improvements so it reads better.

README.md Outdated
NES.cssはCSSのみで構成されています。JavaScriptは不要です。


## Browser Support

NES.css is compatible with the newest version of below browsers.
Copy link
Contributor

Choose a reason for hiding this comment

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

Proposed:

NES.css is compatible with the newest version of the following browsers.

README.md Outdated
@@ -44,17 +47,23 @@ TODO: 詳細を書く

## CSS Only

NES.css only require CSS. You can use this without JavaScript.
Copy link
Contributor

Choose a reason for hiding this comment

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

Proposed:

NES.css only requires CSS and doesn't depend on any JavaScript.

README.md Outdated
## Usage

NES.css is only providing component. You can define your layout.
Copy link
Contributor

@JamesIves JamesIves Nov 29, 2018

Choose a reason for hiding this comment

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

Proposed:

NES.css only provides components. You will need to define your own layout.

Copy link
Contributor

Choose a reason for hiding this comment

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

  • You will need to define your own layout?

Copy link
Contributor

Choose a reason for hiding this comment

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

Woops, yep!

README.md Outdated
## Usage

NES.css is only providing component. You can define your layout.

Default font is `Press Start 2P`. When you using this framework with any language except English (link Japanese), please use another font as necessary.
Copy link
Contributor

Choose a reason for hiding this comment

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

Proposed:

The default font is Press Start 2P which only supports English characters. When you're using this framework with any language except English, please use another font.

Should link Japanese be an actual link?

Copy link

@jimmiehansson jimmiehansson Nov 29, 2018

Choose a reason for hiding this comment

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

It would be good to keep a more permanent link alternative available for this font

Copy link

@jimmiehansson jimmiehansson Nov 29, 2018

Choose a reason for hiding this comment

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

Here's the link from Dafont https://www.dafont.com/press-start-2p.font
Correction, authors link is better since this would be a better source for licensing as well.
https://www.zone38.net/font/

Copy link
Contributor Author

@DanSnow DanSnow Nov 30, 2018

Choose a reason for hiding this comment

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

That's a typo. It's "like", not "link". Sorry.

README.md Outdated
次のブラウザの最新バージョンをサポートしています。

* Chrome
* Firefox
* Safari

Not test
Copy link
Contributor

Choose a reason for hiding this comment

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

Proposed:

Untested

@BcRikko
Copy link
Member

BcRikko commented Nov 29, 2018

I'm not good at English, so I really appreciate this PR 😍

@DanSnow
Copy link
Contributor Author

DanSnow commented Nov 30, 2018

@JamesIves Thank for the suggests.

I have pushed a new commit, can you please check again?

@JamesIves
Copy link
Contributor

JamesIves commented Nov 30, 2018

Looks good to me!

@Baldomo
Copy link
Contributor

Baldomo commented Nov 30, 2018

Hey I (mistakenly, I didn't check properly beforehand) created a whole new PR for this, although I split the README into Japanese-only and English-only. Should I close it myself?

@BcRikko
Copy link
Member

BcRikko commented Nov 30, 2018

Thank you everyone 😍

@BcRikko
Copy link
Member

BcRikko commented Nov 30, 2018

@Baldomo You don't have to close #46 👍

@BcRikko BcRikko merged commit dc8c7a1 into nostalgic-css:master Nov 30, 2018
@BcRikko BcRikko added the documentation Adding, removing, or updating documentation label Dec 1, 2018
@BcRikko
Copy link
Member

BcRikko commented Dec 18, 2018

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Adding, removing, or updating documentation released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants