@christian, could you take a look at this CL overall and its dependency chain?
@domfarolino and @foolip, could you take a look at the following files specifically:
But also sanity check more generally the different architectural approaches I explored here [1] and here [2]?
There are 5 CLs that build on top of this one to implement [2] incrementally, and I could use some high level guidance if this is directionally correct.
[1] Implementation options https://docs.google.com/document/d/1Zevn7l93U8Kj3hqr_AanifIRMDuJMy4mMyExGVY96v8/edit?resourcekey=0-Bwk0c3EyEKP3p0NtDFR1fw&tab=t.0#heading=h.gpzmcty9cy9a
[2] Web Platform API options https://github.com/fedidcg/federation-element
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
although I would have added the html_tag_names.json5 change to this CL as well
| Code-Review | +1 |
lgtm but probably best to let someone who knows this better than me review this as well (maybe mason?)
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
LGTM
[Can you add a link to the public proposal right above here?
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Can you add a link to the public proposal right above here?
Oh yeah, that's a great idea, done!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
7 is the latest approved patch-set.
The change was submitted with unreviewed changes in the following files:
```
The name of the file: third_party/blink/renderer/core/html/html_federation_element.idl
Insertions: 1, Deletions: 0.
@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+// Proposal https://github.com/fedidcg/federation-element
[
Exposed=Window,
RuntimeEnabled=FederationElement
```
[fedcm] Introduce the <federation> element (Blink Skeleton)
This CL adds the minimal plumbing for the <federation> element in Blink,
including the runtime feature flag, tag registration, and basic
HTMLFederationElement class.
Explainer:
https://github.com/fedidcg/federation-element
Intent To Prototype:
https://groups.google.com/a/chromium.org/g/blink-dev/c/uXmdPPN2NUI/m/DpbG_eKBCQAJ
Design Doc:
https://docs.google.com/document/d/1Zevn7l93U8Kj3hqr_AanifIRMDuJMy4mMyExGVY96v8/edit?resourcekey=0-Bwk0c3EyEKP3p0NtDFR1fw&tab=t.0
This is the first in a series of CLs to introduce a native <federation> element.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |