Merge lp:~kamstrup/libunity/libunity.fix-841902 into lp:libunity

[Mikkel Kamstrup Erlandsen]

Adds a new Filter type FilterCheckOptionsCompact that indicates to Unity that it should use a compact layout to render the tiles. Right not that specifically means 'use 3 columns'. Compared to Mirco's work I changed the value of the enum to match the class names and string names used in FilterOptionsCompact. That is, not using notation reverse polish.

This is a rework of Mirco's branch lp:~macslow/libunity/libunity.fix-841902.

[Michal Hruby (mhr3)]

Thanks, Mirco will be happy to have this in. :)

review: Approve

[Mikkel Kamstrup Erlandsen (kamstrup)]

Finally! It is in! :-D

=== modified file 'src/unity-lens-filters.vala'
--- src/unity-lens-filters.vala	2012-01-09 15:32:37 +0000
+++ src/unity-lens-filters.vala	2012-01-17 19:38:31 +0000
@@ -39,7 +39,8 @@
   CHECK_OPTIONS = 0,
   RADIO_OPTIONS,
   MULTIRANGE,
-  RATINGS
+  RATINGS,
+  CHECK_OPTIONS_COMPACT
 }
 
 /*
@@ -94,6 +95,7 @@
     switch (renderer)
     {
       case FilterRenderer.CHECK_OPTIONS: return "filter-checkoption";
+      case FilterRenderer.CHECK_OPTIONS_COMPACT: return "filter-checkoption-compact";
       case FilterRenderer.RADIO_OPTIONS: return "filter-radiooption";
       case FilterRenderer.RATINGS: return "filter-ratings";
       case FilterRenderer.MULTIRANGE: return "filter-multirange";
@@ -106,6 +108,7 @@
     switch (renderer_name)
     {
       case "filter-checkoption": return FilterRenderer.CHECK_OPTIONS;
+      case "filter-checkoption-compact": return FilterRenderer.CHECK_OPTIONS_COMPACT;
       case "filter-radiooption": return FilterRenderer.RADIO_OPTIONS;
       case "filter-ratings": return FilterRenderer.RATINGS;
       case "filter-multirange": return FilterRenderer.MULTIRANGE;
@@ -351,6 +354,20 @@
   }
 }
 
+public class CheckOptionFilterCompact : OptionsFilter
+{
+  public CheckOptionFilterCompact (string id,
+                                   string display_name,
+                                   Icon? icon_hint=null,
+                                   bool collapsed=false)
+  {
+    Object (id:id, display_name:display_name,
+            icon_hint:icon_hint, collapsed:collapsed,
+            renderer:FilterRenderer.CHECK_OPTIONS_COMPACT, visible:true,
+            filtering:false);
+  }
+}
+
 /* RatingsFilter allows the user to choose a rating between 0.0f and 1.0f in
  * 0.2f blocks (i.e. a rating up to five stars).
  */

=== modified file 'src/unity-scope-private.vala'
--- src/unity-scope-private.vala	2012-01-11 10:51:07 +0000
+++ src/unity-scope-private.vala	2012-01-17 19:38:31 +0000
@@ -215,6 +215,11 @@
                                         model.get_string (iter, FilterColumn.NAME),
                                         icon_hint);
         break;
+      case FilterRenderer.CHECK_OPTIONS_COMPACT:
+        filter = new CheckOptionFilterCompact (model.get_string (iter, FilterColumn.ID),
+                                               model.get_string (iter, FilterColumn.NAME),
+                                               icon_hint);
+        break;
       case FilterRenderer.MULTIRANGE:
         filter = new MultiRangeFilter (model.get_string (iter, FilterColumn.ID),
                                        model.get_string (iter, FilterColumn.NAME),