Code

more unreffing temporary styles properly
[inkscape.git] / src / ui / dialog / filedialog.cpp
index e5f01e8bf9a58705a84db20bd7d54534c279a5fa..2a5ededbb475bf88fd9f0f35b722587cb484a9fd 100644 (file)
@@ -1,45 +1,46 @@
-/*
+/**
  * Implementation of the file dialog interfaces defined in filedialog.h
  *
  * Authors:
  *   Bob Jamison
  *   Other dudes from The Inkscape Organization
  *
- * Copyright (C) 2004-2006 The Inkscape Organization
+ * Copyright (C) 2004-2007 Bob Jamison
+ * Copyright (C) 2006 Johan Engelen <johan@shouraizou.nl>
+ * Copyright (C) 2004-2007 The Inkscape Organization
  *
  * Released under GNU GPL, read the file 'COPYING' for more information
  */
 
+
 #ifdef HAVE_CONFIG_H
 # include <config.h>
 #endif
 
+//General includes
+#include <unistd.h>
+#include <sys/stat.h>
+#include <errno.h>
+#include <set>
 
-
+//Gtk includes
+#include <gtkmm.h>
+#include <glibmm/i18n.h>
+#include <glib/gstdio.h>
 //Temporary ugly hack
-//Remove these after the get_filter() calls in
+//Remove this after the get_filter() calls in
 //show() on both classes are fixed
 #include <gtk/gtkfilechooser.h>
-
 //Another hack
 #include <gtk/gtkentry.h>
 #include <gtk/gtkexpander.h>
+#ifdef WITH_GNOME_VFS
+# include <libgnomevfs/gnome-vfs-init.h>  // gnome_vfs_initialized
+#endif
+
 
-#include <unistd.h>
-#include <sys/stat.h>
-#include <glibmm/i18n.h>
-#include <gtkmm/box.h>
-#include <gtkmm/colorbutton.h>
-#include <gtkmm/frame.h>
-#include <gtkmm/filechooserdialog.h>
-#include <gtkmm/menubar.h>
-#include <gtkmm/menu.h>
-#include <gtkmm/entry.h>
-#include <gtkmm/expander.h>
-#include <gtkmm/comboboxtext.h>
-#include <gtkmm/stock.h>
-#include <gdkmm/pixbuf.h>
 
+//Inkscape includes
 #include "prefs-utils.h"
 #include <dialogs/dialog-events.h>
 #include <extension/input.h>
@@ -54,6 +55,7 @@
 #include "ui/widget/scalar-unit.h"
 
 
+//Routines from file.cpp
 #undef INK_DUMP_FILENAME_CONV
 
 #ifdef INK_DUMP_FILENAME_CONV
@@ -61,6 +63,8 @@ void dump_str( const gchar* str, const gchar* prefix );
 void dump_ustr( const Glib::ustring& ustr );
 #endif
 
+
+
 namespace Inkscape
 {
 namespace UI
@@ -112,8 +116,9 @@ fileDialogExtensionToPattern(Glib::ustring &pattern,
 /**
  *  Hack:  Find all entry widgets in a container
  */
-void findEntryWidgets(Gtk::Container *parent,
-                      std::vector<Gtk::Entry *> &result)
+static void
+findEntryWidgets(Gtk::Container *parent,
+                 std::vector<Gtk::Entry *> &result)
 {
     if (!parent)
         return;
@@ -136,8 +141,9 @@ void findEntryWidgets(Gtk::Container *parent,
 /**
  *  Hack:  Find all expander widgets in a container
  */
-void findExpanderWidgets(Gtk::Container *parent,
-                        std::vector<Gtk::Expander *> &result)
+static void
+findExpanderWidgets(Gtk::Container *parent,
+                    std::vector<Gtk::Expander *> &result)
 {
     if (!parent)
         return;
@@ -158,6 +164,7 @@ void findExpanderWidgets(Gtk::Container *parent,
 /*#########################################################################
 ### SVG Preview Widget
 #########################################################################*/
+
 /**
  * Simple class for displaying an SVG file in the "preview widget."
  * Currently, this is just a wrapper of the sp_svg_view Gtk widget.
@@ -224,9 +231,8 @@ bool SVGPreview::setDocument(SPDocument *doc)
     document = doc;
 
     //This should remove it from the box, and free resources
-    if (viewerGtk) {
+    if (viewerGtk)
         gtk_widget_destroy(viewerGtk);
-    }
 
     viewerGtk  = sp_svg_view_widget_new(doc);
     GtkWidget *vbox = (GtkWidget *)gobj();
@@ -236,13 +242,18 @@ bool SVGPreview::setDocument(SPDocument *doc)
     return true;
 }
 
+
 bool SVGPreview::setFileName(Glib::ustring &theFileName)
 {
     Glib::ustring fileName = theFileName;
 
     fileName = Glib::filename_to_utf8(fileName);
 
-    SPDocument *doc = sp_document_new (fileName.c_str(), 0);
+    /**
+     * I don't know why passing false to keepalive is bad.  But it
+     * prevents the display of an svg with a non-ascii filename
+     */              
+    SPDocument *doc = sp_document_new (fileName.c_str(), true);
     if (!doc) {
         g_warning("SVGView: error loading document '%s'\n", fileName.c_str());
         return false;
@@ -328,7 +339,7 @@ void SVGPreview::showImage(Glib::ustring &theFileName)
           "<svg\n"
           "xmlns=\"http://www.w3.org/2000/svg\"\n"
           "xmlns:xlink=\"http://www.w3.org/1999/xlink\"\n"
-          "width=\"%d\" height=\"%d\">\n"
+          "width=\"%d\" height=\"%d\">\n"  //# VALUES HERE
           "<rect\n"
           "  style=\"fill:#eeeeee;stroke:none\"\n"
           "  x=\"-100\" y=\"-100\" width=\"4000\" height=\"4000\"/>\n"
@@ -344,7 +355,7 @@ void SVGPreview::showImage(Glib::ustring &theFileName)
           "  style=\"font-size:24.000000;font-style:normal;font-weight:normal;"
           "    fill:#000000;fill-opacity:1.0000000;stroke:none;"
           "    font-family:Bitstream Vera Sans\"\n"
-          "  x=\"10\" y=\"26\">%d x %d</text>\n"
+          "  x=\"10\" y=\"26\">%d x %d</text>\n" //# VALUES HERE
           "</svg>\n\n";
 
     //if (!Glib::get_charset()) //If we are not utf8
@@ -384,7 +395,7 @@ void SVGPreview::showNoPreview()
           "<svg\n"
           "xmlns=\"http://www.w3.org/2000/svg\"\n"
           "xmlns:xlink=\"http://www.w3.org/1999/xlink\"\n"
-          "width=\"%d\" height=\"%d\">\n"
+          "width=\"%d\" height=\"%d\">\n" //# VALUES HERE
           "<g transform=\"translate(-190,24.27184)\" style=\"opacity:0.12\">\n"
           "<path\n"
           "style=\"font-size:12;fill:#ffffff;fill-rule:evenodd;stroke:#000000;stroke-width:0.936193pt\"\n"
@@ -447,7 +458,7 @@ void SVGPreview::showNoPreview()
           "font-stretch:normal;fill:#000000;fill-opacity:1.0000000;stroke:none;stroke-width:1.0000000pt;"
           "stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1.0000000;"
           "font-family:Bitstream Vera Sans;text-anchor:middle;writing-mode:lr\"\n"
-          "x=\"190\" y=\"240\">%s</text></g>\n"
+          "x=\"190\" y=\"240\">%s</text></g>\n"  //# VALUE HERE
           "</svg>\n\n";
 
     //Fill in the template
@@ -463,6 +474,11 @@ void SVGPreview::showNoPreview()
 
 }
 
+
+/**
+ * Inform the user that the svg file is too large to be displayed.
+ * This does not check for sizes of embedded images (yet) 
+ */ 
 void SVGPreview::showTooLarge(long fileLength)
 {
 
@@ -476,7 +492,7 @@ void SVGPreview::showTooLarge(long fileLength)
           "<svg\n"
           "xmlns=\"http://www.w3.org/2000/svg\"\n"
           "xmlns:xlink=\"http://www.w3.org/1999/xlink\"\n"
-          "width=\"%d\" height=\"%d\">\n"
+          "width=\"%d\" height=\"%d\">\n"  //# VALUES HERE
           "<g transform=\"translate(-170,24.27184)\" style=\"opacity:0.12\">\n"
           "<path\n"
           "style=\"font-size:12;fill:#ffffff;fill-rule:evenodd;stroke:#000000;stroke-width:0.936193pt\"\n"
@@ -539,13 +555,13 @@ void SVGPreview::showTooLarge(long fileLength)
           "font-stretch:normal;fill:#000000;fill-opacity:1.0000000;stroke:none;stroke-width:1.0000000pt;"
           "stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1.0000000;"
           "font-family:Bitstream Vera Sans;text-anchor:middle;writing-mode:lr\"\n"
-          "x=\"170\" y=\"215\">%5.1f MB</text>\n"
+          "x=\"170\" y=\"215\">%5.1f MB</text>\n" //# VALUE HERE
           "<text xml:space=\"preserve\"\n"
           "style=\"font-size:24.000000;font-style:normal;font-variant:normal;font-weight:bold;"
           "font-stretch:normal;fill:#000000;fill-opacity:1.0000000;stroke:none;stroke-width:1.0000000pt;"
           "stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1.0000000;"
           "font-family:Bitstream Vera Sans;text-anchor:middle;writing-mode:lr\"\n"
-          "x=\"180\" y=\"245\">%s</text>\n"
+          "x=\"180\" y=\"245\">%s</text>\n" //# VALUE HERE
           "</svg>\n\n";
 
     //Fill in the template
@@ -563,28 +579,30 @@ void SVGPreview::showTooLarge(long fileLength)
 
 }
 
+
+/**
+ * Return true if the string ends with the given suffix
+ */ 
 static bool
 hasSuffix(Glib::ustring &str, Glib::ustring &ext)
 {
     int strLen = str.length();
     int extLen = ext.length();
     if (extLen > strLen)
-    {
         return false;
-    }
     int strpos = strLen-1;
     for (int extpos = extLen-1 ; extpos>=0 ; extpos--, strpos--)
-    {
+        {
         Glib::ustring::value_type ch = str[strpos];
         if (ch != ext[extpos])
-        {
+            {
             if ( ((ch & 0xff80) != 0) ||
                  static_cast<Glib::ustring::value_type>( g_ascii_tolower( static_cast<gchar>(0x07f & ch) ) ) != ext[extpos] )
-            {
+                {
                 return false;
+                }
             }
         }
-    }
     return true;
 }
 
@@ -604,9 +622,7 @@ isValidImageFile(Glib::ustring &fileName)
             {
             Glib::ustring ext = extensions[j];
             if (hasSuffix(fileName, ext))
-                {
                 return true;
-                }
             }
         }
     return false;
@@ -618,15 +634,22 @@ bool SVGPreview::set(Glib::ustring &fileName, int dialogType)
     if (!Glib::file_test(fileName, Glib::FILE_TEST_EXISTS))
         return false;
 
-    gchar *fName = (gchar *)fileName.c_str();
-    //g_message("fname:%s\n", fName);
+    //g_message("fname:%s", fileName.c_str());
 
+    if (Glib::file_test(fileName, Glib::FILE_TEST_IS_DIR)) {
+        showNoPreview();
+        return false;
+    }
 
     if (Glib::file_test(fileName, Glib::FILE_TEST_IS_REGULAR))
         {
+        Glib::ustring fileNameUtf8 = Glib::filename_to_utf8(fileName);
+        gchar *fName = (gchar *)fileNameUtf8.c_str();
         struct stat info;
-        if (stat(fName, &info))
+        if (g_stat(fName, &info))
             {
+            g_warning("SVGPreview::set() : %s : %s",
+                           fName, strerror(errno));
             return FALSE;
             }
         long fileLen = info.st_size;
@@ -637,29 +660,24 @@ bool SVGPreview::set(Glib::ustring &fileName, int dialogType)
             return FALSE;
             }
         }
-
+        
     Glib::ustring svg = ".svg";
     Glib::ustring svgz = ".svgz";
 
     if ((dialogType == SVG_TYPES || dialogType == IMPORT_TYPES) &&
            (hasSuffix(fileName, svg) || hasSuffix(fileName, svgz)   )
-         )
-        {
+        ) {
         bool retval = setFileName(fileName);
         showingNoPreview = false;
         return retval;
-        }
-    else if (isValidImageFile(fileName))
-        {
+    } else if (isValidImageFile(fileName)) {
         showImage(fileName);
         showingNoPreview = false;
         return true;
-        }
-    else
-        {
+    } else {
         showNoPreview();
         return false;
-        }
+    }
 }
 
 
@@ -697,19 +715,25 @@ public:
     /**
      *
      */
-    FileDialogBase(const Glib::ustring &title) :
-                        Gtk::FileChooserDialog(title)
-        {
-        }
+    FileDialogBase(const Glib::ustring &title, FileDialogType type, gchar const* preferenceBase) :
+        Gtk::FileChooserDialog(title),
+        preferenceBase(preferenceBase ? preferenceBase : "unknown"),
+        dialogType(type)
+    {
+        internalSetup();
+    }
 
     /**
      *
      */
     FileDialogBase(const Glib::ustring &title,
-                   Gtk::FileChooserAction dialogType) :
-                   Gtk::FileChooserDialog(title, dialogType)
-        {
-        }
+                   Gtk::FileChooserAction dialogType, FileDialogType type, gchar const* preferenceBase) :
+        Gtk::FileChooserDialog(title, dialogType),
+        preferenceBase(preferenceBase ? preferenceBase : "unknown"),
+        dialogType(type)
+    {
+        internalSetup();
+    }
 
     /**
      *
@@ -717,42 +741,10 @@ public:
     virtual ~FileDialogBase()
         {}
 
-};
-
-
-
-/*#########################################################################
-### F I L E    O P E N
-#########################################################################*/
-
-/**
- * Our implementation class for the FileOpenDialog interface..
- */
-class FileOpenDialogImpl : public FileOpenDialog, public FileDialogBase
-{
-public:
-
-    FileOpenDialogImpl(const Glib::ustring &dir,
-                       FileDialogType fileTypes,
-                       const Glib::ustring &title);
-
-    virtual ~FileOpenDialogImpl();
-
-    bool show();
-
-    Inkscape::Extension::Extension *getSelectionType();
-
-    Glib::ustring getFilename();
-
-    std::vector<Glib::ustring> getFilenames ();
-
 protected:
+    void cleanup( bool showConfirmed );
 
-
-
-private:
-
-
+    Glib::ustring preferenceBase;
     /**
      * What type of 'open' are we? (open, import, place, etc)
      */
@@ -763,120 +755,139 @@ private:
      */
     SVGPreview svgPreview;
 
-    /**
-     * Callback for seeing if the preview needs to be drawn
-     */
-    void updatePreviewCallback();
+    //# Child widgets
+    Gtk::CheckButton previewCheckbox;
 
-    /**
-     * Fix to allow the user to type the file name
-     */
-    Gtk::Entry fileNameEntry;
+private:
+    void internalSetup();
 
     /**
-     *  Create a filter menu for this type of dialog
+     * Callback for user changing preview checkbox
      */
-    void createFilterMenu();
+    void _previewEnabledCB();
 
     /**
-     * Callback for user input into fileNameEntry
+     * Callback for seeing if the preview needs to be drawn
      */
-    void fileNameEntryChangedCallback();
+    void _updatePreviewCallback();
+};
 
-    /**
-     * Callback for user changing which item is selected on the list
-     */
-    void fileSelectedCallback();
 
+void FileDialogBase::internalSetup()
+{
+    bool enablePreview = 
+        (bool)prefs_get_int_attribute( preferenceBase.c_str(),
+             "enable_preview", 1 );
 
-    /**
-     * Filter name->extension lookup
-     */
-    std::map<Glib::ustring, Inkscape::Extension::Extension *> extensionMap;
+    previewCheckbox.set_label( Glib::ustring(_("Enable Preview")) );
+    previewCheckbox.set_active( enablePreview );
 
-    /**
-     * The extension to use to write this file
-     */
-    Inkscape::Extension::Extension *extension;
+    previewCheckbox.signal_toggled().connect(
+        sigc::mem_fun(*this, &FileDialogBase::_previewEnabledCB) );
 
-    /**
-     * Filename that was given
-     */
-    Glib::ustring myFilename;
+    //Catch selection-changed events, so we can adjust the text widget
+    signal_update_preview().connect(
+         sigc::mem_fun(*this, &FileDialogBase::_updatePreviewCallback) );
 
-};
+    //###### Add a preview widget
+    set_preview_widget(svgPreview);
+    set_preview_widget_active( enablePreview );
+    set_use_preview_label (false);
+
+}
 
 
+void FileDialogBase::cleanup( bool showConfirmed )
+{
+    if ( showConfirmed )
+        prefs_set_int_attribute( preferenceBase.c_str(),
+               "enable_preview", previewCheckbox.get_active() );
+}
+
+
+void FileDialogBase::_previewEnabledCB()
+{
+    bool enabled = previewCheckbox.get_active();
+    set_preview_widget_active(enabled);
+    if ( enabled ) {
+        _updatePreviewCallback();
+    }
+}
 
 
 
 /**
  * Callback for checking if the preview needs to be redrawn
  */
-void FileOpenDialogImpl::updatePreviewCallback()
+void FileDialogBase::_updatePreviewCallback()
 {
     Glib::ustring fileName = get_preview_filename();
-    if (fileName.length() < 1)
+
+#ifdef WITH_GNOME_VFS
+    if ( fileName.empty() && gnome_vfs_initialized() ) {
+        fileName = get_preview_uri();
+    }
+#endif
+
+    if (fileName.empty()) {
         return;
+    }
+
     svgPreview.set(fileName, dialogType);
 }
 
 
-
-
+/*#########################################################################
+### F I L E    O P E N
+#########################################################################*/
 
 /**
- * Callback for fileNameEntry widget
+ * Our implementation class for the FileOpenDialog interface..
  */
-void FileOpenDialogImpl::fileNameEntryChangedCallback()
+class FileOpenDialogImpl : public FileOpenDialog, public FileDialogBase
 {
-    Glib::ustring rawFileName = fileNameEntry.get_text();
+public:
 
-    Glib::ustring fileName = Glib::filename_from_utf8(rawFileName);
+    FileOpenDialogImpl(const Glib::ustring &dir,
+                       FileDialogType fileTypes,
+                       const Glib::ustring &title);
 
-    //g_message("User hit return.  Text is '%s'\n", fileName.c_str());
+    virtual ~FileOpenDialogImpl();
 
-    if (!Glib::path_is_absolute(fileName)) {
-        //try appending to the current path
-        // not this way: fileName = get_current_folder() + "/" + fName;
-        std::vector<Glib::ustring> pathSegments;
-        pathSegments.push_back( get_current_folder() );
-        pathSegments.push_back( fileName );
-        fileName = Glib::build_filename(pathSegments);
-    }
+    bool show();
 
-    //g_message("path:'%s'\n", fName.c_str());
+    Inkscape::Extension::Extension *getSelectionType();
 
-    if (Glib::file_test(fileName, Glib::FILE_TEST_IS_DIR)) {
-        set_current_folder(fileName);
-    } else if (Glib::file_test(fileName, Glib::FILE_TEST_IS_REGULAR)) {
-        //dialog with either (1) select a regular file or (2) cd to dir
-        //simulate an 'OK'
-        set_filename(fileName);
-        response(Gtk::RESPONSE_OK);
-    }
-}
+    Glib::ustring getFilename();
 
+    std::vector<Glib::ustring> getFilenames ();
 
+private:
 
+    /**
+     *  Create a filter menu for this type of dialog
+     */
+    void createFilterMenu();
+
+    /**
+     * Filter name->extension lookup
+     */
+    std::map<Glib::ustring, Inkscape::Extension::Extension *> extensionMap;
+
+    /**
+     * The extension to use to write this file
+     */
+    Inkscape::Extension::Extension *extension;
+
+    /**
+     * Filename that was given
+     */
+    Glib::ustring myFilename;
+
+};
 
 
-/**
- * Callback for fileNameEntry widget
- */
-void FileOpenDialogImpl::fileSelectedCallback()
-{
-    Glib::ustring fileName     = get_filename();
-    if (!Glib::get_charset()) //If we are not utf8
-        fileName = Glib::filename_to_utf8(fileName);
-    //g_message("User selected '%s'\n",
-    //       filename().c_str());
 
-#ifdef INK_DUMP_FILENAME_CONV
-    ::dump_ustr( get_filename() );
-#endif
-    fileNameEntry.set_text(fileName);
-}
 
 
 
@@ -940,7 +951,7 @@ void FileOpenDialogImpl::createFilterMenu()
 FileOpenDialogImpl::FileOpenDialogImpl(const Glib::ustring &dir,
                                        FileDialogType fileTypes,
                                        const Glib::ustring &title) :
-                                       FileDialogBase(title)
+    FileDialogBase(title, fileTypes, "dialogs.open")
 {
 
 
@@ -948,6 +959,12 @@ FileOpenDialogImpl::FileOpenDialogImpl(const Glib::ustring &dir,
     /* And also Multiple Files */
     set_select_multiple(true);
 
+#ifdef WITH_GNOME_VFS
+    if (gnome_vfs_initialized()) {
+        set_local_only(false);
+    }
+#endif
+
     /* Initalize to Autodetect */
     extension = NULL;
     /* No filename to start out with */
@@ -968,35 +985,16 @@ FileOpenDialogImpl::FileOpenDialogImpl(const Glib::ustring &dir,
         set_current_folder(udir.c_str());
         }
 
-    //###### Add the file types menu
-    createFilterMenu();
-
-    //###### Add a preview widget
-    set_preview_widget(svgPreview);
-    set_preview_widget_active(true);
-    set_use_preview_label (false);
-
-    //Catch selection-changed events, so we can adjust the text widget
-    signal_update_preview().connect(
-         sigc::mem_fun(*this, &FileOpenDialogImpl::updatePreviewCallback) );
 
+    set_extra_widget( previewCheckbox );
 
-    //###### Add a text entry bar, and tie it to file chooser events
-    fileNameEntry.set_text(get_current_folder());
-    set_extra_widget(fileNameEntry);
-    fileNameEntry.grab_focus();
 
-    //Catch when user hits [return] on the text field
-    fileNameEntry.signal_activate().connect(
-         sigc::mem_fun(*this, &FileOpenDialogImpl::fileNameEntryChangedCallback) );
+    //###### Add the file types menu
+    createFilterMenu();
 
-    //Catch selection-changed events, so we can adjust the text widget
-    signal_selection_changed().connect(
-         sigc::mem_fun(*this, &FileOpenDialogImpl::fileSelectedCallback) );
 
     add_button(Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
-    add_button(Gtk::Stock::OPEN,   Gtk::RESPONSE_OK);
-
+    set_default(*add_button(Gtk::Stock::OPEN,   Gtk::RESPONSE_OK));
 }
 
 
@@ -1055,10 +1053,16 @@ FileOpenDialogImpl::show()
             extension = extensionMap[gtk_file_filter_get_name(filter)];
             }
         myFilename = get_filename();
+#ifdef WITH_GNOME_VFS
+        if (myFilename.empty() && gnome_vfs_initialized())
+            myFilename = get_uri();
+#endif
+        cleanup( true );
         return TRUE;
         }
     else
        {
+       cleanup( false );
        return FALSE;
        }
 }
@@ -1092,6 +1096,10 @@ FileOpenDialogImpl::getFilename (void)
 std::vector<Glib::ustring>FileOpenDialogImpl::getFilenames()
 {    
     std::vector<Glib::ustring> result = get_filenames();
+#ifdef WITH_GNOME_VFS
+    if (result.empty() && gnome_vfs_initialized())
+        result = get_uris();
+#endif
     return result;
 }
 
@@ -1131,38 +1139,21 @@ public:
     bool show();
 
     Inkscape::Extension::Extension *getSelectionType();
+    virtual void setSelectionType( Inkscape::Extension::Extension * key );
 
     Glib::ustring getFilename();
 
+    void change_title(const Glib::ustring& title);
+    void change_path(const Glib::ustring& path);
+    void updateNameAndExtension();
 
 private:
 
-    /**
-     * What type of 'open' are we? (save, export, etc)
-     */
-    FileDialogType dialogType;
-
-    /**
-     * Our svg preview widget
-     */
-    SVGPreview svgPreview;
-
     /**
      * Fix to allow the user to type the file name
      */
     Gtk::Entry *fileNameEntry;
 
-    /**
-     * Callback for seeing if the preview needs to be drawn
-     */
-    void updatePreviewCallback();
-
-
-
-    /**
-     * Allow the specification of the output file type
-     */
-    Gtk::HBox fileTypeBox;
 
     /**
      * Allow the specification of the output file type
@@ -1176,8 +1167,10 @@ private:
     std::vector<FileType> fileTypes;
 
     //# Child widgets
-    Gtk::CheckButton fileTypeCheckbox;
+    Gtk::HBox childBox;
+    Gtk::VBox checksBox;
 
+    Gtk::CheckButton fileTypeCheckbox;
 
     /**
      * Callback for user input into fileNameEntry
@@ -1190,8 +1183,6 @@ private:
     void createFileTypeMenu();
 
 
-    bool append_extension;
-
     /**
      * The extension to use to write this file
      */
