Skip to content

zh-cn: add "HTML Cheatsheet" #27816

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

Merged
merged 5 commits into from
Aug 15, 2025
Merged

zh-cn: add "HTML Cheatsheet" #27816

merged 5 commits into from
Aug 15, 2025

Conversation

Yanko1013
Copy link
Contributor

Description

请指教!

Please advise!

@Yanko1013 Yanko1013 requested a review from a team as a code owner June 25, 2025 15:35
@Yanko1013 Yanko1013 requested review from t7yang and removed request for a team June 25, 2025 15:35
@github-actions github-actions bot added the l10n-zh Issues related to Chinese content. label Jun 25, 2025
@yin1999
Copy link
Member

yin1999 commented Jun 26, 2025

@mdn/core-dev Is there anyone can fix the issue happened in CI

image

@caugner
Copy link
Contributor

caugner commented Jun 26, 2025

@mdn/core-dev Is there anyone can fix the issue happened in CI

Thanks for the ping. #27822 should resolve this.

Copy link
Contributor

Copy link
Member

@yin1999 yin1999 left a comment

Choose a reason for hiding this comment

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

Some suggestions

Comment on lines 54 to 56
Used to group elements: for example,
to <span style="color:blue">style
them&#x3C;/span>.</pre
Copy link
Member

Choose a reason for hiding this comment

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

Please translate this example

<td>强调文本</td>
<td>{{HTMLElement("em")}}</td>
<td id="em-example">
<pre class="brush: html">&#x3C;em>I'm posh&#x3C;/em>.</pre>
Copy link
Member

Choose a reason for hiding this comment

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

+1

<td>{{HTMLElement("i")}}</td>
<td id="i-example">
<pre class="brush: html">
Mark a phrase in &#x3C;i>italics&#x3C;/i>.</pre
Copy link
Member

Choose a reason for hiding this comment

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

+1

Copy link
Member

Choose a reason for hiding this comment

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

Please remove this image, the build platform automatically references assets in en-US

Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Aug 14, 2025
@github-actions github-actions bot removed the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Aug 15, 2025
@yin1999 yin1999 changed the title zh-cn: add "HTML Cheatsheet" & "HTML Guides" zh-cn: add "HTML Cheatsheet" Aug 15, 2025
Copy link
Member

@yin1999 yin1999 left a comment

Choose a reason for hiding this comment

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

Reviewed locally. Thank you @Yanko1013.

@yin1999 yin1999 merged commit 3950572 into mdn:main Aug 15, 2025
5 of 6 checks passed
@github-project-automation github-project-automation bot moved this from Review requested to Done in MDN translated-content l10n-zh Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-zh Issues related to Chinese content.
Projects
Development

Successfully merging this pull request may close these issues.

3 participants