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

Imperative Slot Assignment #55

Merged
merged 3 commits into from
Sep 13, 2022
Merged

Conversation

thescientist13
Copy link
Collaborator

Updated section re: Imperative Slot API.

Aside from an example and status links and per chats on the topic, mostly just focused on the timeline / roadmap for Safari to ship, as their feature tracker marks this API as supported.

But let me know if we think we should back-fill with all the historical content as well, concerns, dissenting opinions, etc.

@thescientist13 thescientist13 mentioned this pull request Sep 5, 2022
20 tasks
@thescientist13 thescientist13 changed the title Report 2022 imperative slot assignment Imperative Slot Assignment Sep 5, 2022
</section>
<section>
<h3>Key Scenarios</h3>
<p>---</p>
<p>Some of the scenarios called out in the proposal include not having to pre-compute the slot names ahead of time, as well as being able to conditionally load content into a slot.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would be cool if @cdata might be able to share whether this API does nice things for <model-viewer> dynamic slot situation…

@Westbrook Westbrook force-pushed the report-2022-imperative-slot-assignment branch from fe94fe3 to 38712d5 Compare September 13, 2022 23:10
@Westbrook Westbrook merged commit 65aba37 into main Sep 13, 2022
@Westbrook Westbrook deleted the report-2022-imperative-slot-assignment branch September 13, 2022 23:10
github-actions bot added a commit that referenced this pull request Sep 13, 2022
SHA: 65aba37
Reason: push, by @Westbrook

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants