This repository has been archived by the owner on Jun 29, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 11
Enhancement: FlowItem should give access to Panel's header-leading-content slot #958
Labels
Milestone
Comments
I would say just use the center slot |
Can they do that in FlowItem? |
Not currently, we only allow a string for that right now. We could add another slot. |
I'll add a bullet to our sync. |
asangma
added a commit
that referenced
this issue
May 11, 2020
asangma
added a commit
that referenced
this issue
May 11, 2020
asangma
added a commit
that referenced
this issue
May 11, 2020
asangma
added a commit
that referenced
this issue
May 11, 2020
@kat10140 Easiest way to test this one is to inspect "Item Three" and "Item Two" in |
Looks like the back button (calcite-action) is using that slot, if that's what i'm looking for? |
VURF! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
Right now, in FlowItem, we are taking over Panel’s 'header-leading-content' slot with the back button when there are more than 1 FlowItem’s in a Flow.
What do y’all think about retaining access to Panel’s
header-leading-content
slot available in FlowItem? We’d add the back-button as it’s own thing (i.e. not using a slot) or just “overload” theheader-leading-content
slot in Panel.Acceptance Criteria
No breaking changes.
Relevant Info
LayerSwitcher and future possibilities.
Which Component
FlowItem and Panel
Example Use Case
re: LayerSwitcher
Right now, Kevin will be using FlowItem’s
menu-actions
slot to add an Action to invoke the "layer switcher". This will work, but there might be design advantages to having the trigger on in the leading area:The text was updated successfully, but these errors were encountered: