-
Notifications
You must be signed in to change notification settings - Fork 37
AppHeader examples using slot property #3762
Copy link
Copy link
Open
Labels
DocumentationImprovements or additions to documentationImprovements or additions to documentationHeaderP2Priority 2 (should have): Improves completeness and reduces the post-launch support load.Priority 2 (should have): Improves completeness and reduces the post-launch support load.
Metadata
Metadata
Assignees
Labels
DocumentationImprovements or additions to documentationImprovements or additions to documentationHeaderP2Priority 2 (should have): Improves completeness and reduces the post-launch support load.Priority 2 (should have): Improves completeness and reduces the post-launch support load.
Type
Projects
Status
🔖 Ready
Info
Reported by Alex T
Our AppHeader examples are using a
slotproperty on some of our components, https://design.alberta.ca/components/app-header/#tab-1, specifically, Header with Navigation example.This works fine in Angular because Angular correctly identifies
slotas a global property available on all components. React however does not make this same identification and triggers an error on build, because theslotproperty isn't actually placed on the component in the wrapper.This was discussed here briefly in a PR.
Workaround
There is a workaround for this right now. People can surround the component using the
slotproperty with adivusing that same property and all of this will work as intended.Acceptance Criteria:
slotsas our other components currently use