Replace actions with <slot> in `header_ci_component`
Description
Currently, the header_ci_component
takes a prop called actions
and renders different elements based on each action type... This is problematic for a couple of reasons, most notably:
- Unnecessary coupling between this component and clients of it. Why should it care what kind of actions are sent to it?
- Over complexity for something that's only used in one other place.
Proposal
Let's replace this with a <slot>
so that the client header_component
can fully own the rendering and behavior of these buttons. This would hopefully remove the need for the non-reactive actions
array managed by the header_component
.
Context
The actions are structured in a semi-generic way, while only being used in one place. This feels quite over-engineered.
/cc @fh1ch
Edited by Paul Slaughter