Fix bug rendering margin in output viewer.
[pspp] / src / ui / gui / psppire-output-view.c
index f1397aa792f5d9b855b2c87ad6e76c4a635b6f51..a7367ed76a9d85723d195e2ce98b21035c57463b 100644 (file)
@@ -1,5 +1,5 @@
 /* PSPPIRE - a graphical user interface for PSPP.
-   Copyright (C) 2008-2014 Free Software Foundation.
+   Copyright (C) 2008-2015 Free Software Foundation.
 
    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -54,7 +54,7 @@ struct psppire_output_view
     GtkLayout *output;
     int render_width;
     int max_width;
-    int y;
+    glong y;
 
     struct string_map render_opts;
     GtkTreeView *overview;
@@ -76,52 +76,17 @@ struct psppire_output_view
 
 enum
   {
-    COL_TITLE,                  /* Table title. */
+    COL_NAME,                   /* Table name. */
     COL_ADDR,                   /* Pointer to the table */
-    COL_Y,                      /* Y position of top of title. */
+    COL_Y,                      /* Y position of top of name. */
     N_COLS
   };
 
-static void on_dwgarea_realize (GtkWidget *widget, gpointer data);
-
 static gboolean
-expose_event_callback (GtkWidget *widget, GdkEventExpose *event, gpointer data)
+draw_callback (GtkWidget *widget, cairo_t *cr, gpointer data)
 {
-  struct psppire_output_view *view = data;
   struct xr_rendering *r = g_object_get_data (G_OBJECT (widget), "rendering");
-  cairo_t *cr = gdk_cairo_create (widget->window);
-
-  const GtkStyle *style = gtk_widget_get_style (GTK_WIDGET (view->output));
-
-  PangoFontDescription *font_desc;
-  char *font_name;
-
-  gchar *fgc =
-    gdk_color_to_string (&style->text[gtk_widget_get_state (GTK_WIDGET (view->output))]);
-
-  string_map_replace (&view->render_opts, "foreground-color", fgc);
-
-  free (fgc);
-
-  /* Use GTK+ default font as proportional font. */
-  font_name = pango_font_description_to_string (style->font_desc);
-  string_map_replace (&view->render_opts, "prop-font", font_name);
-  g_free (font_name);
-
-  /* Derived emphasized font from proportional font. */
-  font_desc = pango_font_description_copy (style->font_desc);
-  pango_font_description_set_style (font_desc, PANGO_STYLE_ITALIC);
-  font_name = pango_font_description_to_string (font_desc);
-  string_map_replace (&view->render_opts, "emph-font", font_name);
-  g_free (font_name);
-  pango_font_description_free (font_desc);
-
-  xr_rendering_apply_options (r, &view->render_opts);
-
-  xr_rendering_draw (r, cr, event->area.x, event->area.y,
-                     event->area.width, event->area.height);
-  cairo_destroy (cr);
-
+  xr_rendering_draw_all (r, cr);
   return TRUE;
 }
 
@@ -135,7 +100,6 @@ free_rendering (gpointer rendering_)
 static void
 create_xr (struct psppire_output_view *view)
 {
-  const GtkStyle *style = gtk_widget_get_style (GTK_WIDGET (view->output));
   struct text_item *text_item;
   PangoFontDescription *font_desc;
   struct xr_rendering *r;
@@ -144,21 +108,30 @@ create_xr (struct psppire_output_view *view)
   cairo_t *cr;
   gchar *fgc;
 
-  cr = gdk_cairo_create (GTK_WIDGET (view->output)->window);
+  GtkStyleContext *context = gtk_widget_get_style_context (GTK_WIDGET (view->output));
+  GtkStateFlags state = gtk_widget_get_state_flags (GTK_WIDGET (view->output));
+  GdkRGBA *fg_color;
 
-  /* Set the widget's text color as the foreground color for the output driver */
-  fgc = gdk_color_to_string (&style->text[gtk_widget_get_state (GTK_WIDGET (view->output))]);
+  gtk_style_context_get (context, state,
+    "font", &font_desc, "color", &fg_color, NULL);
+
+  cr = gdk_cairo_create (gtk_widget_get_window (GTK_WIDGET (view->output)));
 
+  /* Set the widget's text color as the foreground color for the output driver */
+  /* gdk_rgba_to_string() would be perfect, but xr's parse_color does not      */
+  /* understand the rgb(255,128,33) format. Therefore we do it ourself.        */
+  fgc = xasprintf("#%4x%4x%4x",(int)(fg_color->red*0xffff),
+    (int)(fg_color->green*0xffff),(int)(fg_color->blue*0xffff));
   string_map_insert (&view->render_opts, "foreground-color", fgc);
   g_free (fgc);
+  gdk_rgba_free (fg_color);
 
   /* Use GTK+ default font as proportional font. */
-  font_name = pango_font_description_to_string (style->font_desc);
+  font_name = pango_font_description_to_string (font_desc);
   string_map_insert (&view->render_opts, "prop-font", font_name);
   g_free (font_name);
 
   /* Derived emphasized font from proportional font. */
-  font_desc = pango_font_description_copy (style->font_desc);
   pango_font_description_set_style (font_desc, PANGO_STYLE_ITALIC);
   font_name = pango_font_description_to_string (font_desc);
   string_map_insert (&view->render_opts, "emph-font", font_name);
@@ -182,32 +155,72 @@ create_xr (struct psppire_output_view *view)
   text_item = text_item_create (TEXT_ITEM_PARAGRAPH, "X");
   r = xr_rendering_create (view->xr, text_item_super (text_item), cr);
   xr_rendering_measure (r, &font_width, &view->font_height);
-  /* xr_rendering_destroy (r); */
   text_item_unref (text_item);
 
   cairo_destroy (cr);
 }
 
+/* Return the horizontal position to place a widget whose 
+   width is CHILD_WIDTH */
+static gint
+get_xpos (const struct psppire_output_view *view, gint child_width)
+{
+  GdkWindow *gdkw = gtk_widget_get_window (GTK_WIDGET (view->output));
+  guint w = gdk_window_get_width (gdkw);
+  int gutter = 0;
+  g_object_get (view->output, "border-width", &gutter, NULL);
+  return (gtk_widget_get_direction (GTK_WIDGET (view->output)) ==  GTK_TEXT_DIR_RTL) ? w - child_width - gutter: gutter;
+}
+
+static void
+create_drawing_area (struct psppire_output_view *view,
+                     GtkWidget *drawing_area, struct xr_rendering *r,
+                     int tw, int th)
+{
+  /* Enable this to help with debugging.  It shows you which widgets are being
+     put where. */
+  if (0)
+    {
+      GdkRGBA green = {0, 1, 0, 1};
+      gtk_widget_override_background_color (GTK_WIDGET (view->output),
+                                           GTK_STATE_NORMAL, &green);
+      GdkRGBA red = {1, 0, 0, 1};
+      gtk_widget_override_background_color (drawing_area, GTK_STATE_NORMAL, &red);
+    }
+
+  g_object_set_data_full (G_OBJECT (drawing_area),
+                          "rendering", r, free_rendering);
+
+  g_signal_connect (drawing_area, "draw",
+                    G_CALLBACK (draw_callback), view);
+
+  gtk_widget_set_size_request (drawing_area, tw, th);
+  gint xpos = get_xpos (view, tw);
+
+  gtk_layout_put (view->output, drawing_area, xpos, view->y);
+
+  gtk_widget_show (drawing_area);
+}
+
 static void
 rerender (struct psppire_output_view *view)
 {
   struct output_view_item *item;
+  GdkWindow *gdkw = gtk_widget_get_window (GTK_WIDGET (view->output));
   cairo_t *cr;
 
-  if (!view->n_items)
+  if (!view->n_items || ! gdkw)
     return;
 
-  string_map_clear (&view->render_opts);
-  xr_driver_destroy (view->xr);
-  create_xr (view);
-
-  cr = gdk_cairo_create (GTK_WIDGET (view->output)->window);
-
+  cr = gdk_cairo_create (gdkw);
+  if (view->xr == NULL)
+    create_xr (view);
   view->y = 0;
   view->max_width = 0;
   for (item = view->items; item < &view->items[view->n_items]; item++)
     {
       struct xr_rendering *r;
+      GtkAllocation alloc;
       int tw, th;
 
       if (view->y > 0)
@@ -221,10 +234,27 @@ rerender (struct psppire_output_view *view)
         }
 
       xr_rendering_measure (r, &tw, &th);
-      g_object_set_data_full (G_OBJECT (item->drawing_area),
-                              "rendering", r, free_rendering);
-      gtk_widget_set_size_request (item->drawing_area, tw, th);
-      gtk_layout_move (view->output, item->drawing_area, 0, view->y);
+
+      gint xpos = get_xpos (view, tw);
+
+      if (!item->drawing_area)
+        {
+          item->drawing_area = gtk_drawing_area_new ();
+          create_drawing_area (view, item->drawing_area, r, tw, th);
+        }
+      else
+        {
+          g_object_set_data_full (G_OBJECT (item->drawing_area),
+                                  "rendering", r, free_rendering);
+          gtk_widget_set_size_request (item->drawing_area, tw, th);
+          gtk_layout_move (view->output, item->drawing_area, xpos, view->y);
+        }
+
+      alloc.x = xpos;
+      alloc.y = view->y;
+      alloc.width = tw;
+      alloc.height = th;
+      gtk_widget_size_allocate(item->drawing_area,&alloc);
 
       if (view->max_width < tw)
         view->max_width = tw;
@@ -239,13 +269,14 @@ void
 psppire_output_view_put (struct psppire_output_view *view,
                          const struct output_item *item)
 {
+  struct output_view_item *view_item;
   GtkWidget *drawing_area;
   struct xr_rendering *r;
-  struct string title;
+  struct string name;
   GtkTreeStore *store;
+  cairo_t *cr = NULL;
   GtkTreePath *path;
   GtkTreeIter iter;
-  cairo_t *cr;
   int tw, th;
 
   if (is_text_item (item))
@@ -266,33 +297,31 @@ psppire_output_view_put (struct psppire_output_view *view,
   if (view->n_items >= view->allocated_items)
     view->items = x2nrealloc (view->items, &view->allocated_items,
                                 sizeof *view->items);
-  view->items[view->n_items].item = output_item_ref (item);
-  view->items[view->n_items].drawing_area = drawing_area = gtk_drawing_area_new ();
-  view->n_items++;
-
-  cr = gdk_cairo_create (GTK_WIDGET (view->output)->window);
-  if (view->xr == NULL)
-    create_xr (view);
+  view_item = &view->items[view->n_items++];
+  view_item->item = output_item_ref (item);
+  view_item->drawing_area = NULL;
 
-  if (view->y > 0)
-    view->y += view->font_height / 2;
+  if (gtk_widget_get_window (GTK_WIDGET (view->output)))
+    {
+      view_item->drawing_area = drawing_area = gtk_drawing_area_new ();
 
-  r = xr_rendering_create (view->xr, item, cr);
-  if (r == NULL)
-    goto done;
+      cr = gdk_cairo_create (gtk_widget_get_window (GTK_WIDGET (view->output)));
+      if (view->xr == NULL)
+        create_xr (view);
 
-  xr_rendering_measure (r, &tw, &th);
+      if (view->y > 0)
+        view->y += view->font_height / 2;
 
-  g_object_set_data_full (G_OBJECT (drawing_area), "rendering", r, free_rendering);
-  g_signal_connect (drawing_area, "realize",
-                    G_CALLBACK (on_dwgarea_realize), view);
-  g_signal_connect (drawing_area, "expose_event",
-                    G_CALLBACK (expose_event_callback), view);
+      r = xr_rendering_create (view->xr, item, cr);
+      if (r == NULL)
+        goto done;
 
-  gtk_widget_set_size_request (drawing_area, tw, th);
-  gtk_layout_put (view->output, drawing_area, 0, view->y);
+      xr_rendering_measure (r, &tw, &th);
 
-  gtk_widget_show (drawing_area);
+      create_drawing_area (view, drawing_area, r, tw, th);
+    }
+  else
+    tw = th = 0;
 
   if (view->overview
       && (!is_text_item (item)
@@ -301,7 +330,7 @@ psppire_output_view_put (struct psppire_output_view *view,
     {
       store = GTK_TREE_STORE (gtk_tree_view_get_model (view->overview));
 
-      ds_init_empty (&title);
+      ds_init_empty (&name);
       if (is_text_item (item)
           && text_item_get_type (to_text_item (item)) == TEXT_ITEM_COMMAND_OPEN)
         {
@@ -315,38 +344,38 @@ psppire_output_view_put (struct psppire_output_view *view,
           gtk_tree_store_append (store, &iter, p);
         }
 
-      ds_clear (&title);
+      ds_clear (&name);
       if (is_text_item (item))
-        ds_put_cstr (&title, text_item_get_text (to_text_item (item)));
+        ds_put_cstr (&name, text_item_get_text (to_text_item (item)));
       else if (is_message_item (item))
         {
           const struct message_item *msg_item = to_message_item (item);
           const struct msg *msg = message_item_get_msg (msg_item);
-          ds_put_format (&title, "%s: %s", _("Message"),
+          ds_put_format (&name, "%s: %s", _("Message"),
                          msg_severity_to_string (msg->severity));
         }
       else if (is_table_item (item))
         {
-          const char *caption = table_item_get_caption (to_table_item (item));
-          if (caption != NULL)
-            ds_put_format (&title, "Table: %s", caption);
+          const char *title = table_item_get_title (to_table_item (item));
+          if (title != NULL)
+            ds_put_format (&name, "Table: %s", title);
           else
-            ds_put_cstr (&title, "Table");
+            ds_put_cstr (&name, "Table");
         }
       else if (is_chart_item (item))
         {
           const char *s = chart_item_get_title (to_chart_item (item));
           if (s != NULL)
-            ds_put_format (&title, "Chart: %s", s);
+            ds_put_format (&name, "Chart: %s", s);
           else
-            ds_put_cstr (&title, "Chart");
+            ds_put_cstr (&name, "Chart");
         }
       gtk_tree_store_set (store, &iter,
-                          COL_TITLE, ds_cstr (&title),
+                          COL_NAME, ds_cstr (&name),
                          COL_ADDR, item,
-                          COL_Y, view->y,
+                          COL_Y, (view->y),
                           -1);
-      ds_destroy (&title);
+      ds_destroy (&name);
 
       path = gtk_tree_model_get_path (GTK_TREE_MODEL (store), &iter);
       gtk_tree_view_expand_row (view->overview, path, TRUE);
@@ -383,9 +412,9 @@ on_row_activate (GtkTreeView *overview,
   y = g_value_get_long (&value);
   g_value_unset (&value);
 
-  vadj = gtk_layout_get_vadjustment (view->output);
-  min = vadj->lower;
-  max = vadj->upper - vadj->page_size;
+  vadj = gtk_scrollable_get_vadjustment (GTK_SCROLLABLE (view->output));
+  min = gtk_adjustment_get_lower (vadj);
+  max = gtk_adjustment_get_upper (vadj) - gtk_adjustment_get_page_size (vadj);
   if (y < min)
     y = min;
   else if (y > max)
@@ -394,32 +423,14 @@ on_row_activate (GtkTreeView *overview,
 }
 
 static void
-copy_base_to_bg (GtkWidget *dest, GtkWidget *src)
-{
-  int i;
-  for (i = 0; i < 5; ++i)
-    {
-      gtk_widget_modify_bg (dest, i, &gtk_widget_get_style (src)->base[i]);
-      gtk_widget_modify_fg (dest, i, &gtk_widget_get_style (src)->text[i]);
-    }
-}
-
-/* Copy the base style from the parent widget to the container and all its
-   children.  We do this because the container's primary purpose is to display
-   text.  This way psppire appears to follow the chosen gnome theme. */
-static void
-on_style_set (GtkWidget *toplevel, GtkStyle *prev,
-              struct psppire_output_view *view)
+on_style_updated (GtkWidget *toplevel, struct psppire_output_view *view)
 {
-  copy_base_to_bg (GTK_WIDGET (view->output), toplevel);
-  gtk_container_foreach (GTK_CONTAINER (view->output),
-                         (GtkCallback) copy_base_to_bg, view->output);
-}
-
-static void
-on_dwgarea_realize (GtkWidget *dwg_area, gpointer data)
-{
-  copy_base_to_bg (dwg_area, gtk_widget_get_toplevel (dwg_area));
+  if (!view->n_items || !gtk_widget_get_window (GTK_WIDGET (view->output)))
+    return;
+  string_map_clear (&view->render_opts);
+  xr_driver_destroy (view->xr);
+  create_xr (view);
+  rerender (view);
 }
 
 enum {
@@ -525,7 +536,7 @@ clipboard_get_cb (GtkClipboard     *clipboard,
 
   if ( g_file_get_contents (filename, &text, &length, NULL) )
     {
-      gtk_selection_data_set (selection_data, selection_data->target,
+      gtk_selection_data_set (selection_data, gtk_selection_data_get_target (selection_data),
                              8,
                              (const guchar *) text, length);
     }
@@ -598,6 +609,7 @@ on_size_allocate (GtkWidget    *widget,
                   struct psppire_output_view *view)
 {
   int new_render_width = MAX (300, allocation->width);
+
   if (view->render_width != new_render_width)
     {
       view->render_width = new_render_width;
@@ -635,15 +647,18 @@ psppire_output_view_new (GtkLayout *output, GtkTreeView *overview,
   view->print_n_pages = 0;
   view->paginated = FALSE;
 
-  g_signal_connect (view->toplevel, "style-set", G_CALLBACK (on_style_set), view);
+  g_signal_connect (output, "style-updated", G_CALLBACK (on_style_updated), view);
 
   g_signal_connect (output, "size-allocate", G_CALLBACK (on_size_allocate), view);
 
+  gtk_style_context_add_class (gtk_widget_get_style_context (GTK_WIDGET (output)),
+                              GTK_STYLE_CLASS_VIEW);
+
   if (overview)
     {
       model = GTK_TREE_MODEL (gtk_tree_store_new (
                                 N_COLS,
-                                G_TYPE_STRING,  /* COL_TITLE */
+                                G_TYPE_STRING,  /* COL_NAME */
                                 G_TYPE_POINTER, /* COL_ADDR */
                                 G_TYPE_LONG));  /* COL_Y */
       gtk_tree_view_set_model (overview, model);
@@ -658,7 +673,7 @@ psppire_output_view_new (GtkLayout *output, GtkTreeView *overview,
       gtk_tree_view_append_column (GTK_TREE_VIEW (overview), column);
       renderer = gtk_cell_renderer_text_new ();
       gtk_tree_view_column_pack_start (column, renderer, TRUE);
-      gtk_tree_view_column_add_attribute (column, renderer, "text", COL_TITLE);
+      gtk_tree_view_column_add_attribute (column, renderer, "text", COL_NAME);
 
       g_signal_connect (GTK_TREE_VIEW (overview),
                         "row-activated", G_CALLBACK (on_row_activate), view);
@@ -681,8 +696,8 @@ psppire_output_view_destroy (struct psppire_output_view *view)
   if (!view)
     return;
 
-  g_signal_handlers_disconnect_by_func (view->toplevel,
-                                        G_CALLBACK (on_style_set), view);
+  g_signal_handlers_disconnect_by_func (view->output,
+                                        G_CALLBACK (on_style_updated), view);
 
   string_map_destroy (&view->render_opts);
 
@@ -699,7 +714,26 @@ psppire_output_view_destroy (struct psppire_output_view *view)
 
   free (view);
 }
-\f
+
+void
+psppire_output_view_clear (struct psppire_output_view *view)
+{
+  size_t i;
+
+  view->max_width = 0;
+  view->y = 0;
+
+  for (i = 0; i < view->n_items; i++)
+    {
+      gtk_container_remove (GTK_CONTAINER (view->output),
+                            view->items[i].drawing_area);
+      output_item_unref (view->items[i].item);
+    }
+  free (view->items);
+  view->items = NULL;
+  view->n_items = view->allocated_items = 0;
+}
+
 /* Export. */
 
 void
@@ -793,7 +827,7 @@ paginate (GtkPrintOperation *operation,
                              view->items[view->print_item++].item);
       while (xr_driver_need_new_page (view->print_xrd))
        {
-         xr_driver_next_page (view->print_xrd, NULL);
+         xr_driver_next_page (view->print_xrd, get_cairo_context_from_print_context (context));
          view->print_n_pages ++;
        }
       return FALSE;
@@ -874,3 +908,47 @@ psppire_output_view_print (struct psppire_output_view *view,
 
   g_object_unref (print);
 }
+\f
+struct psppire_output_view_driver
+  {
+    struct output_driver driver;
+    struct psppire_output_view *view;
+  };
+
+static struct psppire_output_view_driver *
+psppire_output_view_driver_cast (struct output_driver *driver)
+{
+  return UP_CAST (driver, struct psppire_output_view_driver, driver);
+}
+
+static void
+psppire_output_view_submit (struct output_driver *this,
+                            const struct output_item *item)
+{
+  struct psppire_output_view_driver *povd = psppire_output_view_driver_cast (this);
+
+  if (is_table_item (item))
+    psppire_output_view_put (povd->view, item);
+}
+
+static struct output_driver_class psppire_output_view_driver_class =
+  {
+    "PSPPIRE Output View",      /* name */
+    NULL,                       /* destroy */
+    psppire_output_view_submit, /* submit */
+    NULL,                       /* flush */
+  };
+
+void
+psppire_output_view_register_driver (struct psppire_output_view *view)
+{
+  struct psppire_output_view_driver *povd;
+  struct output_driver *d;
+
+  povd = xzalloc (sizeof *povd);
+  povd->view = view;
+  d = &povd->driver;
+  output_driver_init (d, &psppire_output_view_driver_class, "PSPPIRE Output View",
+                      SETTINGS_DEVICE_UNFILTERED);
+  output_driver_register (d);
+}