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

zh-ch: upate 'HTML table advanced features and accessibility' #23962

Merged
merged 5 commits into from
Oct 12, 2024

Conversation

abing22333
Copy link
Contributor

Description

upate 'HTML table advanced features and accessibility'

@abing22333 abing22333 requested a review from a team as a code owner October 10, 2024 13:00
@abing22333 abing22333 requested review from t7yang and removed request for a team October 10, 2024 13:00
@github-actions github-actions bot added the l10n-zh Issues related to Chinese content. label Oct 10, 2024
Copy link
Contributor

github-actions bot commented Oct 10, 2024

Preview URLs

Flaws (16)

URL: /zh-CN/docs/Learn/HTML/Tables/Advanced
Title: HTML 表格进阶特性和无障碍
Flaw count: 16

  • macros:
    • /zh-CN/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_events_state does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_events_state
    • /zh-CN/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_filtering_conditional_rendering does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_interactivity_filtering_conditional_rendering
    • /zh-CN/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_accessibility does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_accessibility
    • /zh-CN/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_resources does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/React_resources
    • /zh-CN/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Ember_getting_started does not exist but fell back to /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks/Ember_getting_started
    • and 10 more flaws omitted
  • translation_differences:
    • Differences in the important macros (2 in common of 4 possible)
External URLs (2)

URL: /zh-CN/docs/Learn/HTML/Tables/Advanced
Title: HTML 表格进阶特性和无障碍

(comment last updated: 2024-10-12 01:11:54)

@abing22333
Copy link
Contributor Author

Adding structure with <thead>, <tbody>, and <tfoot> 的开始,强调了要按顺序使用这三个元素,

To use them, they should be included in the following order:

  • The element must wrap the part of the table that is the header — this is usually the first row containing the column headings, but this is not necessarily always the case. If you are using / elements, the table header should come just below those.
  • The element needs to wrap the main part of the table content that isn't the table header or footer.
  • The element needs to wrap the part of the table that is the footer — this might be a final row with items in the previous rows summed, for example.

但是接下来的 Active learning: Adding table structure 中没有按顺序使用。

  1. First of all, make a local copy of spending-record.html and minimal-table.css in a new folder.
  2. Try opening it in a browser — You'll see that it looks OK, but it could stand to be improved. The "SUM" row that contains a summation of the spent amounts seems to be in the wrong place, and there are some details missing from the code.
  3. Put the obvious headers row inside a <thead> element, the "SUM" row inside a <tfoot> element, and the rest of the content inside a <tbody> element.

前后自相矛盾了,到时候抽空给 content 提个pr

files/zh-cn/learn/html/tables/advanced/index.md Outdated Show resolved Hide resolved
files/zh-cn/learn/html/tables/advanced/index.md Outdated Show resolved Hide resolved
files/zh-cn/learn/html/tables/advanced/index.md Outdated Show resolved Hide resolved
files/zh-cn/learn/html/tables/advanced/index.md Outdated Show resolved Hide resolved
files/zh-cn/learn/html/tables/advanced/index.md Outdated Show resolved Hide resolved
files/zh-cn/learn/html/tables/advanced/index.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@abing22333 abing22333 left a comment

Choose a reason for hiding this comment

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

fix bug

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.

👍

files/zh-cn/learn/html/tables/advanced/index.md Outdated Show resolved Hide resolved
@yin1999 yin1999 merged commit 166abc1 into mdn:main Oct 12, 2024
7 checks passed
@abing22333 abing22333 deleted the table_advanced branch November 11, 2024 13:42
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.

2 participants