Skip to content

Commit

Permalink
Fix NOTE and WARNING
Browse files Browse the repository at this point in the history
  • Loading branch information
graynorton committed Oct 1, 2024
1 parent 075f03d commit c927854
Showing 1 changed file with 13 additions and 11 deletions.
24 changes: 13 additions & 11 deletions packages/lit-dev-content/site/docs/v3/data/signals.md
Original file line number Diff line number Diff line change
Expand Up @@ -244,12 +244,12 @@ The performance savings of `watch()` will tend to scale with the amount of
template logic and the number of bindings that can be skipped in an update, so
this will be more significant in templates with lots of logic and bindings.

> [!NOTE]
>
> <!-- keep Prettier at bay -->
>
> `@lit-labs/signals` does not contain a signal-aware `repeat()` directive yet.
> Changes to the contents arrays will perform a full render until then.
<div class="alert alert-info">

`@lit-labs/signals` does not contain a signal-aware `repeat()` directive yet.
Changes to the contents arrays will perform a full render until then.

</div>

#### Auto-pinpoint updates with the signals `html` template tag

Expand All @@ -275,11 +275,13 @@ import {SignalWatcher, html, signal} from '@lit-labs/signals';
}
```

> [!WARNING]
> <!-- Keep Prettier at bay -->
> The signals `html` tag doesn't yet work well with lit-analyzer. The analyzer
> will report type errors on bindings that use signals becuase it sees an
> assigment of `Signal<T>` to `T`.
<div class="alert alert-warning">

The signals `html` tag doesn't yet work well with lit-analyzer. The analyzer
will report type errors on bindings that use signals becuase it sees an
assigment of `Signal<T>` to `T`.

</div>

## Ensuring proper polyfill installation

Expand Down

0 comments on commit c927854

Please sign in to comment.