@@ -1206,27 +1197,16 @@ private:
      * Filename that was given
      */
     Glib::ustring myFilename;
-};
-
 
+    /**
+     * List of known file extensions.
+     */
+    std::set<Glib::ustring> knownExtensions;
+};
 
 
 
 
-/**
- * Callback for checking if the preview needs to be redrawn
- */
-void FileSaveDialogImpl::updatePreviewCallback()
-{
-    Glib::ustring fileName = get_preview_filename();
-    if (!fileName.c_str())
-        return;
-    bool retval = svgPreview.set(fileName, dialogType);
-    set_preview_widget_active(retval);
-}
-
-
-
 /**
  * Callback for fileNameEntry widget
  */
@@ -1274,9 +1254,13 @@ void FileSaveDialogImpl::fileTypeChangedCallback()
         return;
     FileType type = fileTypes[sel];
     //g_message("selected: %s\n", type.name.c_str());
+
+    extension = type.extension;
     Gtk::FileFilter filter;
     filter.add_pattern(type.pattern);
     set_filter(filter);
+
+    updateNameAndExtension();
 }
 
 
@@ -1285,6 +1269,7 @@ void FileSaveDialogImpl::createFileTypeMenu()
 {
     Inkscape::Extension::DB::OutputList extension_list;
     Inkscape::Extension::db.get_output_list(extension_list);
+    knownExtensions.clear();
 
     for (Inkscape::Extension::DB::OutputList::iterator current_item = extension_list.begin();
          current_item != extension_list.end(); current_item++)
@@ -1298,6 +1283,7 @@ void FileSaveDialogImpl::createFileTypeMenu()
         type.name     = (_(omod->get_filetypename()));
         type.pattern  = "*";
         Glib::ustring extension = omod->get_extension();
+        knownExtensions.insert( extension.casefold() );
         fileDialogExtensionToPattern (type.pattern, extension);
         type.extension= omod;
         fileTypeComboBox.append_text(type.name);
@@ -1326,14 +1312,17 @@ FileSaveDialogImpl::FileSaveDialogImpl(const Glib::ustring &dir,
             FileDialogType fileTypes,
             const Glib::ustring &title,
             const Glib::ustring &default_key) :
-            FileDialogBase(title, Gtk::FILE_CHOOSER_ACTION_SAVE)
+    FileDialogBase(title, Gtk::FILE_CHOOSER_ACTION_SAVE, fileTypes, "dialogs.save_as")
 {
-    append_extension = (bool)prefs_get_int_attribute("dialogs.save_as",
-                                                  "append_extension", 1);
-
     /* One file at a time */
     set_select_multiple(false);
 
+#ifdef WITH_GNOME_VFS
+    if (gnome_vfs_initialized()) {
+        set_local_only(false);
+    }
+#endif
+
     /* Initalize to Autodetect */
     extension = NULL;
     /* No filename to start out with */
@@ -1350,7 +1339,7 @@ FileSaveDialogImpl::FileSaveDialogImpl(const Glib::ustring &dir,
         // leaving a trailing backslash on the directory name leads to the infamous
         // double-directory bug on win32
         if (len != 0 && udir[len - 1] == '\\') udir.erase(len - 1);
-        set_current_folder(udir.c_str());
+        myFilename = udir;
         }
 
     //###### Add the file types menu
@@ -1358,29 +1347,21 @@ FileSaveDialogImpl::FileSaveDialogImpl(const Glib::ustring &dir,
 
     //###### Do we want the .xxx extension automatically added?
     fileTypeCheckbox.set_label(Glib::ustring(_("Append filename extension automatically")));
-    fileTypeCheckbox.set_active(append_extension);
+    fileTypeCheckbox.set_active( (bool)prefs_get_int_attribute("dialogs.save_as",
+                                                               "append_extension", 1) );
 
-    fileTypeBox.pack_start(fileTypeCheckbox);
     createFileTypeMenu();
     fileTypeComboBox.set_size_request(200,40);
     fileTypeComboBox.signal_changed().connect(
          sigc::mem_fun(*this, &FileSaveDialogImpl::fileTypeChangedCallback) );
 
-    fileTypeBox.pack_start(fileTypeComboBox);
 
-    set_extra_widget(fileTypeBox);
-    //get_vbox()->pack_start(fileTypeBox, false, false, 0);
-    //get_vbox()->reorder_child(fileTypeBox, 2);
-
-    //###### Add a preview widget
-    set_preview_widget(svgPreview);
-    set_preview_widget_active(true);
-    set_use_preview_label (false);
-
-    //Catch selection-changed events, so we can adjust the text widget
-    signal_update_preview().connect(
-         sigc::mem_fun(*this, &FileSaveDialogImpl::updatePreviewCallback) );
+    childBox.pack_start( checksBox );
+    childBox.pack_end( fileTypeComboBox );
+    checksBox.pack_start( fileTypeCheckbox );
+    checksBox.pack_start( previewCheckbox );
 
+    set_extra_widget( childBox );
 
     //Let's do some customization
     fileNameEntry = NULL;
@@ -1412,7 +1393,7 @@ FileSaveDialogImpl::FileSaveDialogImpl(const Glib::ustring &dir,
     //    checkbox.set_sensitive(FALSE);
 
     add_button(Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
-    add_button(Gtk::Stock::SAVE,   Gtk::RESPONSE_OK);
+    set_default(*add_button(Gtk::Stock::SAVE,   Gtk::RESPONSE_OK));
 
     show_all_children();
 }
@@ -1450,10 +1431,7 @@ FileSaveDialogImpl::~FileSaveDialogImpl()
 bool
 FileSaveDialogImpl::show()
 {
-    Glib::ustring s = Glib::filename_to_utf8 (get_current_folder());
-    if (s.length() == 0) 
-        s = getcwd (NULL, 0);
-    set_current_folder(Glib::filename_from_utf8(s)); //hack to force initial dir listing
+    change_path(myFilename);
     set_modal (TRUE);                      //Window
     sp_transientize((GtkWidget *)gobj());  //Make transient
     gint b = run();                        //Dialog
@@ -1462,27 +1440,23 @@ FileSaveDialogImpl::show()
 
     if (b == Gtk::RESPONSE_OK)
         {
-        int sel = fileTypeComboBox.get_active_row_number ();
-        if (sel>=0 && sel< (int)fileTypes.size())
-            {
-            FileType &type = fileTypes[sel];
-            extension = type.extension;
-            }
-        myFilename = get_filename();
-
-        /*
+        updateNameAndExtension();
 
-        // FIXME: Why do we have more code
+        // Store changes of the "Append filename automatically" checkbox back to preferences.
+        prefs_set_int_attribute("dialogs.save_as", "append_extension", fileTypeCheckbox.get_active());
 
-        append_extension = checkbox.get_active();
-        prefs_set_int_attribute("dialogs.save_as", "append_extension", append_extension);
+        // Store the last used save-as filetype to preferences.
         prefs_set_string_attribute("dialogs.save_as", "default",
-                  ( extension != NULL ? extension->get_id() : "" ));
-        */
+                                   ( extension != NULL ? extension->get_id() : "" ));
+
+        cleanup( true );
+
         return TRUE;
         }
     else
         {
+        cleanup( false );
+
         return FALSE;
         }
 }
@@ -1497,6 +1471,47 @@ FileSaveDialogImpl::getSelectionType()
     return extension;
 }
 
+void FileSaveDialogImpl::setSelectionType( Inkscape::Extension::Extension * key )
+{
+    // If no pointer to extension is passed in, look up based on filename extension.
+    if ( !key ) {
+        // Not quite UTF-8 here.
+        gchar *filenameLower = g_ascii_strdown(myFilename.c_str(), -1);
+        for ( int i = 0; !key && (i < (int)fileTypes.size()); i++ ) {
+            Inkscape::Extension::Output *ext = dynamic_cast<Inkscape::Extension::Output*>(fileTypes[i].extension);
+            if ( ext && ext->get_extension() ) {
+                gchar *extensionLower = g_ascii_strdown( ext->get_extension(), -1 );
+                if ( g_str_has_suffix(filenameLower, extensionLower) ) {
+                    key = fileTypes[i].extension;
+                }
+                g_free(extensionLower);
+            }
+        }
+        g_free(filenameLower);
+    }
+
+    // Ensure the proper entry in the combo box is selected.
+    if ( key ) {
+        extension = key;
+        gchar const * extensionID = extension->get_id();
+        if ( extensionID ) {
+            for ( int i = 0; i < (int)fileTypes.size(); i++ ) {
+                Inkscape::Extension::Extension *ext = fileTypes[i].extension;
+                if ( ext ) {
+                    gchar const * id = ext->get_id();
+                    if ( id && ( strcmp(extensionID, id) == 0) ) {
+                        int oldSel = fileTypeComboBox.get_active_row_number();
+                        if ( i != oldSel ) {
+                            fileTypeComboBox.set_active(i);
+                        }
+                        break;
+                    }
+                }
+            }
+        }
+    }
+}
+
 
 /**
  * Get the file name chosen by the user.   Valid after an [OK]
@@ -1508,9 +1523,85 @@ FileSaveDialogImpl::getFilename()
 }
 
 
+void 
+FileSaveDialogImpl::change_title(const Glib::ustring& title)
+{
+    this->set_title(title);
+}
 
+/**
+  * Change the default save path location.
+  */
+void 
+FileSaveDialogImpl::change_path(const Glib::ustring& path)
+{
+    myFilename = path;
+    if (Glib::file_test(myFilename, Glib::FILE_TEST_IS_DIR)) {
+        //fprintf(stderr,"set_current_folder(%s)\n",myFilename.c_str());
+        set_current_folder(myFilename);
+    } else {
+        //fprintf(stderr,"set_filename(%s)\n",myFilename.c_str());
+        if ( Glib::file_test( myFilename, Glib::FILE_TEST_EXISTS ) ) {
+            set_filename(myFilename);
+        } else {
+            std::string dirName = Glib::path_get_dirname( myFilename  );
+            if ( dirName != get_current_folder() ) {
+                set_current_folder(dirName);
+            }
+        }
+        Glib::ustring basename = Glib::path_get_basename(myFilename);
+        //fprintf(stderr,"set_current_name(%s)\n",basename.c_str());
+        try {
+            set_current_name( Glib::filename_to_utf8(basename) );
+        } catch ( Glib::ConvertError& e ) {
+            g_warning( "Error converting save filename to UTF-8." );
+            // try a fallback.
+            set_current_name( basename );
+        }
+    }
+}
 
+void FileSaveDialogImpl::updateNameAndExtension()
+{
+    // Pick up any changes the user has typed in.
+    Glib::ustring tmp = get_filename();
+#ifdef WITH_GNOME_VFS
+    if ( tmp.empty() && gnome_vfs_initialized() ) {
+        tmp = get_uri();
+    }
+#endif
+    if ( !tmp.empty() ) {
+        myFilename = tmp;
+    }
+
+    Inkscape::Extension::Output* newOut = extension ? dynamic_cast<Inkscape::Extension::Output*>(extension) : 0;
+    if ( fileTypeCheckbox.get_active() && newOut ) {
+        try {
+            bool appendExtension = true;
+            Glib::ustring utf8Name = Glib::filename_to_utf8( myFilename );
+            Glib::ustring::size_type pos = utf8Name.rfind('.');
+            if ( pos != Glib::ustring::npos ) {
+                Glib::ustring trail = utf8Name.substr( pos );
+                Glib::ustring foldedTrail = trail.casefold();
+                if ( (trail == ".") 
+                     | (foldedTrail != Glib::ustring( newOut->get_extension() ).casefold()
+                        && ( knownExtensions.find(foldedTrail) != knownExtensions.end() ) ) ) {
+                    utf8Name = utf8Name.erase( pos );
+                } else {
+                    appendExtension = false;
+                }
+            }
 
+            if (appendExtension) {
+                utf8Name = utf8Name + newOut->get_extension();
+                myFilename = Glib::filename_from_utf8( utf8Name );
+                change_path(myFilename);
+            }
+        } catch ( Glib::ConvertError& e ) {
+            // ignore
+        }
+    }
+}
 
 
 
@@ -1540,27 +1631,101 @@ public:
     Glib::ustring getFilename();
 
 
-private:
+    /**
+     * Return the scope of the export.  One of the enumerated types
+     * in ScopeType     
+     */
+    ScopeType getScope()
+        { 
+        if (pageButton.get_active())
+            return SCOPE_PAGE;
+        else if (selectionButton.get_active())
+            return SCOPE_SELECTION;
+        else if (customButton.get_active())
+            return SCOPE_CUSTOM;
+        else
+            return SCOPE_DOCUMENT;
 
+        }
+    
     /**
-     * What type of 'open' are we? (save, export, etc)
+     * Return left side of the exported region
      */
-    FileDialogType dialogType;
+    double getSourceX()
+        { return sourceX0Spinner.getValue(); }
+    
+    /**
+     * Return the top of the exported region
+     */
+    double getSourceY()
+        { return sourceY1Spinner.getValue(); }
+    
+    /**
+     * Return the width of the exported region
+     */
+    double getSourceWidth()
+        { return sourceWidthSpinner.getValue(); }
+    
+    /**
+     * Return the height of the exported region
+     */
+    double getSourceHeight()
+        { return sourceHeightSpinner.getValue(); }
 
     /**
-     * Our svg preview widget
+     * Return the units of the coordinates of exported region
      */
-    SVGPreview svgPreview;
+    Glib::ustring getSourceUnits()
+        { return sourceUnitsSpinner.getUnitAbbr(); }
 
     /**
-     * Fix to allow the user to type the file name
+     * Return the width of the destination document
      */
-    Gtk::Entry *fileNameEntry;
+    double getDestinationWidth()
+        { return destWidthSpinner.getValue(); }
 
     /**
-     * Callback for seeing if the preview needs to be drawn
+     * Return the height of the destination document
+     */
+    double getDestinationHeight()
+        { return destHeightSpinner.getValue(); }
+
+    /**
+     * Return the height of the exported region
+     */
+    Glib::ustring getDestinationUnits()
+        { return destUnitsSpinner.getUnitAbbr(); }
+
+    /**
+     * Return the destination DPI image resulution, if bitmap
+     */
+    double getDestinationDPI()
+        { return destDPISpinner.getValue(); }
+
+    /**
+     * Return whether we should use Cairo for rendering
+     */
+    bool getUseCairo()
+        { return cairoButton.get_active(); }
+
+    /**
+     * Return whether we should use antialiasing
+     */
+    bool getUseAntialias()
+        { return antiAliasButton.get_active(); }
+
+    /**
+     * Return the background color for exporting
+     */
+    unsigned long getBackground()
+        { return backgroundButton.get_color().get_pixel(); }
+
+private:
+
+    /**
+     * Fix to allow the user to type the file name
      */
-    void updatePreviewCallback();
+    Gtk::Entry *fileNameEntry;
 
     //##########################################
     //# EXTRA WIDGET -- SOURCE SIDE
@@ -1574,15 +1739,16 @@ private:
     Gtk::RadioButton      documentButton;
     Gtk::RadioButton      pageButton;
     Gtk::RadioButton      selectionButton;
+    Gtk::RadioButton      customButton;
 
     Gtk::Table                      sourceTable;
-    Inkscape::UI::Widget::Scalar    sourceX0;
-    Inkscape::UI::Widget::Scalar    sourceY0;
-    Inkscape::UI::Widget::Scalar    sourceX1;
-    Inkscape::UI::Widget::Scalar    sourceY1;
-    Inkscape::UI::Widget::Scalar    sourceWidth;
-    Inkscape::UI::Widget::Scalar    sourceHeight;
-    Inkscape::UI::Widget::UnitMenu  sourceUnits;
+    Inkscape::UI::Widget::Scalar    sourceX0Spinner;
+    Inkscape::UI::Widget::Scalar    sourceY0Spinner;
+    Inkscape::UI::Widget::Scalar    sourceX1Spinner;
+    Inkscape::UI::Widget::Scalar    sourceY1Spinner;
+    Inkscape::UI::Widget::Scalar    sourceWidthSpinner;
+    Inkscape::UI::Widget::Scalar    sourceHeightSpinner;
+    Inkscape::UI::Widget::UnitMenu  sourceUnitsSpinner;
 
 
     //##########################################
@@ -1593,10 +1759,10 @@ private:
     Gtk::VBox        destBox;
 
     Gtk::Table                      destTable;
-    Inkscape::UI::Widget::Scalar    destWidth;
-    Inkscape::UI::Widget::Scalar    destHeight;
-    Inkscape::UI::Widget::Scalar    destDPI;
-    Inkscape::UI::Widget::UnitMenu  destUnits;
+    Inkscape::UI::Widget::Scalar    destWidthSpinner;
+    Inkscape::UI::Widget::Scalar    destHeightSpinner;
+    Inkscape::UI::Widget::Scalar    destDPISpinner;
+    Inkscape::UI::Widget::UnitMenu  destUnitsSpinner;
 
     Gtk::HBox        otherOptionBox;
     Gtk::CheckButton cairoButton;
@@ -1660,20 +1826,6 @@ private:
 
 
 
-/**
- * Callback for checking if the preview needs to be redrawn
- */
-void FileExportDialogImpl::updatePreviewCallback()
-{
-    Glib::ustring fileName = get_preview_filename();
-    if (!fileName.c_str())
-        return;
-    bool retval = svgPreview.set(fileName, dialogType);
-    set_preview_widget_active(retval);
-}
-
-
-
 /**
  * Callback for fileNameEntry widget
  */
@@ -1772,22 +1924,28 @@ FileExportDialogImpl::FileExportDialogImpl(const Glib::ustring &dir,
             FileDialogType fileTypes,
             const Glib::ustring &title,
             const Glib::ustring &default_key) :
-            FileDialogBase(title, Gtk::FILE_CHOOSER_ACTION_SAVE),
-            sourceX0("X0",         _("Source left bound")),
-            sourceY0("Y0",         _("Source top bound")),
-            sourceX1("X1",         _("Source right bound")),
-            sourceY1("Y1",         _("Source bottom bound")),
-            sourceWidth("Width",   _("Source width")),
-            sourceHeight("Height", _("Source height")),
-            destWidth("Width",   _("Destination width")),
-            destHeight("Height", _("Destination height")),
-            destDPI("DPI", _("Dots per inch resolution"))
+            FileDialogBase(title, Gtk::FILE_CHOOSER_ACTION_SAVE, fileTypes, "dialogs.export"),
+            sourceX0Spinner("X0",         _("Left edge of source")),
+            sourceY0Spinner("Y0",         _("Top edge of source")),
+            sourceX1Spinner("X1",         _("Right edge of source")),
+            sourceY1Spinner("Y1",         _("Bottom edge of source")),
+            sourceWidthSpinner("Width",   _("Source width")),
+            sourceHeightSpinner("Height", _("Source height")),
+            destWidthSpinner("Width",     _("Destination width")),
+            destHeightSpinner("Height",   _("Destination height")),
+            destDPISpinner("DPI",         _("Resolution (dots per inch)"))
 {
     append_extension = (bool)prefs_get_int_attribute("dialogs.save_as", "append_extension", 1);
 
     /* One file at a time */
     set_select_multiple(false);
 
+#ifdef WITH_GNOME_VFS
+    if (gnome_vfs_initialized()) {
+        set_local_only(false);
+    }
+#endif
+
     /* Initalize to Autodetect */
     extension = NULL;
     /* No filename to start out with */
@@ -1824,20 +1982,24 @@ FileExportDialogImpl::FileExportDialogImpl(const Glib::ustring &dir,
     selectionButton.set_group(scopeGroup);
     scopeBox.pack_start(selectionButton);
 
+    customButton.set_label(_("Custom"));
+    customButton.set_group(scopeGroup);
+    scopeBox.pack_start(customButton);
+
     sourceBox.pack_start(scopeBox);
 
 
 
     //dimension buttons
     sourceTable.resize(3,3);
-    sourceTable.attach(sourceX0,     0,1,0,1);
-    sourceTable.attach(sourceY0,     1,2,0,1);
-    sourceUnits.setUnitType(UNIT_TYPE_LINEAR);
-    sourceTable.attach(sourceUnits,  2,3,0,1);
-    sourceTable.attach(sourceX1,     0,1,1,2);
-    sourceTable.attach(sourceY1,     1,2,1,2);
-    sourceTable.attach(sourceWidth,  0,1,2,3);
-    sourceTable.attach(sourceHeight, 1,2,2,3);
+    sourceTable.attach(sourceX0Spinner,     0,1,0,1);
+    sourceTable.attach(sourceY0Spinner,     1,2,0,1);
+    sourceUnitsSpinner.setUnitType(UNIT_TYPE_LINEAR);
+    sourceTable.attach(sourceUnitsSpinner,  2,3,0,1);
+    sourceTable.attach(sourceX1Spinner,     0,1,1,2);
+    sourceTable.attach(sourceY1Spinner,     1,2,1,2);
+    sourceTable.attach(sourceWidthSpinner,  0,1,2,3);
+    sourceTable.attach(sourceHeightSpinner, 1,2,2,3);
 
     sourceBox.pack_start(sourceTable);
     sourceFrame.set_label(_("Source"));
@@ -1851,11 +2013,11 @@ FileExportDialogImpl::FileExportDialogImpl(const Glib::ustring &dir,
 
 
     destTable.resize(3,3);
-    destTable.attach(destWidth,    0,1,0,1);
-    destTable.attach(destHeight,   1,2,0,1);
-    destUnits.setUnitType(UNIT_TYPE_LINEAR);
-    destTable.attach(destUnits,    2,3,0,1);
-    destTable.attach(destDPI,      0,1,1,2);
+    destTable.attach(destWidthSpinner,    0,1,0,1);
+    destTable.attach(destHeightSpinner,   1,2,0,1);
+    destUnitsSpinner.setUnitType(UNIT_TYPE_LINEAR);
+    destTable.attach(destUnitsSpinner,    2,3,0,1);
+    destTable.attach(destDPISpinner,      0,1,1,2);
 
     destBox.pack_start(destTable);
 
@@ -1902,16 +2064,6 @@ FileExportDialogImpl::FileExportDialogImpl(const Glib::ustring &dir,
 
 
 
-    //###### PREVIEW WIDGET
-    set_preview_widget(svgPreview);
-    set_preview_widget_active(true);
-    set_use_preview_label (false);
-
-    //Catch selection-changed events, so we can adjust the text widget
-    signal_update_preview().connect(
-         sigc::mem_fun(*this, &FileExportDialogImpl::updatePreviewCallback) );
-
-
     //Let's do some customization
     fileNameEntry = NULL;
     Gtk::Container *cont = get_toplevel();
@@ -1942,7 +2094,7 @@ FileExportDialogImpl::FileExportDialogImpl(const Glib::ustring &dir,
     //    checkbox.set_sensitive(FALSE);
 
     add_button(Gtk::Stock::CANCEL, Gtk::RESPONSE_CANCEL);
-    add_button(Gtk::Stock::SAVE,   Gtk::RESPONSE_OK);
+    set_default(*add_button(Gtk::Stock::SAVE,   Gtk::RESPONSE_OK));
 
     show_all_children();
 }
@@ -1999,6 +2151,11 @@ FileExportDialogImpl::show()
             extension = type.extension;
             }
         myFilename = get_filename();
+#ifdef WITH_GNOME_VFS
+        if ( myFilename.empty() && gnome_vfs_initialized() ) {
+            myFilename = get_uri();
+        }
+#endif
 
         /*