Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Feb 3, 2019

@ghost ghost requested a review from justinfagnani as a code owner February 3, 2019 08:01
@ghost ghost changed the title Revise per feedback on "try LitElement" tutorial [docs] Revise per feedback on "try LitElement" tutorial Feb 3, 2019
@ghost ghost changed the title [docs] Revise per feedback on "try LitElement" tutorial [docs] Try LitElement: Update tutorial Feb 5, 2019

To loop over the new `myArray` property, add the following code to your template:

```js
Copy link
Contributor

Choose a reason for hiding this comment

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

this is HTML at the top level here. I think you need to add a `html`` to make it highlight correctly.

To loop over the new `myArray` property, add the following code to your template:

```js
<ul>${this.myArray.map(item => html`<li>${item}</li>`)}</ul>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I might put the expression on a new line for readability.

@ghost ghost merged commit 62f229e into master Feb 5, 2019
@ghost ghost deleted the fix-try branch March 25, 2019 19:12
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants