Skip to content

Remove css classes #7720

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 19 commits into from
Aug 9, 2021
Merged

Remove css classes #7720

merged 19 commits into from
Aug 9, 2021

Conversation

wbamberg
Copy link
Collaborator

@wbamberg wbamberg commented Aug 8, 2021

Part of #5865. This removes various things in the CSS docs that were stopping elements being converted to Markdown. These are mostly redundant classes, but also a few unconvertible elements like <u>.

I think the only thing that might be controversial here is that I've dealt with the originaldocinfo class, and associated structures, by just removing them. I don't see any point in including this any more, if there ever was. But if people feel that we should keep it, we could just make it a callout.

@wbamberg wbamberg requested a review from a team as a code owner August 8, 2021 22:39
@wbamberg wbamberg requested review from ericwbailey and removed request for a team August 8, 2021 22:39
@github-actions

This comment has been minimized.

@teoli2003 teoli2003 merged commit dc35b69 into mdn:main Aug 9, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 9, 2022
@wbamberg wbamberg deleted the remove-css-classes branch October 15, 2022 17:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants