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

For Chinese publications, line head indent at the beginning of a paragraph usually uses two character width spaces #86

Closed
dotmonkey opened this issue May 8, 2020 · 2 comments · Fixed by #87

Comments

@dotmonkey
Copy link

Short description of the issue/suggestion:

For Chinese publications, line head indent at the beginning of a paragraph usually uses two character width spaces. Publications like magazines, with multi-column content and less text in each column, might apply a single character width line head indent at the beginning of paragraph as well.

Steps to reproduce the issue/enhancement:
None.

What is the expected behaviour?
Line head indent at the beginning of a paragraph is applied to all paragraphs. Nearly all books and magazines make use of this method.

What is the current behaviour?

Line head indent does not apply to the first paragraph but to the rest of the paragraphs. This method is mostly seen in Western publications.

reference document:
https://www.w3.org/TR/clreq/#line_head_indent_at_the_beginning_of_paragraphs

@JayPanoz
Copy link
Collaborator

JayPanoz commented May 8, 2020

Thanks for catching this.

I can see I should correct:

Will do later today in the develop branch and if I may, cc you so that you can check it’s fine for merging in master.

@dotmonkey
Copy link
Author

Thanks for catching this.

I can see I should correct:

Will do later today in the develop branch and if I may, cc you so that you can check it’s fine for merging in master.

I have test it, looks good. Thank you.

JayPanoz added a commit that referenced this issue May 16, 2020
This corrects the indent in default stylesheets for CJK (i.e. if book doesn’t have any author stylesheet). Closes #86
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 a pull request may close this issue.

2 participants