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

Remove duplicate list in glossary article #28552

Merged
merged 2 commits into from
Aug 14, 2023
Merged

Remove duplicate list in glossary article #28552

merged 2 commits into from
Aug 14, 2023

Conversation

glmvc
Copy link
Contributor

@glmvc glmvc commented Aug 14, 2023

Description

Remove duplicate list of some semantic HTML elements within the "See also" section.

Motivation

I don't think that the list is needed a second time.
There's already the same listing in a separate section right before and linking to the HTML element reference within "See also" should be sufficient.

Additional details

/

Related issues and pull requests

/

@glmvc glmvc requested a review from a team as a code owner August 14, 2023 08:20
@glmvc glmvc requested review from hamishwillee and removed request for a team August 14, 2023 08:20
@github-actions github-actions bot added the Content:Glossary Glossary entries label Aug 14, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 14, 2023

Preview URLs

(comment last updated: 2023-08-14 08:27:31)

@bsmth bsmth self-assigned this Aug 14, 2023
@bsmth
Copy link
Member

bsmth commented Aug 14, 2023

Taking a look now 👀

Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Thanks a lot @glmvc - I think this makes sense 👍🏻

@bsmth bsmth merged commit 8578969 into mdn:main Aug 14, 2023
6 checks passed
@glmvc glmvc deleted the glmvc-patch-1 branch August 14, 2023 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Glossary Glossary entries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants