[gnome-control-center/wip/jsparber/background: 39/58] [feat] add revealer to slide text

[Julian Sparber <jsparber src gnome org>]

commit 901e4403dc8b4c376a8ea78594dc07d1df38c4f9
Author: Julian Sparber <julian sparber net>
Date:   Thu Nov 9 01:38:57 2017 +0100

    [feat] add revealer to slide text

 panels/background/background.ui         |   89 ++++++++++++++++---------------
 panels/background/cc-background-panel.c |   43 +++++++++++----
 2 files changed, 77 insertions(+), 55 deletions(-)
---
diff --git a/panels/background/background.ui b/panels/background/background.ui
index 7c037ed..29da671 100644
--- a/panels/background/background.ui
+++ b/panels/background/background.ui
@@ -48,65 +48,66 @@
           </packing>
         </child>
         <child>
-          <object class="GtkBox" id="slide-box">
+          <object class="GtkRevealer" id="wallpaper-info">
             <property name="visible">True</property>
-            <property name="can_focus">False</property>
-            <property name="margin_top">10</property>
+            <property name="reveal-child">True</property>
             <child>
-              <object class="GtkBox" id="hbox2">
+              <object class="GtkBox" id="slide-box">
                 <property name="visible">True</property>
                 <property name="can_focus">False</property>
-                <property name="halign">center</property>
+                <property name="margin_top">10</property>
                 <child>
-                  <object class="GtkImage" id="slide_image">
+                  <object class="GtkBox" id="hbox2">
                     <property name="visible">True</property>
                     <property name="can_focus">False</property>
-                    <property name="icon_name">slideshow-symbolic</property>
+                    <property name="halign">center</property>
+                    <child>
+                      <object class="GtkImage" id="slide_image">
+                        <property name="visible">True</property>
+                        <property name="can_focus">False</property>
+                        <property name="icon_name">slideshow-symbolic</property>
+                      </object>
+                      <packing>
+                        <property name="expand">False</property>
+                        <property name="fill">True</property>
+                        <property name="position">0</property>
+                      </packing>
+                    </child>
+                    <child>
+                      <object class="GtkLabel" id="strut">
+                        <property name="visible">True</property>
+                        <property name="can_focus">False</property>
+                        <property name="label"> </property>
+                      </object>
+                      <packing>
+                        <property name="expand">False</property>
+                        <property name="fill">True</property>
+                        <property name="position">1</property>
+                      </packing>
+                    </child>
+                    <child>
+                      <object class="GtkLabel" id="slide-label">
+                        <property name="visible">True</property>
+                        <property name="can_focus">False</property>
+                        <property name="xalign">0</property>
+                        <property name="label" translatable="yes" comments="This refers to a slideshow 
background">Changes throughout the day</property>
+                      </object>
+                      <packing>
+                        <property name="expand">False</property>
+                        <property name="fill">True</property>
+                        <property name="position">1</property>
+                      </packing>
+                    </child>
                   </object>
                   <packing>
-                    <property name="expand">False</property>
+                    <property name="expand">True</property>
                     <property name="fill">True</property>
                     <property name="position">0</property>
                   </packing>
                 </child>
-                <child>
-                  <object class="GtkLabel" id="strut">
-                    <property name="visible">True</property>
-                    <property name="can_focus">False</property>
-                    <property name="label"> </property>
-                  </object>
-                  <packing>
-                    <property name="expand">False</property>
-                    <property name="fill">True</property>
-                    <property name="position">1</property>
-                  </packing>
-                </child>
-                <child>
-                  <object class="GtkLabel" id="slide-label">
-                    <property name="visible">True</property>
-                    <property name="can_focus">False</property>
-                    <property name="xalign">0</property>
-                    <property name="label" translatable="yes" comments="This refers to a slideshow 
background">Changes throughout the day</property>
-                  </object>
-                  <packing>
-                    <property name="expand">False</property>
-                    <property name="fill">True</property>
-                    <property name="position">1</property>
-                  </packing>
-                </child>
               </object>
-              <packing>
-                <property name="expand">True</property>
-                <property name="fill">True</property>
-                <property name="position">0</property>
-              </packing>
             </child>
           </object>
-          <packing>
-            <property name="expand">False</property>
-            <property name="fill">True</property>
-            <property name="position">2</property>
-          </packing>
         </child>
       </object>
       <packing>
@@ -116,7 +117,7 @@
       </packing>
     </child>
     <child>
-      <object class="GtkBox" id="backgroud-gallery-box">
+      <object class="GtkBox" id="background-gallery-box">
         <property name="visible">True</property>
         <property name="can_focus">False</property>
         <property name="expand">True</property>
diff --git a/panels/background/cc-background-panel.c b/panels/background/cc-background-panel.c
index 73600d3..c4b37da 100644
--- a/panels/background/cc-background-panel.c
+++ b/panels/background/cc-background-panel.c
@@ -146,7 +146,8 @@ update_preview (CcBackgroundPanel *panel,
       changes_with_time = cc_background_item_changes_with_time (current_background);
     }
 
-  gtk_widget_set_visible (WID ("slide-box"), changes_with_time);
+  gtk_revealer_set_reveal_child (GTK_REVEALER (WID ("wallpaper-info")),
+                                 changes_with_time);
 
   gtk_widget_queue_draw (WID ("background-desktop-drawingarea"));
 }
@@ -180,16 +181,16 @@ get_or_create_cached_pixbuf (CcBackgroundPanel *panel,
       gdk_pixbuf_get_width (pixbuf) != preview_width ||
       gdk_pixbuf_get_height (pixbuf) != preview_height) {
 
-      gtk_widget_get_allocation (widget, &allocation);
-      scale_factor = gtk_widget_get_scale_factor (widget);
-      pixbuf = cc_background_item_get_frame_thumbnail (background,
-                                                       panel->thumb_factory,
-                                                       preview_width,
-                                                       preview_height,
-                                                       scale_factor,
-                                                       -2, TRUE);
-      g_object_set_data_full (G_OBJECT (background), "pixbuf", pixbuf, g_object_unref);
-    }
+    gtk_widget_get_allocation (widget, &allocation);
+    scale_factor = gtk_widget_get_scale_factor (widget);
+    pixbuf = cc_background_item_get_frame_thumbnail (background,
+                                                     panel->thumb_factory,
+                                                     preview_width,
+                                                     preview_height,
+                                                     scale_factor,
+                                                     -2, TRUE);
+    g_object_set_data_full (G_OBJECT (background), "pixbuf", pixbuf, g_object_unref);
+  }
 
   return pixbuf;
 }
@@ -200,6 +201,26 @@ on_preview_draw (GtkWidget         *widget,
                  CcBackgroundPanel *panel)
 {
   GdkPixbuf *pixbuf;
+  const gint width = gtk_widget_get_allocated_width (panel);
+  gint height  = gtk_widget_get_allocated_height (panel);
+  gint request_height;
+  const gint preview_width = gtk_widget_get_allocated_width (widget);
+  const gint preview_height = gtk_widget_get_allocated_height (widget);
+  /*g_print ("Height %d", height);
+  if (preview_width > 310) {
+    gtk_widget_set_vexpand (WID ("background-preview"), FALSE);
+    gtk_widget_set_size_request (widget, 310, preview_height);
+  }
+  else {
+    gtk_widget_set_vexpand (WID ("background-preview"), TRUE);
+    gtk_widget_set_size_request (widget, -1, -1);
+  }
+  */
+
+  /*gtk_widget_get_size_request (WID ("background-gallery-box"), NULL, &request_height);
+  g_print ("Height %d\n", height);
+  gtk_widget_set_size_request (WID ("background-gallery-box"), -1, height - 300);
+  */
 
   pixbuf = get_or_create_cached_pixbuf (panel,
                                         widget,