Introduce an ItemGroup

[leolost2605]

Instead of having one big Gtk.Fixed with all of the launchers, separator, workspace, etc. introduce an ItemGroup class which holds the items of one group and add them together in a gtk.box. The ItemGroup works with a listmodel.
This has a couple advantages.

• the resize animation now finally applies again because right now it doesn't do anything since when a launcher gets added it never is the last item with the new workspace switcher.
• this addresses Increase the separation between app launchers and workspaces #511 a bit because the 2 pixels of the separator are now actually respected in the layout. If we want to address that more this will also be much easier because we can just add margins etc. because of the next point
• This simplifies layout handling because we don't have to iterate over multiple lists now and specially layout things like the separator and the dynamic workspace item because things at fixed positions are layout by the box and in the item group we just have a single list we need to layout
• Simplifies the repositioning logic because we just treat a reposition as remove + add

To reduce the diff the listmodels are still in the main itemmanager and just relay the current mechanisms to the item group via the listmodel. Therefore the following points will be for the future:

• The item groups being based on listmodels will allow us to simplify our item management. I.e. the app system and the workspace system can just provide a listmodel with their items and we don't need to go weird routes via connecting to added signals from the systems and to removed signals from the items which connect to removed signals to their apps/workspaces
• Syncing pinned launchers is right now split between appsystem and itemmanager and can be unified in the app system
• We can get rid of all of the type checking if else ifs

[danirabbit]

I'd like to get out this upcoming bug fix release before we make a big change just to make sure we have time to test for regressions :)

[leolost2605]

Ah yes I agree I actually wanted to say that somewhere but I forgot :|

[flodavid]

I tested this change, and the "Backgound Apps" group is always shown, even if there are no background apps currently. I do not think it should be the case.

[flodavid]

Can you apply this patch @leolost2605 ? Its to restore the background_item hiding/reappearing when background apps are removed/added

diff --git a/src/ItemManager.vala b/src/ItemManager.vala
index a1981d56..d65137d3 100644
--- a/src/ItemManager.vala
+++ b/src/ItemManager.vala
@@ -24,10 +24,14 @@
         launchers = new ListStore (typeof (Launcher));
 
         background_item = new BackgroundItem ();
-        background_item.apps_appeared.connect (add_item);
+        background_item.apps_appeared.connect (() => append (background_item));
+        background_item.removed.connect (() => {
+            if (!background_item.has_apps) {
+                remove (background_item);
+            }
+        });
 
         append (new ItemGroup (launchers));
-        append (background_item);
 
         overflow = VISIBLE;
 

 ``

[leolost2605]

@flodavid I think with your patch the background item would be at the end instead of where it's on main but it should be fixed now (I added an item group just for the background item to also make sure it gets smooth reveal/hide animation).

[flodavid]

Nice, thanks

[leolost2605]

Included in #553