Skip to content

Conversation

@GlennGeelen
Copy link
Owner

@GlennGeelen GlennGeelen commented Jun 23, 2020

Update of live view in cells to 0.13.3.

We need to merge this into master when Elements PR https://github.com/DefactoSoftware/elements/pull/498 is merged.

The most important thing that changed is the view adapter.
Phoenix.LiveView => Phoenix.LiveView.Helpers without that live_cell cannot be used.

It feels like with LiveView the projects: ex_cell and elements are more tightly coupled than we want.
@fatboypunk @sn3p
If you guys have any ideas how to get this more loose coupled, let me know I will try to implement that.

@GlennGeelen GlennGeelen force-pushed the glenn/live-cells-0.13.3 branch from b5936ec to 3148d47 Compare July 8, 2020 12:29
@GlennGeelen GlennGeelen closed this Feb 1, 2021
@sn3p
Copy link

sn3p commented Feb 1, 2021

It feels like with LiveView the projects: ex_cell and elements are more tightly coupled than we want.
@fatboypunk @sn3p
If you guys have any ideas how to get this more loose coupled, let me know I will try to implement that.

For me it's still hard to see what is the best way to do this. But I feel this is valuable exploring and I'm happy we're starting to implement LV's. So we will eventually find the best approach and work our way to fit it in our toolbelt, and see if we can make it play nice with ex_cell.

@GlennGeelen
Copy link
Owner Author

This is only the outdated upgrade of LV, the actual branch that implements LV is on the ex_cell repo :) That one can now be updated because Elements is no longer using the separate branch for live_cell.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants