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

Update README.jp.md #754

Merged
merged 1 commit into from Apr 18, 2017
Merged

Update README.jp.md #754

merged 1 commit into from Apr 18, 2017

Conversation

kuboon
Copy link
Contributor

@kuboon kuboon commented Mar 22, 2017

fix translation about single quote

README.jp.md Outdated

シングルクォートは Slim に行をコピーしろと命じます (`|` と同様に) が, 単一行の末尾にスペースが追加されているかチェックします
シングルクォートは Slim に行をコピーしろと命じます (`|` と同様に) が, 末尾にスペースが(なければ)追加されます
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for your PR :)

Slim 3.0.7 always appends trailing white space, even if original text has trailing white space.
So I feel "末尾にスペースが追加されます。" is better.

body
  p
    ' This line does not have trailing white space.
  p
    ' This line has trailing white space. 
$ slimrb sample.smil
<body><p>This line does not have trailing white space. </p><p>This line has trailing white space.  </p></body>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed.
Now Japanese readme is better than English one ;)

@yui-knk
Copy link
Member

yui-knk commented Mar 23, 2017

Can you rebase it to merge one commit?

fix translation about single quote
> Slim 3.0.7 always appends trailing white space, even if original text has trailing white space.
@kuboon
Copy link
Contributor Author

kuboon commented Apr 14, 2017

@yui-knk merged to 1 commit. plz check.

@yui-knk
Copy link
Member

yui-knk commented Apr 18, 2017

Thanks

@yui-knk yui-knk merged commit 14efec2 into slim-template:master Apr 18, 2017
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.

None yet

2 participants