-
Notifications
You must be signed in to change notification settings - Fork 168
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
Update to Lit 2.1 API documentation #623
Conversation
Includes queryAssignedElements.
A live preview of this PR will be available at the URL(s) below. https://pr623-b0bd433---lit-dev-5ftespv5na-uc.a.run.app/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, but I did notice that the options.slot
and options.selector
parameters don't get documented.
I guess TypeDoc must consider those invalid, since they don't match any actual parameter.
It's slightly confusing right now, because the options
descriptions just tell you to look at MDN, but the slot
and selector
options aren't documented there.
Also, we should merge this just after the release, I assume?
There is a confusingly link |
Oh, true! I think it was the phrasing of "See MDN parameters section for available options" that sounded authoritative -- like that was the only place I should look. Also the elements decorator says "Also accepts two more optional properties, slot and selector.", but the nodes one doesn't have a similar sentence. |
This is a great comment. Will address on Lit core. |
…ryAssignedNodes. Raised in lit/lit.dev#623
…ryAssignedNodes. Raised in lit/lit.dev#623
Thanks Al! Addressed your changes on Lit and updated the Lit 2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we be waiting to merge these PRs until we do the release? The docs won't match the released version otherwise, right?
Right! Blocked on Lit 2.1 release. |
* ocument queryAssignedElements first pass. Addresses issues: - #607 - #614 * Edit documentation * Address feedback from Al. Also fix a bug. * Update packages/lit-dev-content/site/docs/components/shadow-dom.md Co-authored-by: Arthur Evans <arthure@google.com> * Update packages/lit-dev-content/site/docs/components/shadow-dom.md Co-authored-by: Arthur Evans <arthure@google.com> * Arthur's feedback - make options into a table. Co-authored-by: Andrew Jakubowicz <ajakubowicz@google.com> Co-authored-by: Arthur Evans <arthure@google.com>
Bump generated API docs so they include new Lit 2.1 features.
Example of a new section: https://pr623-b0bd433---lit-dev-5ftespv5na-uc.a.run.app/docs/api/decorators/#queryAssignedElements
Will merge after Lit 2.1 release.