-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
[Infra] Rethink addon primitives as contextual and not UI components #61
Comments
Now that modifiers are out, I think this would best be served with modifiers. I just made a pull request for ember sortable, very similar in the demands. Looking into this, easily doable except for liquid fire. Would have to drop support for animations. What would be better would be if liquid fire could supply modifiers. Ok looking into that if would be ember-animated now. Would it possibly be better for the end user to wrap panel.body in an animated-if themselves and remove the dependency on animate from this addon completely. |
Ok where this differs from ember sortable is the hiding of the body. A modifier cant really do that except to make the DOM element hidden. Not really the same thing. Where you are going to run into the problem with primitives that dont render anything is where to put the toggle. When it comes down to it a collapsible panel is an if and an isOpen variable. The panels component yields some functionality. |
So what would this look like. Im guessing the below? If user wanted animation they would replace The click me isnt an
I hate to say it but how far is that from below, which I believe is a collapsible panel
|
Currently this addon makes some opinionated choices about how the UI of the collapsible panels is rendered.
A better fit for this addon might be to provide contextual components that don't render anything but instead provide the primitives needed for collapsible panels without the UI.
The text was updated successfully, but these errors were encountered: