JarekToro / responsive-layout

The layout to beat them all!
https://vaadin.com/directory#!addon/responsive-layout
57 stars 24 forks source link

added interface SingleComponentContainer to ResponsiveColumn #34

Closed derBertW closed 7 years ago

derBertW commented 7 years ago

resolves JarekToro/responsive-layout#33

JarekToro commented 7 years ago

Looks Good. With regards the "ComponentAttachDetachNotifier", Seems like a listener mostly used by Vaadin Core SDK not really Users (Developers). Is that correct?

derBertW commented 7 years ago

That should be allright now. As a "Component" which now officially can contain a "Component" there must be a way to send events through the hirarchy. And we are using the implementation from "AbstractClientConnector" which is part of the class hirarchy anyway.

JarekToro commented 7 years ago

Great, thank you.