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

Update Declarative Custom Element section #33

Closed
wants to merge 1 commit into from

Conversation

sashafirsov
Copy link

@sashafirsov sashafirsov commented Aug 8, 2022

Link to initial WCCG discussion issue.
image

Link to initial WCCG discussion issue.
@@ -435,6 +435,7 @@ <h3>Motivation</h3>
<h2>Declarative custom elements</h2>
<dl>
<dt>GitHub Issue(s):</dt>
<dd><a href="https://github.com/w3c/webcomponents-cg/issues/32">WICG/webcomponents#187</a></dd>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I will double check, but does your issue reference the strawman proposal at all (what is linked right below)? I was looking but didn't see it.

I think an issue for this is good, but I think that would be really important to call out the strawman from the description of your issue and declare if you are building off of it, or this proposal is an entirely net new just to help set context, IMO.

I would also second the comment from Justin in that issue that it would probably make sense to have the conversation over in https://github.com/WICG/webcomponents repo, since that is where the current proposal resides.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also might be good to sync with @EisenbergEffect , who is also making some updates to this section / spec, per #41. 👍

Copy link
Author

Choose a reason for hiding this comment

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

I have not found the strawman link, the communication on what to collect meant to be on inserted proposal issue which has to be referenced in this status doc. Otherwise trace-ability is lost.

Copy link
Author

Choose a reason for hiding this comment

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

@thescientist13 , linked issue #32 is updated with strawman and other links from @EisenbergEffect PR #41

Copy link
Author

Choose a reason for hiding this comment

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

issue that it would probably make sense to have the conversation over in https://github.com/WICG/webcomponents repo

The purpose of the issue is to serve the WCCG tracing around proposal and its status in own format, it does not fit into WICG purpose which is features discussions. I agree that WICG would benefit from summary and progress on particular feature, in slightly different format though. CC-in on Slack.

Copy link
Collaborator

Choose a reason for hiding this comment

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

So I think then we would be good if #41 just includes the link to #32 and then it would be part of our current (2022) report? Then I think we could close this one down since I don't think we're doing any back porting of older reports, but maybe that is something to corroborate with @Westbrook .

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, please. We can close this once all relevant info is moved to the new PR.

@thescientist13
Copy link
Collaborator

So looks like #41 landed with this link! So we can close this one down I think 👍

@sashafirsov sashafirsov deleted the patch-1 branch August 27, 2022 23:57
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.

3 participants