Controllers architecture

[dab0mb]

Controllers architecture

[DAB0mB]

Hey @cray0000. I'm glad to see that people are enjoying my work :-) and thanks for the feedback. I've actually noticed these issues initially, but wasn't sure how I should go with the solution; but it was important for me to release a version asap.

Regards issue 1 - Your solution is pretty nice because it makes things much clearer. I would like to suggest another thing - import the view in the controller module and define the controller explicitly, like so:

import React from 'react'
import MyView from '../views/MyView'

class MyController extends React.Component {
  render() {
    return (
      <MyView />
    )
  }
}

// Notice how we match the view to the controller
export default MyView.Controller = MyController

What do you think of that? It's less strict than the first solution and much more flexible but can potentially cause a poor architecture by the user.

For issue 2 - I don't think that using the socket of an af-el should be mandatory. Sometimes we would like to hide some elements based on the component's state, like a dropdown menu. Also, my thought was enabling the usage of af-el generated components across any React component, it shouldn't be bound to a specific view, so it can be reused, for example - a navbar. I agree that something feels a bit odd with this behavior, but I would like to make some more thinking around it to have a better solution. If you have an alternative to that, do share!

Eytan

[DAB0mB]

I'll think what to do with it. Thank you for brinign it to my attention. Closing as it gets old