Implement TreeView with patchable models

[owickstrom]

Implement TreeView with patchable models

[owickstrom]

Thanks for the proposal! I hope I can get some time this weekend to look more closely at it. 👍

[owickstrom]

Hey, sorry it took a while longer.

OK, so in this library I've deliberately not used the concept of models from GTK+. As you say, they effectively implement a bidrectional mutating data flow between the application state and the widgets, and I want gi-gtk-declarative to be unidirectional. The only way for information from the user interface to flow back up to the application logic is through events. (I should document this stuff in some sort of general guidelines for gi-gtk-declarative widget implementation.) I'm not sure I like the "unidirectional" terminology much, but it as at least established in the React world.

When it comes to the TreeView widget, it's likely more complicated than the previously implemented widgets, but can probably be done in a similar way as them. As an example, the Menu widget that's already in this library defines a few data types to model how menus can be built up. It may not cover all features in GTK+ menus (yet), but it fits with the library.

I propose you start by defining such data types for the TreeView widget, and build the Patchable and EventSource instances around that. Also, do leave out drag-and-drop and other advanced features for now, we can always add more events later on. If you do have any TreeView-specific events you want to report up to the user, define a data type for those events and emit them from your subscribe implementation(s).

Does that make sense? Please ask more questions if you have any. I'm glad your here and want to help implement more widget support! 🙂

[eamsden]

@owickstrom I'm interested in implementing this but it definitely requires discussion and your sign-off on the design before I can start working on it.

[eamsden]

The idea of a model underlying a widget recurs in GTK, for instance, GtkMenu and GtkMenuBar, as subclasses of GtkMenuShell, can take GMenuModel (defined in the gio lib with a very abstract and opaque concept of actions) as a model, and update themselves in response to changes in the model in a similar way to GtkTreeView with a GtkTreeModel. If we can figure this out on TreeView then I think we will have a good example to work from when implementing other widgets with underlying models.

[eamsden]

One subtle thing here is that the widget itself can update the model. Users can drag-and-drop to reorder rows, rows can be editable, etc. We need some way to feed this back into the application state (via events probably) without the changes in the application state immediately triggering another update, causing model inconsistency and infinite loops.

[eamsden]

@owickstrom Have you gotten a chance to look at this more closely? I'd really like to start implementing it.