Skip to content

Recognize known html tags#8621

Merged
fisker merged 8 commits intoprettier:masterfrom
fisker:more-tags
Jul 14, 2020
Merged

Recognize known html tags#8621
fisker merged 8 commits intoprettier:masterfrom
fisker:more-tags

Conversation

@fisker
Copy link
Copy Markdown
Member

@fisker fisker commented Jun 23, 2020

  • I’ve added tests to confirm my change works.
  • (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory)
  • (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/pr-XXXX.md file following changelog_unreleased/TEMPLATE.md.
  • I’ve read the contributing guidelines.

Try the playground for this PR

><param name="autoplay" value="true" /><param
name="autoplay"
value="true" /></object

Copy link
Copy Markdown
Member Author

@fisker fisker Jun 23, 2020

Choose a reason for hiding this comment

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

This should be a unrelated bug.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@fisker fisker marked this pull request as ready for review June 23, 2020 05:08
Copy link
Copy Markdown
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Looks better

track: "block",
script: "block",
param: "block",
noscript: "none",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why none? How is it different from block?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I need update tests, current example not good.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

none seems same as inline, I'll check if noscript can set to block

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

On chrome, <noscript> is inline

image

@fisker fisker merged commit 26740d5 into prettier:master Jul 14, 2020
@fisker fisker deleted the more-tags branch July 14, 2020 14:02
@fisker
Copy link
Copy Markdown
Member Author

fisker commented Jul 14, 2020

Forgot snapshot outdated, fixing.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 15, 2021
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.

3 participants