]> www.fi.muni.cz Git - evince.git/blobdiff - pdf/xpdf/gpdf.cc
Attempts at cleaning up component destruction - failed :-(
[evince.git] / pdf / xpdf / gpdf.cc
index 633ab234a865b572e46aca249d1f07b8ef05f85a..0db774b6caf3e07ecad4c60bb626f90409501478 100644 (file)
@@ -41,8 +41,6 @@ extern "C" {
 #include "Error.h"
 #include "config.h"
 
-#define UNTESTED 0
-
 poptContext ctx;
 gint  gpdf_debug=0;
 
@@ -57,14 +55,15 @@ typedef struct _Container Container;
 /* NB. there is a 1 to 1 Container -> Component mapping, this
    is due to how much MDI sucks; unutterably */
 struct _Container {
-  GnomeContainer  *container;
-  GnomeUIHandler  *uih;
+  GnomeContainer    *container;
+  GnomeUIHandler    *uih;
   
-  GnomeViewFrame  *active_view_frame;
+  GnomeViewFrame    *active_view_frame;
   
-  GtkWidget    *app;
-  GtkWidget    *view_widget;
-  Component     *component;
+  GtkWidget        *app;
+  GtkScrolledWindow *scroll;
+  GtkWidget        *view_widget;
+  Component         *component;
   gdouble zoom;
 };
 
@@ -86,6 +85,7 @@ extern "C" {
   static void       container_open_cmd  (GtkWidget *widget, Container *container);
   static void       container_close_cmd (GtkWidget *widget, Container *container);
   static void       container_exit_cmd  (void);
+  static void       container_about_cmd (GtkWidget *widget, Container *container);
   static Component *container_activate_component (Container *container, char *component_goad_id);
   static void       zoom_in_cmd         (GtkWidget *widget, Container *container);
   static void       zoom_out_cmd        (GtkWidget *widget, Container *container);
@@ -107,16 +107,22 @@ static GnomeUIInfo container_file_menu [] = {
 static GnomeUIInfo container_menu_zoom [] = {
        { GNOME_APP_UI_ITEM, N_("_Zoom in"),
          N_("Increase the size of objects in the PDF"),
-         NULL, zoom_in_cmd },
+         zoom_in_cmd },
        { GNOME_APP_UI_ITEM, N_("_Zoom out"),
          N_("Decrease the size of objects in the PDF"),
          zoom_out_cmd },
        GNOMEUIINFO_END
 };
 
+static GnomeUIInfo container_help_menu [] = {
+        GNOMEUIINFO_MENU_ABOUT_ITEM(container_about_cmd, NULL),
+       GNOMEUIINFO_END
+};
+
 static GnomeUIInfo container_main_menu [] = {
        GNOMEUIINFO_MENU_FILE_TREE (container_file_menu),
        { GNOME_APP_UI_SUBTREE, N_("_Zoom"), NULL, container_menu_zoom },
+       GNOMEUIINFO_MENU_HELP_TREE (container_help_menu),
        GNOMEUIINFO_END
 };
 
@@ -142,7 +148,7 @@ extern "C" {
     g_return_val_if_fail (container != NULL, FALSE);
     g_return_val_if_fail (container->view_widget == NULL, FALSE);
 
-    comp = container_activate_component (container, "bonobo-object:image-x-pdf");
+    comp = container_activate_component (container, "bonobo-object:application-x-pdf");
     if (!comp || !(object = comp->server)) {
       gnome_error_dialog (_("Could not launch bonobo object."));
       return FALSE;
@@ -171,8 +177,8 @@ extern "C" {
     GNOME_PersistStream_load (persist,
                              (GNOME_Stream) gnome_object_corba_objref (GNOME_OBJECT (stream)), &ev);
 
-    
-    
+    zoom_set (container);
+        
     GNOME_Unknown_unref (persist, &ev);
     CORBA_Object_release (persist, &ev);
     CORBA_exception_free (&ev);
@@ -230,8 +236,10 @@ extern "C" {
        char *fname = g_strdup (name);
        if (container->view_widget) /* any sort of MDI sucks :-] */
          container = container_new (fname);
-       else
-         open_pdf (container, fname);
+       else {
+         if (!open_pdf (container, fname))
+           container_destroy (container);
+       }
        g_free (fname);
       } else {
        GtkWidget *dialog;
@@ -247,11 +255,35 @@ extern "C" {
     gtk_widget_destroy (GTK_WIDGET (fsel));
   }
 
+  static void 
+  component_destroy (Component *component)
+  {
+    CORBA_Environment ev;
+    g_return_if_fail (component != NULL);
+
+    CORBA_exception_init (&ev);
+    
+    if (component->server)
+      GNOME_Unknown_unref (
+       gnome_object_corba_objref (GNOME_OBJECT (component->server)), &ev);
+    component->server = NULL;
+
+    CORBA_exception_free (&ev);
+  }
+
   static void
   container_destroy (Container *cont)
   {
     containers = g_list_remove (containers, cont);
-    gtk_widget_destroy (cont->app);
+    if (cont->app)
+      gtk_widget_destroy (cont->app);
+    
+    if (cont->component)
+      component_destroy (cont->component);
+    cont->component = NULL;
+    
+    cont->app = NULL;
+    
     g_free (cont);
     if (!containers)
       gtk_main_quit ();
@@ -270,6 +302,34 @@ extern "C" {
       container_destroy ((Container *)containers->data);
   }
 
+
+static void
+container_about_cmd (GtkWidget *widget, Container *container)
+{
+  GtkWidget *about;
+  int i;
+
+  const gchar *authors[] = {
+    N_("Michael Meeks, GNOME port maintainer."),
+    N_("Miguel de Icaza."),
+    N_("Nat Friedman."),
+    NULL
+  };
+  
+#ifdef ENABLE_NLS
+  for (i = 0; authors[i] != NULL; i++)
+    authors [i] = _(authors [i]);
+#endif
+  
+  about = gnome_about_new (_("GPDF"), VERSION,
+                          _("(C) 1996-1999 Derek B. Noonburg."),
+                          authors, NULL, NULL);
+  
+  gnome_dialog_set_parent (GNOME_DIALOG (about), GTK_WINDOW (container->app));
+  gnome_dialog_set_close (GNOME_DIALOG (about), TRUE);
+  gtk_widget_show (about);
+}
+
   /*
    * Enforces the containers zoom factor.
    */
@@ -287,131 +347,20 @@ extern "C" {
   zoom_in_cmd (GtkWidget *widget, Container *container)
   {
     g_return_if_fail (container != NULL);
-    container->zoom *= 1.4;
-    zoom_set (container);
+    if (container->zoom < 180.0) {
+      container->zoom *= 1.4;
+      zoom_set (container);
+    }
   }
 
   static void
   zoom_out_cmd (GtkWidget *widget, Container *container)
   {
     g_return_if_fail (container != NULL);
-    container->zoom /= 1.4;
-    zoom_set (container);
-  }
-
-  static void
-  component_user_activate_request_cb (GnomeViewFrame *view_frame, gpointer data)
-  {
-    Component *component = (Component *) data;
-    Container *container = component->container;
-    
-    /*
-     * If there is a
-     * If there is already an active View, deactivate it.
-     */
-    if (container->active_view_frame != NULL) {
-      /*
-       * This just sends a notice to the embedded View that
-       * it is being deactivated.  We will also forcibly
-       * cover it so that it does not receive any Gtk
-       * events.
-       */
-      gnome_view_frame_view_deactivate (container->active_view_frame);
-      
-      /*
-       * Here we manually cover it if it hasn't acquiesced.
-       * If it has consented to be deactivated, then it will
-       * already have notified us that it is inactive, and
-       * we will have covered it and set active_view_frame
-       * to NULL.  Which is why this check is here.
-       */
-      if (container->active_view_frame != NULL)
-       gnome_view_frame_set_covered (container->active_view_frame, TRUE);
-      
-      container->active_view_frame = NULL;
+    if (container->zoom > 10.0) {
+      container->zoom /= 1.4;
+      zoom_set (container);
     }
-    
-    /*
-     * Activate the View which the user clicked on.  This just
-     * sends a request to the embedded View to activate itself.
-     * When it agrees to be activated, it will notify its
-     * ViewFrame, and our view_activated_cb callback will be
-     * called.
-     *
-     * We do not uncover the View here, because it may not wish to
-     * be activated, and so we wait until it notifies us that it
-     * has been activated to uncover it.
-     */
-    gnome_view_frame_view_activate (view_frame);
-  }
-  
-  static void
-  component_view_activated_cb (GnomeViewFrame *view_frame, gboolean activated, gpointer data)
-  {
-    Component *component = (Component *) data;
-    Container *container = component->container;
-    
-    if (activated) {
-      /*
-       * If the View is requesting to be activated, then we
-       * check whether or not there is already an active
-       * View.
-       */
-      if (container->active_view_frame != NULL) {
-       g_warning ("View requested to be activated but there is already "
-                  "an active View!\n");
-       return;
-      }
-      
-      /*
-       * Otherwise, uncover it so that it can receive
-       * events, and set it as the active View.
-       */
-      gnome_view_frame_set_covered (view_frame, FALSE);
-      container->active_view_frame = view_frame;
-    } else {
-      /*
-       * If the View is asking to be deactivated, always
-       * oblige.  We may have already deactivated it (see
-       * user_activation_request_cb), but there's no harm in
-       * doing it again.  There is always the possibility
-       * that a View will ask to be deactivated when we have
-       * not told it to deactivate itself, and that is
-       * why we cover the view here.
-       */
-      gnome_view_frame_set_covered (view_frame, TRUE);
-      
-      if (view_frame == container->active_view_frame)
-       container->active_view_frame = NULL;
-    }                                                                       
-  }
-  
-  static void
-  component_user_context_cb (GnomeViewFrame *view_frame, gpointer data)
-  {
-    Component *component = (Component *) data;
-    char *executed_verb;
-    GList *l;
-    
-    /*
-     * See if the remote GnomeEmbeddable supports any verbs at
-     * all.
-     */
-    l = gnome_client_site_get_verbs (component->client_site);
-    if (l == NULL)
-      return;
-    gnome_client_site_free_verbs (l);
-    
-    /*
-     * Popup the verb popup and execute the chosen verb.  This
-     * function saves us the work of creating the menu, connecting
-     * the callback, and executing the verb on the remove
-     * GnomeView.  We could implement all this functionality
-     * ourselves if we wanted.
-     */
-    executed_verb = gnome_view_frame_popup_verbs (view_frame);
-    
-    g_free (executed_verb);
   }
 }
 
@@ -441,46 +390,18 @@ container_set_view (Container *container, Component *component)
        view_widget = gnome_view_frame_get_wrapper (view_frame);
        container->view_widget = view_widget;
        container->component   = component;
-/*     gtk_box_pack_start (GTK_BOX (container->app), view_widget,
-       FALSE, FALSE, 5);*/
-       gnome_app_set_contents (GNOME_APP (container->app), view_widget);
-       /*
-        * The "user_activate" signal will be emitted when the user
-        * double clicks on the "cover".  The cover is a transparent
-        * window which sits on top of the component and keeps any
-        * events (mouse, keyboard) from reaching it.  When the user
-        * double clicks on the cover, the container (that's us)
-        * can choose to activate the component.
-        */
-       gtk_signal_connect (GTK_OBJECT (view_frame), "user_activate",
-                           GTK_SIGNAL_FUNC (component_user_activate_request_cb), component);
 
        /*
-        * In-place activation of a component is a two-step process.
-        * After the user double clicks on the component, our signal
-        * callback (compoennt_user_activate_request_cb()) asks the
-        * component to activate itself (see
-        * gnome_view_frame_view_activate()).  The component can then
-        * choose to either accept or refuse activation.  When an
-        * embedded component notifies us of its decision to change
-        * its activation state, the "view_activated" signal is
-        * emitted from the view frame.  It is at that point that we
-        * actually remove the cover so that events can get through.
+        * Show the component.
         */
-       gtk_signal_connect (GTK_OBJECT (view_frame), "view_activated",
-                           GTK_SIGNAL_FUNC (component_view_activated_cb), component);
+       gtk_scrolled_window_add_with_viewport (container->scroll, view_widget);
 
        /*
-        * The "user_context" signal is emitted when the user right
-        * clicks on the wrapper.  We use it to pop up a verb menu.
+        * Activate it ( get it to merge menus etc. )
         */
-       gtk_signal_connect (GTK_OBJECT (view_frame), "user_context",
-                           GTK_SIGNAL_FUNC (component_user_context_cb), component);
+       gnome_view_frame_view_activate (view_frame);
 
-       /*
-        * Show the component.
-        */
-       gtk_widget_show_all (view_widget);
+       gtk_widget_show_all (GTK_WIDGET (container->scroll));
 }
 
 static GnomeObjectClient *
@@ -494,7 +415,7 @@ container_launch_component (GnomeClientSite *client_site,
         * Launch the component.
         */
        object_server = gnome_object_activate_with_goad_id (
-               NULL, component_goad_id, 0, NULL);
+               NULL, component_goad_id, GOAD_ACTIVATE_SHLIB, NULL);
 
        if (object_server == NULL)
                return NULL;
@@ -591,6 +512,35 @@ extern "C" {
 
     return component;
   }
+  
+  static void
+  filenames_dropped (GtkWidget * widget,
+                    GdkDragContext   *context,
+                    gint              x,
+                    gint              y,
+                    GtkSelectionData *selection_data,
+                    guint             info,
+                    guint             time,
+                    Container        *container)
+  {
+    GList *names, *tmp_list;
+    
+    names = gnome_uri_list_extract_filenames ((char *)selection_data->data);
+    tmp_list = names;
+    
+    while (tmp_list) {
+      const char *fname = (const char *)tmp_list->data;
+
+      if (fname) {
+       if (container->view_widget)
+         container = container_new (fname);
+       else
+         open_pdf (container, fname);
+      }
+
+      tmp_list = g_list_next (tmp_list);
+    }
+  }
 }
 
 static void
@@ -611,36 +561,48 @@ container_create_menus (Container *container)
 static void
 container_create_toolbar (Container *container)
 {
-       GnomeUIHandlerMenuItem *toolbar;
+       GnomeUIHandlerToolbarItem *toolbar;
 
-#if UNTESTED > 0
-       gnome_ui_handler_create_menubar (container->uih);
-
-       /*
-        * Create the basic menus out of UIInfo structures.
-        */
+       gnome_ui_handler_create_toolbar (container->uih, "pdf");
        toolbar = gnome_ui_handler_toolbar_parse_uiinfo_list_with_data (container_toolbar, container);
-       gnome_ui_handler_toolbar_add_list (container->uih, "/", menu_list);
-       gnome_ui_handler_toolbar_free_list (menu_list);
-#endif
+       gnome_ui_handler_toolbar_add_list (container->uih, "/", toolbar);
+       gnome_ui_handler_toolbar_free_list (toolbar);
 }
 
 static Container *
 container_new (const char *fname)
 {
        Container *container;
-
+       static GtkTargetEntry drag_types[] =
+       {
+         { "text/uri-list", 0, 0 },
+       };
+       static gint n_drag_types = sizeof (drag_types) / sizeof (drag_types [0]);
+       
        container = g_new0 (Container, 1);
 
        container->app  = gnome_app_new ("pdf-viewer",
                                         "GNOME PDF viewer");
-       container->zoom = 43.0;
+       container->zoom = 86.0;
 
-       gtk_window_set_default_size (GTK_WINDOW (container->app), 400, 400);
+       gtk_drag_dest_set (container->app,
+                          GTK_DEST_DEFAULT_ALL,
+                          drag_types, n_drag_types,
+                          GDK_ACTION_COPY);
+
+       gtk_signal_connect (GTK_OBJECT(container->app),
+                           "drag_data_received",
+                           GTK_SIGNAL_FUNC(filenames_dropped), (gpointer)container);
+
+       gtk_window_set_default_size (GTK_WINDOW (container->app), 600, 600);
        gtk_window_set_policy (GTK_WINDOW (container->app), TRUE, TRUE, FALSE);
 
        container->container   = gnome_container_new ();
        container->view_widget = NULL;
+       container->scroll = GTK_SCROLLED_WINDOW (gtk_scrolled_window_new (NULL, NULL));
+       gtk_scrolled_window_set_policy (container->scroll, GTK_POLICY_AUTOMATIC,
+                                       GTK_POLICY_AUTOMATIC);
+       gnome_app_set_contents (GNOME_APP (container->app), GTK_WIDGET (container->scroll));
 
        /*
         * Create the GnomeUIHandler object which will be used to
@@ -673,9 +635,10 @@ int
 main (int argc, char **argv)
 {
   CORBA_Environment ev;
-  CORBA_ORB orb;
-  char **view_files = NULL;
-  int    i;
+  CORBA_ORB         orb;
+  char            **view_files = NULL;
+  gboolean          loaded;
+  int               i;
   
   CORBA_exception_init (&ev);
   
@@ -696,11 +659,13 @@ main (int argc, char **argv)
 
   /* Load files */
   i = 0;
+  loaded = FALSE;
   if (view_files) {
     for (i = 0; view_files[i]; i++)
-      container_new (view_files[i]);
+      if (container_new (view_files[i]))
+       loaded = TRUE;
   }
-  if (i == 0)
+  if ((i == 0) || !loaded)
     container_new (NULL);
   
   poptFreeContext (ctx);
@@ -709,4 +674,3 @@ main (int argc, char **argv)
        
   return 0;
 }
-