<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hello guys!<div>We are about to merge to a WIP branch (it would be some temporary branch for new UI) new UI for skydive. It gives a huge flexibility to define multiple layouts BUT since UI in skydive is very huge and has a lot of use cases Sylvain mentioned it would be risky to merge UI directly to the master.</div><div>New ui right now placed in this pull request: <a href="https://github.com/skydive-project/skydive/pull/1443#issuecomment-439313769">https://github.com/skydive-project/skydive/pull/1443#issuecomment-439313769</a></div><div>I'd like to ask you to do few things:</div><div>1. when you get time please switch to this pull request (you may get it from my repository) and try to test your use cases on this pull request. please take a special attention to how the layout reacts to layout realtime changes: added node, edge, etc. Try to play with it.</div><div>2. Please let me know if you need to do any changes in UI. </div><div>Sometimes developers may think that UI is not so serious layer of the project. But I can show you a simple errors made in the current UI implementation:</div><div><br clear="all"><div>For instance, <a href="https://github.com/skydive-project/skydive/pull/1443/files#diff-888602f29f79038e04b62f0a7b9aefbeL37">https://github.com/skydive-project/skydive/pull/1443/files#diff-888602f29f79038e04b62f0a7b9aefbeL37</a> :</div><div>it is a simple problem of responsibilities of the layout component: layout component should know nothing about how our application works. So, for this thing we can use pattern: Bridge, Observer.</div><div>We need to have some event observer, emitter to be able to react on some events: for instance, you can see how it is implemented in my revised changeset:</div><div><a href="https://github.com/skydive-project/skydive/pull/1443/files#diff-888602f29f79038e04b62f0a7b9aefbeR37">https://github.com/skydive-project/skydive/pull/1443/files#diff-888602f29f79038e04b62f0a7b9aefbeR37</a><br></div><div><br></div><div>This is very important for us because we want to have a very easily modified gui, etc.</div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Best, Sergey</div></div></div></div></div></div></div></div>