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

Highlight code blocks, fix emphasis #712

Merged
merged 3 commits into from
Dec 4, 2017
Merged

Highlight code blocks, fix emphasis #712

merged 3 commits into from
Dec 4, 2017

Conversation

wiinci
Copy link
Contributor

@wiinci wiinci commented Nov 20, 2017

No description provided.

@wiinci
Copy link
Contributor Author

wiinci commented Nov 20, 2017

Marked as non substantive for IPR from ash-nazg.

@@ -397,7 +408,7 @@ The `createInstance(optional any state)` method on `HTMLTemplateElement`, when
4. Let *parts* be an empty list.
5. For every [descendent](https://dom.spec.whatwg.org/#concept-tree-descendant) node *currentNode* of *instance* in [tree order](https://dom.spec.whatwg.org/#concept-tree-order), run these steps:
1. If *currentNode* is a [template element](https://html.spec.whatwg.org/multipage/scripting.html#the-template-element):
1. Run the concept to _adjust single node case_ with *currentNode***.**
1. Run the concept to _adjust single node case_ with *currentNode.*
Copy link
Collaborator

Choose a reason for hiding this comment

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

. Should appear after *.

<section><h1>Ryosuke Niwa</h1>Email: <a href="mailto:[email protected]”>[email protected]</a></section>
``` html
<section>
<h1>Ryosuke Niwa</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please don't reformat examples like this. Whitespaces after h1 Before "Email:" would not be collapsed so it would affect the final DOM tree.

<section><h1>Ryosuke Niwa</h1>Email: <a href="mailto:[email protected]”>[email protected]</a></section>
``` html
<section>
<h1>Ryosuke Niwa</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ditto.

<template id="foo"><div class="foo {{ f(y) }}">{{ x }} world</div></template>
``` html
<template id="foo">
<div class="foo {{ f(y) }}">{{ x }} world</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ditto.

<template type="my-template-type" id="contactTemplate">
<section><h1>{{name}}</h1>Email: <a href="mailto:{{email}}">{{email}}</a></section>
<section>
<h1>{{name}}</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please undo this reformatting.

@rniwa rniwa merged commit 8f353b6 into WICG:gh-pages Dec 4, 2017
@rniwa
Copy link
Collaborator

rniwa commented Dec 4, 2017

Thanks for the fixes!

@wiinci wiinci deleted the patch-1 branch April 2, 2018 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants