Skip to content

Remove uncovered topic from the introduction#38986

Merged
Josh-Cena merged 1 commit intomdn:mainfrom
youssef-el-atmani:patch-7
Apr 5, 2025
Merged

Remove uncovered topic from the introduction#38986
Josh-Cena merged 1 commit intomdn:mainfrom
youssef-el-atmani:patch-7

Conversation

@youssef-el-atmani
Copy link
Copy Markdown
Contributor

The introduction claims that the article will cover the description list element, but the article actually don't.

Description

Motivation

Additional details

Related issues and pull requests

The introduction claims that the article will cover the `description list` element, but the article actually don't.
@youssef-el-atmani youssef-el-atmani requested a review from a team as a code owner April 5, 2025 09:20
@youssef-el-atmani youssef-el-atmani requested review from hamishwillee and removed request for a team April 5, 2025 09:20
@github-actions github-actions bot added Content:Learn Learning area docs size/xs [PR only] 0-5 LoC changed labels Apr 5, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 5, 2025

Copy link
Copy Markdown
Member

@Josh-Cena Josh-Cena left a comment

Choose a reason for hiding this comment

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

Good catch, thanks

@Josh-Cena Josh-Cena merged commit 59d47ed into mdn:main Apr 5, 2025
7 checks passed
@youssef-el-atmani
Copy link
Copy Markdown
Contributor Author

Thanks @Josh-Cena,

I just checked the Structuring content with HTML section which it present all the tutorials that it will cover.
For each tutorial, it provide a brief overview. That overview is identical to the one included at the start of each tutorial.

Because of that identical overviews, I thought it may be generated automatically, but when I checked, I found that they are not identical anymore; the description list element still exit in the overview of Structuring content with HTML.

That leads me to the following PR #38999

wbamberg added a commit to wbamberg/content that referenced this pull request Apr 6, 2025
* upstream/main: (158 commits)
  fix: remove unrelated add() method reference from openCursor() doc (mdn#38998)
  Fix link (mdn#38995)
  Remove uncovered topic from a tuturial overview. (mdn#38999)
  Change the description of Temporal.Instant (mdn#38996)
  Remove `port` mention for URL `protocol` documentation (mdn#39000)
  Improve the clarity of a sentence (mdn#39003)
  Change if...else to non-zero on control_flow page (mdn#38992)
  Update group-data of EditContext API (mdn#38842)
  improve description (mdn#38845)
  fix: auto-cleanup by bot (mdn#38990)
  New pages: HTMLFormElement.rel and .relList (mdn#38900)
  Add live sample embeds for SVG path examples (mdn#38928)
  Remove uncovered topic from the introduction (mdn#38986)
  Remove self-links (mdn#38982)
  Synchronize with BCD v6.0.3 (mdn#38983)
  Fix grammar (mdn#38981)
  Remove external Web Speech API articles (mdn#38980)
  Add missing {{Specifications}} macros (mdn#38975)
  Remove browser-compat for CSS types without BCD (mdn#38970)
  Update async_function_star_ to have a working example (mdn#38972)
  ...
cssinate pushed a commit to cssinate/content that referenced this pull request Apr 11, 2025
The introduction claims that the article will cover the `description list` element, but the article actually don't.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Learn Learning area docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants