-
Notifications
You must be signed in to change notification settings - Fork 59
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
Translate bidi-xhtml into zh-hans #513
Conversation
✅ Deploy Preview for i18n-drafts ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -67,7 +67,6 @@ <h2 class="notoc">Objectives</h2> | |||
|
|||
|
|||
<div id="audience"> | |||
<p><span id="intendedAudience" class="leadin">Intended audience:</span> HTML/XHTML and CSS content authors implementing pages in right-to-left scripts such as Arabic and Hebrew, or having to deal with embedded right-to-left script text. This material is applicable whether you create documents in an editor, or via scripting.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know we have been removing this at the top of articles, but i left it in place for the tutorials because it's common practice to describe the intended audience for a tutorial or training session.
We should mention who should read this somewhere. We could either leave the text in place, or we could add a paragraph to the introduction.
It may be good, however, to remove "/XHTML". I think the final sentence can also go (ie. This material is applicable...).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough. I added it back. Thanks.
The links are not very obvious. I think it may help to add the following style for the tutorials:
|
Merging. Copied to CVS as well, including the updated Thank you very much! |
Preview