Code

Purging old axonometric grid code: reworked it into the new grid class structure...
authorjohanengelen <johanengelen@users.sourceforge.net>
Sun, 25 Mar 2007 22:05:48 +0000 (22:05 +0000)
committerjohanengelen <johanengelen@users.sourceforge.net>
Sun, 25 Mar 2007 22:05:48 +0000 (22:05 +0000)
16 files changed:
src/attributes.cpp
src/attributes.h
src/desktop.cpp
src/display/canvas-axonomgrid.cpp
src/display/canvas-axonomgrid.h
src/display/canvas-grid.cpp
src/display/canvas-grid.h
src/grid-snapper.cpp
src/grid-snapper.h
src/preferences-skeleton.h
src/snap.cpp
src/snap.h
src/sp-namedview.cpp
src/sp-namedview.h
src/ui/dialog/document-properties.cpp
src/ui/dialog/document-properties.h

index 5f7a710c3bfdfd409c84ca56aea763979f472cfa..858f71804f890f65f4f1f5435c73beb8b201599d 100644 (file)
@@ -60,7 +60,6 @@ static SPStyleProp const props[] = {
     /* SPNamedView */
     {SP_ATTR_VIEWONLY, "viewonly"},
     {SP_ATTR_SHOWGRID, "showgrid"},
-    {SP_ATTR_GRIDTYPE, "gridtype"},
     {SP_ATTR_SHOWGUIDES, "showguides"},
     {SP_ATTR_GRIDTOLERANCE, "gridtolerance"},
     {SP_ATTR_GUIDETOLERANCE, "guidetolerance"},
@@ -70,8 +69,6 @@ static SPStyleProp const props[] = {
     {SP_ATTR_GRIDORIGINY, "gridoriginy"},
     {SP_ATTR_GRIDSPACINGX, "gridspacingx"},
     {SP_ATTR_GRIDSPACINGY, "gridspacingy"},
-    {SP_ATTR_GRIDANGLEX, "gridanglex"},
-    {SP_ATTR_GRIDANGLEZ, "gridanglez"},
     {SP_ATTR_GRIDCOLOR, "gridcolor"},
     {SP_ATTR_GRIDOPACITY, "gridopacity"},
     {SP_ATTR_GRIDEMPCOLOR, "gridempcolor"},
index 242c5c24691f763c40745a9683b201a2432ed51c..253724ac42dc4625a355bf9dc6aa8f23ed7cb292 100644 (file)
@@ -61,7 +61,6 @@ enum SPAttributeEnum {
     /* SPNamedView */
     SP_ATTR_VIEWONLY,
     SP_ATTR_SHOWGRID,
-    SP_ATTR_GRIDTYPE,
     SP_ATTR_SHOWGUIDES,
     SP_ATTR_GRIDTOLERANCE,
     SP_ATTR_GUIDETOLERANCE,
@@ -71,8 +70,6 @@ enum SPAttributeEnum {
     SP_ATTR_GRIDORIGINY,
     SP_ATTR_GRIDSPACINGX,
     SP_ATTR_GRIDSPACINGY,
-    SP_ATTR_GRIDANGLEX,
-    SP_ATTR_GRIDANGLEZ,
     SP_ATTR_GRIDCOLOR,
     SP_ATTR_GRIDOPACITY,
     SP_ATTR_GRIDEMPCOLOR,
index 4727f3e511e564de3194df3cf7fc9a2e78eea74d..0d7d43646f9353752289ae27dbd04a42fc9a1c32 100644 (file)
@@ -1413,9 +1413,6 @@ _update_snap_distances (SPDesktop *desktop)
     nv.snap_manager.grid.setDistance(sp_convert_distance_full(nv.gridtolerance,
                                                                       *nv.gridtoleranceunit,
                                                                       px));
-    nv.snap_manager.axonomgrid.setDistance(sp_convert_distance_full(nv.gridtolerance,
-                                                                      *nv.gridtoleranceunit,
-                                                                      px));
     //new grid snappers
     for ( GSList const *l = nv.grids; l != NULL; l = l->next) {
         Inkscape::CanvasGrid *grid = (Inkscape::CanvasGrid*) l->data;
index 9dfbed9255b9089dfede98270c5e86e0e3b1c454..d384fb5d428beb3ba344a6ca991a26a8be8fd8fc 100644 (file)
@@ -1,25 +1,22 @@
-#define SP_CANVAS_AXONOMGRID_C
+#define CANVAS_AXONOMGRID_C
 
 /*
- * SPCAxonomGrid
- *
- * Copyright (C) 2006 Johan Engelen <johan@shouraizou.nl>
- * Copyright (C) 2000 Lauris Kaplinski
- *
- */                         
- /* 
+ * Copyright (C) 2006-2007 Johan Engelen <johan@shouraizou.nl>
+ */
+
+ /*
   * Current limits are: one axis (y-axis) is always vertical. The other two
-  * axes are bound to a certain range of angles. The z-axis always has an angle 
+  * axes are bound to a certain range of angles. The z-axis always has an angle
   * smaller than 90 degrees (measured from horizontal, 0 degrees being a line extending
   * to the right). The x-axis will always have an angle between 0 and 90 degrees.
   * When I quickly think about it: all possibilities are probably covered this way. Eg.
   * a z-axis with negative angle can be replaced with an x-axis, etc.
-  */              
-  
+  */
+
  /*
-  *  TODO:  LOTS LOTS LOTS. Optimization etc.
-  *
+  *  TODO:  LOTS LOTS LOTS. Clean up code. dirty as hell
+  * THIS FILE AND THE HEADER FILE NEED HUGE CLEANING UP. PLEASE DO NOT HESISTATE TO DO SO.
+*  For example: the line drawing code should not be here. There _must_ be a function somewhere else that can provide this functionality...
   */
 
 #include "sp-canvas-util.h"
 #include "display-forward.h"
 #include <libnr/nr-pixops.h>
 
-#define SAFE_SETPIXEL   //undefine this when it is certain that setpixel is never called with invalid params
 
-enum {
-    ARG_0,
-    ARG_ORIGINX,
-    ARG_ORIGINY,
-    ARG_ANGLEX,
-    ARG_SPACINGY,
-    ARG_ANGLEZ,
-    ARG_COLOR,
-    ARG_EMPCOLOR,
-    ARG_EMPSPACING
-};
+#include "canvas-grid.h"
+#include "desktop-handles.h"
+#include "helper/units.h"
+#include "svg/svg-color.h"
+#include "xml/node-event-vector.h"
+#include "sp-object.h"
+
+#include "sp-namedview.h"
+#include "inkscape.h"
+#include "desktop.h"
+
+#include "../document.h"
+
+#define SAFE_SETPIXEL   //undefine this when it is certain that setpixel is never called with invalid params
 
 enum Dim3 { X=0, Y, Z };
 
@@ -50,144 +49,9 @@ enum Dim3 { X=0, Y, Z };
 static double deg_to_rad(double deg) { return deg*M_PI/180.0;}
 
 
-static void sp_caxonomgrid_class_init (SPCAxonomGridClass *klass);
-static void sp_caxonomgrid_init (SPCAxonomGrid *grid);
-static void sp_caxonomgrid_destroy (GtkObject *object);
-static void sp_caxonomgrid_set_arg (GtkObject *object, GtkArg *arg, guint arg_id);
-
-static void sp_caxonomgrid_update (SPCanvasItem *item, NR::Matrix const &affine, unsigned int flags);
-static void sp_caxonomgrid_render (SPCanvasItem *item, SPCanvasBuf *buf);
-
-static SPCanvasItemClass * parent_class;
-
-GtkType
-sp_caxonomgrid_get_type (void)
-{
-    static GtkType caxonomgrid_type = 0;
-
-    if (!caxonomgrid_type) {
-        GtkTypeInfo caxonomgrid_info = {
-            "SPCAxonomGrid",
-            sizeof (SPCAxonomGrid),
-            sizeof (SPCAxonomGridClass),
-            (GtkClassInitFunc) sp_caxonomgrid_class_init,
-            (GtkObjectInitFunc) sp_caxonomgrid_init,
-            NULL, NULL,
-            (GtkClassInitFunc) NULL
-        };
-        caxonomgrid_type = gtk_type_unique (sp_canvas_item_get_type (), &caxonomgrid_info);
-    }
-    return caxonomgrid_type;
-}
-
-static void
-sp_caxonomgrid_class_init (SPCAxonomGridClass *klass)
-{
-
-    GtkObjectClass *object_class;
-    SPCanvasItemClass *item_class;
-
-    object_class = (GtkObjectClass *) klass;
-    item_class = (SPCanvasItemClass *) klass;
-
-    parent_class = (SPCanvasItemClass*)gtk_type_class (sp_canvas_item_get_type ());
-
-    gtk_object_add_arg_type ("SPCAxonomGrid::originx", GTK_TYPE_DOUBLE, GTK_ARG_WRITABLE, ARG_ORIGINX);
-    gtk_object_add_arg_type ("SPCAxonomGrid::originy", GTK_TYPE_DOUBLE, GTK_ARG_WRITABLE, ARG_ORIGINY);
-    gtk_object_add_arg_type ("SPCAxonomGrid::anglex", GTK_TYPE_DOUBLE, GTK_ARG_WRITABLE, ARG_ANGLEX);
-    gtk_object_add_arg_type ("SPCAxonomGrid::spacingy", GTK_TYPE_DOUBLE, GTK_ARG_WRITABLE, ARG_SPACINGY);
-    gtk_object_add_arg_type ("SPCAxonomGrid::anglez", GTK_TYPE_DOUBLE, GTK_ARG_WRITABLE, ARG_ANGLEZ);
-    gtk_object_add_arg_type ("SPCAxonomGrid::color", GTK_TYPE_INT, GTK_ARG_WRITABLE, ARG_COLOR);
-    gtk_object_add_arg_type ("SPCAxonomGrid::empcolor", GTK_TYPE_INT, GTK_ARG_WRITABLE, ARG_EMPCOLOR);
-    gtk_object_add_arg_type ("SPCAxonomGrid::empspacing", GTK_TYPE_INT, GTK_ARG_WRITABLE, ARG_EMPSPACING);
-
-    object_class->destroy = sp_caxonomgrid_destroy;
-    object_class->set_arg = sp_caxonomgrid_set_arg;
-
-    item_class->update = sp_caxonomgrid_update;
-    item_class->render = sp_caxonomgrid_render;
-  
-}
-
-static void
-sp_caxonomgrid_init (SPCAxonomGrid *grid)
-{
-    grid->origin[NR::X] = grid->origin[NR::Y] = 0.0;
-//    grid->spacing[X] = grid->spacing[Y] = grid->spacing[Z] = 8.0;
-    grid->color = 0x0000ff7f;
-    grid->empcolor = 0x3F3FFF40;
-    grid->empspacing = 5;
-}
-
-static void
-sp_caxonomgrid_destroy (GtkObject *object)
-{
-    g_return_if_fail (object != NULL);
-    g_return_if_fail (SP_IS_CAXONOMGRID (object));
-
-    if (GTK_OBJECT_CLASS (parent_class)->destroy)
-        (* GTK_OBJECT_CLASS (parent_class)->destroy) (object);
-}
-
-static void
-sp_caxonomgrid_set_arg (GtkObject *object, GtkArg *arg, guint arg_id)
-{
-    SPCanvasItem *item = SP_CANVAS_ITEM (object);
-    SPCAxonomGrid *grid = SP_CAXONOMGRID (object);
-    
-    switch (arg_id) {
-    case ARG_ORIGINX:
-        grid->origin[NR::X] = GTK_VALUE_DOUBLE (* arg);
-        sp_canvas_item_request_update (item);
-        break;
-    case ARG_ORIGINY:
-        grid->origin[NR::Y] = GTK_VALUE_DOUBLE (* arg);
-        sp_canvas_item_request_update (item);
-        break;
-    case ARG_ANGLEX:
-        grid->angle_deg[X] = GTK_VALUE_DOUBLE (* arg);
-        if (grid->angle_deg[X] < 0.0) grid->angle_deg[X] = 0.0;
-        if (grid->angle_deg[X] > 89.0) grid->angle_deg[X] = 89.0;
-        grid->angle_rad[X] = deg_to_rad(grid->angle_deg[X]);
-        grid->tan_angle[X] = tan(grid->angle_rad[X]);
-        sp_canvas_item_request_update (item);
-        break;
-    case ARG_SPACINGY:
-        grid->lengthy = GTK_VALUE_DOUBLE (* arg);
-        if (grid->lengthy < 0.01) grid->lengthy = 0.01;
-        sp_canvas_item_request_update (item);
-        break;
-    case ARG_ANGLEZ:
-        grid->angle_deg[Z] = GTK_VALUE_DOUBLE (* arg);
-        if (grid->angle_deg[Z] < 0.0) grid->angle_deg[Z] = 0.0;
-        if (grid->angle_deg[X] > 89.0) grid->angle_deg[X] = 89.0;
-        grid->angle_rad[Z] = deg_to_rad(grid->angle_deg[Z]);
-        grid->tan_angle[Z] = tan(grid->angle_rad[Z]);
-        sp_canvas_item_request_update (item);
-        break;
-    case ARG_COLOR:
-        grid->color = GTK_VALUE_INT (* arg);
-        sp_canvas_item_request_update (item);
-        break;
-    case ARG_EMPCOLOR:
-        grid->empcolor = GTK_VALUE_INT (* arg);
-        sp_canvas_item_request_update (item);
-        break;
-    case ARG_EMPSPACING:
-        grid->empspacing = GTK_VALUE_INT (* arg);
-        // std::cout << "Emphasis Spacing: " << grid->empspacing << std::endl;
-        sp_canvas_item_request_update (item);
-        break;
-    default:
-        break;
-    }
-}
-
-
-
 /**
     \brief  This function renders a pixel on a particular buffer.
-                
+
     The topleft of the buffer equals
                         ( rect.x0 , rect.y0 )  in screen coordinates
                         ( 0 , 0 )  in setpixel coordinates
@@ -195,32 +59,32 @@ sp_caxonomgrid_set_arg (GtkObject *object, GtkArg *arg, guint arg_id)
                         ( rect.x1 , rect,y1 )  in screen coordinates
                         ( rect.x1 - rect.x0 , rect.y1 - rect.y0 )  in setpixel coordinates
 */
-static void 
+static void
 sp_caxonomgrid_setpixel (SPCanvasBuf *buf, gint x, gint y, guint32 rgba) {
 #ifdef SAFE_SETPIXEL
     if ( (x >= buf->rect.x0) && (x < buf->rect.x1) && (y >= buf->rect.y0) && (y < buf->rect.y1) ) {
-#endif        
-        guint r, g, b, a;          
+#endif
+        guint r, g, b, a;
         r = NR_RGBA32_R (rgba);
         g = NR_RGBA32_G (rgba);
         b = NR_RGBA32_B (rgba);
-        a = NR_RGBA32_A (rgba);  
+        a = NR_RGBA32_A (rgba);
         guchar * p = buf->buf + (y - buf->rect.y0) * buf->buf_rowstride + (x - buf->rect.x0) * 3;
         p[0] = NR_COMPOSEN11_1111 (r, a, p[0]);
         p[1] = NR_COMPOSEN11_1111 (g, a, p[1]);
         p[2] = NR_COMPOSEN11_1111 (b, a, p[2]);
 #ifdef SAFE_SETPIXEL
     }
-#endif    
+#endif
 }
 
 /**
     \brief  This function renders a line on a particular canvas buffer,
             using Bresenham's line drawing function.
-            http://www.cs.unc.edu/~mcmillan/comp136/Lecture6/Lines.html 
+            http://www.cs.unc.edu/~mcmillan/comp136/Lecture6/Lines.html
             Coordinates are interpreted as SCREENcoordinates
 */
-static void 
+static void
 sp_caxonomgrid_drawline (SPCanvasBuf *buf, gint x0, gint y0, gint x1, gint y1, guint32 rgba) {
     int dy = y1 - y0;
     int dx = x1 - x0;
@@ -255,7 +119,7 @@ sp_caxonomgrid_drawline (SPCanvasBuf *buf, gint x0, gint y0, gint x1, gint y1, g
             sp_caxonomgrid_setpixel(buf, x0, y0, rgba);
         }
     }
-    
+
 }
 
 static void
@@ -281,182 +145,524 @@ sp_grid_vline (SPCanvasBuf *buf, gint x, gint ys, gint ye, guint32 rgba)
     }
 }
 
+namespace Inkscape {
+
+
 /**
-    \brief  This function renders the grid on a particular canvas buffer
-    \param  item  The grid to render on the buffer
-    \param  buf   The buffer to render the grid on
-    
-    This function gets called a touch more than you might believe,
-    about once per tile.  This means that it could probably be optimized
-    and help things out.
-
-    Basically this function has to determine where in the canvas it is,
-    and how that associates with the grid.  It does this first by looking
-    at the bounding box of the buffer, and then calculates where the grid
-    starts in that buffer.  It will then step through grid lines until
-    it is outside of the buffer.
-
-    For each grid line it is drawn using the function \c sp_grid_hline
-    or \c sp_grid_vline.  These are convience functions for the sake
-    of making the function easier to read.
-
-    Also, there are emphasized lines on the grid.  While the \c syg and
-    \c sxg variable track grid positioning, the \c xlinestart and \c
-    ylinestart variables track the 'count' of what lines they are.  If
-    that count is a multiple of the line seperation between emphasis
-    lines, then that line is drawn in the emphasis color.
-*/
-static void
-sp_caxonomgrid_render (SPCanvasItem * item, SPCanvasBuf * buf)
+* A DIRECT COPY-PASTE FROM DOCUMENT-PROPERTIES.CPP  TO QUICKLY GET RESULTS
+*
+ * Helper function that attachs widgets in a 3xn table. The widgets come in an
+ * array that has two entries per table row. The two entries code for four
+ * possible cases: (0,0) means insert space in first column; (0, non-0) means
+ * widget in columns 2-3; (non-0, 0) means label in columns 1-3; and
+ * (non-0, non-0) means two widgets in columns 2 and 3.
+**/
+#define SPACE_SIZE_X 15
+#define SPACE_SIZE_Y 10
+static inline void
+attach_all (Gtk::Table &table, const Gtk::Widget *arr[], unsigned size, int start = 0)
+{
+    for (unsigned i=0, r=start; i<size/sizeof(Gtk::Widget*); i+=2)
+    {
+        if (arr[i] && arr[i+1])
+        {
+            table.attach (const_cast<Gtk::Widget&>(*arr[i]),   1, 2, r, r+1,
+                      Gtk::FILL|Gtk::EXPAND, (Gtk::AttachOptions)0,0,0);
+            table.attach (const_cast<Gtk::Widget&>(*arr[i+1]), 2, 3, r, r+1,
+                      Gtk::FILL|Gtk::EXPAND, (Gtk::AttachOptions)0,0,0);
+        }
+        else
+        {
+            if (arr[i+1])
+                table.attach (const_cast<Gtk::Widget&>(*arr[i+1]), 1, 3, r, r+1,
+                      Gtk::FILL|Gtk::EXPAND, (Gtk::AttachOptions)0,0,0);
+            else if (arr[i])
+            {
+                Gtk::Label& label = reinterpret_cast<Gtk::Label&> (const_cast<Gtk::Widget&>(*arr[i]));
+                label.set_alignment (0.0);
+                table.attach (label, 0, 3, r, r+1,
+                      Gtk::FILL|Gtk::EXPAND, (Gtk::AttachOptions)0,0,0);
+            }
+            else
+            {
+                Gtk::HBox *space = manage (new Gtk::HBox);
+                space->set_size_request (SPACE_SIZE_X, SPACE_SIZE_Y);
+                table.attach (*space, 0, 1, r, r+1,
+                      (Gtk::AttachOptions)0, (Gtk::AttachOptions)0,0,0);
+            }
+        }
+        ++r;
+    }
+}
+
+CanvasAxonomGrid::CanvasAxonomGrid (SPDesktop *desktop, Inkscape::XML::Node * in_repr)
+    : CanvasGrid(desktop, in_repr), table(1, 1)
+{
+
+    origin[NR::X] = origin[NR::Y] = 0.0;
+//            nv->gridcolor = (nv->gridcolor & 0xff) | (DEFAULTGRIDCOLOR & 0xffffff00);
+//     case SP_ATTR_GRIDOPACITY:
+//            nv->gridcolor = (nv->gridcolor & 0xffffff00) | (DEFAULTGRIDCOLOR & 0xff);
+    color = 0xff3f3f20;
+    empcolor = 0xFF3F3F40;
+    empspacing = 5;
+    gridunit = &sp_unit_get_by_id(SP_UNIT_PX);
+    angle_deg[X] = angle_deg[Z] = 30;
+    angle_deg[Y] =0;
+    lengthy = 1;
+
+    angle_rad[X] = deg_to_rad(angle_deg[X]);
+    tan_angle[X] = tan(angle_rad[X]);
+    angle_rad[Z] = deg_to_rad(angle_deg[Z]);
+    tan_angle[Z] = tan(angle_rad[Z]);
+
+    snapper = new CanvasAxonomGridSnapper(this, namedview, 0);
+
+    // initialize widgets:
+    vbox.set_border_width(2);
+    table.set_spacings(2);
+    vbox.pack_start(table, false, false, 0);
+
+    _rumg.init (_("Grid _units:"), "units", _wr, repr);
+    _rsu_ox.init (_("_Origin X:"), _("X coordinate of grid origin"),
+                  "originx", _rumg, _wr, repr);
+    _rsu_oy.init (_("O_rigin Y:"), _("Y coordinate of grid origin"),
+                  "originy", _rumg, _wr, repr);
+    _rsu_sy.init (_("Spacing _Y:"), _("Base length of z-axis"),
+                  "spacingy", _rumg, _wr, repr);
+    _rsu_ax.init (_("Angle X:"), _("Angle of x-axis"),
+                  "gridanglex", _rumg, _wr, repr);
+    _rsu_az.init (_("Angle Z:"), _("Angle of z-axis"),
+                  "gridanglez", _rumg, _wr, repr);
+    _rcp_gcol.init (_("Grid line _color:"), _("Grid line color"),
+                    _("Color of grid lines"), "color", "opacity", _wr, repr);
+    _rcp_gmcol.init (_("Ma_jor grid line color:"), _("Major grid line color"),
+                     _("Color of the major (highlighted) grid lines"),
+                     "empcolor", "empopacity", _wr, repr);
+    _rsi.init (_("_Major grid line every:"), _("lines"), "empspacing", _wr, repr);
+
+    const Gtk::Widget* widget_array[] =
+    {
+        0,                  _rcbgrid._button,
+        _rumg._label,       _rumg._sel,
+        0,                  _rsu_ox.getSU(),
+        0,                  _rsu_oy.getSU(),
+        0,                  _rsu_sy.getSU(),
+        0,                  _rsu_ax.getSU(),
+        0,                  _rsu_az.getSU(),
+        _rcp_gcol._label,   _rcp_gcol._cp,
+        0,                  0,
+        _rcp_gmcol._label,  _rcp_gmcol._cp,
+        _rsi._label,        &_rsi._hbox,
+    };
+
+    attach_all (table, widget_array, sizeof(widget_array));
+
+    vbox.show();
+
+    if (repr) readRepr();
+    updateWidgets();
+}
+
+CanvasAxonomGrid::~CanvasAxonomGrid ()
+{
+   if (snapper) delete snapper;
+}
+
+
+/* fixme: Collect all these length parsing methods and think common sane API */
+
+static gboolean sp_nv_read_length(const gchar *str, guint base, gdouble *val, const SPUnit **unit)
+{
+    if (!str) {
+        return FALSE;
+    }
+
+    gchar *u;
+    gdouble v = g_ascii_strtod(str, &u);
+    if (!u) {
+        return FALSE;
+    }
+    while (isspace(*u)) {
+        u += 1;
+    }
+
+    if (!*u) {
+        /* No unit specified - keep default */
+        *val = v;
+        return TRUE;
+    }
+
+    if (base & SP_UNIT_DEVICE) {
+        if (u[0] && u[1] && !isalnum(u[2]) && !strncmp(u, "px", 2)) {
+            *unit = &sp_unit_get_by_id(SP_UNIT_PX);
+            *val = v;
+            return TRUE;
+        }
+    }
+
+    if (base & SP_UNIT_ABSOLUTE) {
+        if (!strncmp(u, "pt", 2)) {
+            *unit = &sp_unit_get_by_id(SP_UNIT_PT);
+        } else if (!strncmp(u, "mm", 2)) {
+            *unit = &sp_unit_get_by_id(SP_UNIT_MM);
+        } else if (!strncmp(u, "cm", 2)) {
+            *unit = &sp_unit_get_by_id(SP_UNIT_CM);
+        } else if (!strncmp(u, "m", 1)) {
+            *unit = &sp_unit_get_by_id(SP_UNIT_M);
+        } else if (!strncmp(u, "in", 2)) {
+            *unit = &sp_unit_get_by_id(SP_UNIT_IN);
+        } else {
+            return FALSE;
+        }
+        *val = v;
+        return TRUE;
+    }
+
+    return FALSE;
+}
+
+static gboolean sp_nv_read_opacity(const gchar *str, guint32 *color)
 {
-    SPCAxonomGrid *grid = SP_CAXONOMGRID (item);
+    if (!str) {
+        return FALSE;
+    }
+
+    gchar *u;
+    gdouble v = g_ascii_strtod(str, &u);
+    if (!u) {
+        return FALSE;
+    }
+    v = CLAMP(v, 0.0, 1.0);
+
+    *color = (*color & 0xffffff00) | (guint32) floor(v * 255.9999);
+
+    return TRUE;
+}
+
+
+
+void
+CanvasAxonomGrid::readRepr()
+{
+    gchar const* value;
+    if ( (value = repr->attribute("originx")) ) {
+        sp_nv_read_length(value, SP_UNIT_ABSOLUTE | SP_UNIT_DEVICE, &origin[NR::X], &gridunit);
+        origin[NR::X] = sp_units_get_pixels(origin[NR::X], *(gridunit));
+    }
+    if ( (value = repr->attribute("originy")) ) {
+        sp_nv_read_length(value, SP_UNIT_ABSOLUTE | SP_UNIT_DEVICE, &origin[NR::Y], &gridunit);
+        origin[NR::Y] = sp_units_get_pixels(origin[NR::Y], *(gridunit));
+    }
 
-    sp_canvas_prepare_buffer (buf);
-              
+    if ( (value = repr->attribute("spacingy")) ) {
+        sp_nv_read_length(value, SP_UNIT_ABSOLUTE | SP_UNIT_DEVICE, &lengthy, &gridunit);
+        lengthy = sp_units_get_pixels(lengthy, *(gridunit));
+        if (lengthy < 1.0) lengthy = 1.0;
+    }
+
+    if ( (value = repr->attribute("gridanglex")) ) {
+        angle_deg[X] = g_ascii_strtod(value, NULL);
+        if (angle_deg[X] < 1.0) angle_deg[X] = 1.0;
+        if (angle_deg[X] > 89.0) angle_deg[X] = 89.0;
+        angle_rad[X] = deg_to_rad(angle_deg[X]);
+        tan_angle[X] = tan(angle_rad[X]);
+    }
+
+    if ( (value = repr->attribute("gridanglez")) ) {
+        angle_deg[Z] = g_ascii_strtod(value, NULL);
+        if (angle_deg[Z] < 1.0) angle_deg[Z] = 1.0;
+        if (angle_deg[Z] > 89.0) angle_deg[Z] = 89.0;
+        angle_rad[Z] = deg_to_rad(angle_deg[Z]);
+        tan_angle[Z] = tan(angle_rad[Z]);
+    }
+
+    if ( (value = repr->attribute("color")) ) {
+        color = (color & 0xff) | sp_svg_read_color(value, color);
+    }
+
+    if ( (value = repr->attribute("empcolor")) ) {
+        empcolor = (empcolor & 0xff) | sp_svg_read_color(value, empcolor);
+    }
+
+    if ( (value = repr->attribute("opacity")) ) {
+        sp_nv_read_opacity(value, &color);
+    }
+    if ( (value = repr->attribute("empopacity")) ) {
+        sp_nv_read_opacity(value, &empcolor);
+    }
+
+    if ( (value = repr->attribute("empspacing")) ) {
+        empspacing = atoi(value);
+    }
+
+    sp_canvas_item_request_update (canvasitem);
+
+    return;
+}
+
+/**
+ * Called when XML node attribute changed; updates dialog widgets if change was not done by widgets themselves.
+ */
+void
+CanvasAxonomGrid::onReprAttrChanged (Inkscape::XML::Node * repr, const gchar *key, const gchar *oldval, const gchar *newval, bool is_interactive)
+{
+    readRepr();
+
+    if ( ! (_wr.isUpdating()) )
+        updateWidgets();
+}
+
+
+
+
+Gtk::Widget &
+CanvasAxonomGrid::getWidget()
+{
+    return vbox;
+}
+
+
+/**
+ * Update dialog widgets from object's values.
+ */
+void
+CanvasAxonomGrid::updateWidgets()
+{
+    if (_wr.isUpdating()) return;
+
+    _wr.setUpdating (true);
+
+//    _rrb_gridtype.setValue (nv->gridtype);
+    _rumg.setUnit (gridunit);
+
+    gdouble val;
+    val = origin[NR::X];
+    val = sp_pixels_get_units (val, *(gridunit));
+    _rsu_ox.setValue (val);
+    val = origin[NR::Y];
+    val = sp_pixels_get_units (val, *(gridunit));
+    _rsu_oy.setValue (val);
+    val = lengthy;
+    double gridy = sp_pixels_get_units (val, *(gridunit));
+    _rsu_sy.setValue (gridy);
+
+    _rsu_ax.setValue(angle_deg[X]);
+    _rsu_az.setValue(angle_deg[Z]);
+
+    _rcp_gcol.setRgba32 (color);
+    _rcp_gmcol.setRgba32 (empcolor);
+    _rsi.setValue (empspacing);
+
+    _wr.setUpdating (false);
+
+    return;
+}
+
+
+
+void
+CanvasAxonomGrid::Update (NR::Matrix const &affine, unsigned int flags)
+{
+    ow = origin * affine;
+    sw = NR::Point(fabs(affine[0]),fabs(affine[3]));
+    
+    for(int dim = 0; dim < 2; dim++) {
+        gint scaling_factor = empspacing;
+
+        if (scaling_factor <= 1)
+            scaling_factor = 5;
+
+        scaled = FALSE;
+        int watchdog = 0;
+        while (  (sw[dim] < 8.0) & (watchdog < 100) ) {
+            scaled = TRUE;
+            sw[dim] *= scaling_factor;
+            // First pass, go up to the major line spacing, then
+            // keep increasing by two.
+            scaling_factor = 2;
+            watchdog++;
+        }
+
+    }
+
+    spacing_ylines = sw[NR::X] * lengthy  /(tan_angle[X] + tan_angle[Z]);
+    lyw            = lengthy * sw[NR::Y];
+    lxw_x          = (lengthy / tan_angle[X]) * sw[NR::X];
+    lxw_z          = (lengthy / tan_angle[Z]) * sw[NR::X];
+
+    if (empspacing == 0) {
+        scaled = TRUE;
+    }
+
+}
+
+void
+CanvasAxonomGrid::Render (SPCanvasBuf *buf)
+{
      // gc = gridcoordinates (the coordinates calculated from the grids origin 'grid->ow'.
      // sc = screencoordinates ( for example "buf->rect.x0" is in screencoordinates )
-     // bc = buffer patch coordinates 
-     
+     // bc = buffer patch coordinates
+
      // tl = topleft ; br = bottomright
     NR::Point buf_tl_gc;
     NR::Point buf_br_gc;
-    buf_tl_gc[NR::X] = buf->rect.x0 - grid->ow[NR::X];
-    buf_tl_gc[NR::Y] = buf->rect.y0 - grid->ow[NR::Y];
-    buf_br_gc[NR::X] = buf->rect.x1 - grid->ow[NR::X];
-    buf_br_gc[NR::Y] = buf->rect.y1 - grid->ow[NR::Y];
-
+    buf_tl_gc[NR::X] = buf->rect.x0 - ow[NR::X];
+    buf_tl_gc[NR::Y] = buf->rect.y0 - ow[NR::Y];
+    buf_br_gc[NR::X] = buf->rect.x1 - ow[NR::X];
+    buf_br_gc[NR::Y] = buf->rect.y1 - ow[NR::Y];
 
     gdouble x;
     gdouble y;
 
     // render the three separate line groups representing the main-axes:
-    // x-axis always goes from topleft to bottomright. (0,0) - (1,1)  
-    const gdouble xintercept_y_bc = (buf_tl_gc[NR::X] * grid->tan_angle[X]) - buf_tl_gc[NR::Y] ;
-    const gdouble xstart_y_sc = ( xintercept_y_bc - floor(xintercept_y_bc/grid->lyw)*grid->lyw ) + buf->rect.y0;
-    const gint  xlinestart = (gint) Inkscape::round( (xstart_y_sc - grid->ow[NR::Y]) / grid->lyw );
+    // x-axis always goes from topleft to bottomright. (0,0) - (1,1)
+    const gdouble xintercept_y_bc = (buf_tl_gc[NR::X] * tan_angle[X]) - buf_tl_gc[NR::Y] ;
+    const gdouble xstart_y_sc = ( xintercept_y_bc - floor(xintercept_y_bc/lyw)*lyw ) + buf->rect.y0;
+    const gint  xlinestart = (gint) Inkscape::round( (xstart_y_sc - ow[NR::Y]) / lyw );
     gint xlinenum;
     // lijnen vanaf linker zijkant.
-    for (y = xstart_y_sc, xlinenum = xlinestart; y < buf->rect.y1; y += grid->lyw, xlinenum++) {
+    for (y = xstart_y_sc, xlinenum = xlinestart; y < buf->rect.y1; y += lyw, xlinenum++) {
         const gint x0 = buf->rect.x0;
         const gint y0 = (gint) Inkscape::round(y);
-        const gint x1 = x0 + (gint) Inkscape::round( (buf->rect.y1 - y) / grid->tan_angle[X] );
+        const gint x1 = x0 + (gint) Inkscape::round( (buf->rect.y1 - y) / tan_angle[X] );
         const gint y1 = buf->rect.y1;
-            
-        if (!grid->scaled && (xlinenum % grid->empspacing) == 0) {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->empcolor);
+
+        if (!scaled && (xlinenum % empspacing) == 0) {
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, empcolor);
         } else {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->color);
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, color);
         }
     }
     // lijnen vanaf bovenkant.
-    const gdouble xstart_x_sc = buf->rect.x0 + (grid->lxw_x - (xstart_y_sc - buf->rect.y0) / grid->tan_angle[X]) ;
-    for (x = xstart_x_sc, xlinenum = xlinestart; x < buf->rect.x1; x += grid->lxw_x, xlinenum--) {
+    const gdouble xstart_x_sc = buf->rect.x0 + (lxw_x - (xstart_y_sc - buf->rect.y0) / tan_angle[X]) ;
+    for (x = xstart_x_sc, xlinenum = xlinestart; x < buf->rect.x1; x += lxw_x, xlinenum--) {
         const gint y0 = buf->rect.y0;
         const gint y1 = buf->rect.y1;
         const gint x0 = (gint) Inkscape::round(x);
-        const gint x1 = x0 + (gint) Inkscape::round( (y1 - y0) / grid->tan_angle[X] );
-            
-        if (!grid->scaled && (xlinenum % grid->empspacing) == 0) {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->empcolor);
+        const gint x1 = x0 + (gint) Inkscape::round( (y1 - y0) / tan_angle[X] );
+
+        if (!scaled && (xlinenum % empspacing) == 0) {
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, empcolor);
         } else {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->color);
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, color);
         }
     }
-    
+
 
     // y-axis lines (vertical)
-    const gdouble ystart_x_sc = floor (buf_tl_gc[NR::X] / grid->spacing_ylines) * grid->spacing_ylines + grid->ow[NR::X];
-    const gint  ylinestart = (gint) Inkscape::round((ystart_x_sc - grid->ow[NR::X]) / grid->spacing_ylines);
+    const gdouble ystart_x_sc = floor (buf_tl_gc[NR::X] / spacing_ylines) * spacing_ylines + ow[NR::X];
+    const gint  ylinestart = (gint) Inkscape::round((ystart_x_sc - ow[NR::X]) / spacing_ylines);
     gint ylinenum;
-    for (x = ystart_x_sc, ylinenum = ylinestart; x < buf->rect.x1; x += grid->spacing_ylines, ylinenum++) {
+    for (x = ystart_x_sc, ylinenum = ylinestart; x < buf->rect.x1; x += spacing_ylines, ylinenum++) {
         const gint x0 = (gint) Inkscape::round(x);
 
-        if (!grid->scaled && (ylinenum % grid->empspacing) == 0) {
-            sp_grid_vline (buf, x0, buf->rect.y0, buf->rect.y1 - 1, grid->empcolor);
+        if (!scaled && (ylinenum % empspacing) == 0) {
+            sp_grid_vline (buf, x0, buf->rect.y0, buf->rect.y1 - 1, empcolor);
         } else {
-            sp_grid_vline (buf, x0, buf->rect.y0, buf->rect.y1 - 1, grid->color);
+            sp_grid_vline (buf, x0, buf->rect.y0, buf->rect.y1 - 1, color);
         }
     }
 
-    // z-axis always goes from bottomleft to topright. (0,1) - (1,0)  
-    const gdouble zintercept_y_bc = (buf_tl_gc[NR::X] * -grid->tan_angle[Z]) - buf_tl_gc[NR::Y] ;
-    const gdouble zstart_y_sc = ( zintercept_y_bc - floor(zintercept_y_bc/grid->lyw)*grid->lyw ) + buf->rect.y0;
-    const gint  zlinestart = (gint) Inkscape::round( (zstart_y_sc - grid->ow[NR::Y]) / grid->lyw );
+    // z-axis always goes from bottomleft to topright. (0,1) - (1,0)
+    const gdouble zintercept_y_bc = (buf_tl_gc[NR::X] * -tan_angle[Z]) - buf_tl_gc[NR::Y] ;
+    const gdouble zstart_y_sc = ( zintercept_y_bc - floor(zintercept_y_bc/lyw)*lyw ) + buf->rect.y0;
+    const gint  zlinestart = (gint) Inkscape::round( (zstart_y_sc - ow[NR::Y]) / lyw );
     gint zlinenum;
     // lijnen vanaf linker zijkant.
-    for (y = zstart_y_sc, zlinenum = zlinestart; y < buf->rect.y1; y += grid->lyw, zlinenum++) {
+    for (y = zstart_y_sc, zlinenum = zlinestart; y < buf->rect.y1; y += lyw, zlinenum++) {
         const gint x0 = buf->rect.x0;
         const gint y0 = (gint) Inkscape::round(y);
-        const gint x1 = x0 + (gint) Inkscape::round( (y - buf->rect.y0 ) / grid->tan_angle[Z] );
+        const gint x1 = x0 + (gint) Inkscape::round( (y - buf->rect.y0 ) / tan_angle[Z] );
         const gint y1 = buf->rect.y0;
-            
-        if (!grid->scaled && (zlinenum % grid->empspacing) == 0) {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->empcolor);
+
+        if (!scaled && (zlinenum % empspacing) == 0) {
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, empcolor);
         } else {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->color);
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, color);
         }
     }
-    // lijnen vanaf onderkant.
-    const gdouble zstart_x_sc = buf->rect.x0 + (y - buf->rect.y1) / grid->tan_angle[Z] ;
-    for (x = zstart_x_sc; x < buf->rect.x1; x += grid->lxw_z, zlinenum--) {
+    // draw lines from bottom-up
+    const gdouble zstart_x_sc = buf->rect.x0 + (y - buf->rect.y1) / tan_angle[Z] ;
+    for (x = zstart_x_sc; x < buf->rect.x1; x += lxw_z, zlinenum--) {
         const gint y0 = buf->rect.y1;
         const gint y1 = buf->rect.y0;
         const gint x0 = (gint) Inkscape::round(x);
-        const gint x1 = x0 + (gint) Inkscape::round( (buf->rect.y1 - buf->rect.y0) / grid->tan_angle[Z] );
-            
-        if (!grid->scaled && (zlinenum % grid->empspacing) == 0) {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->empcolor);
+        const gint x1 = x0 + (gint) Inkscape::round( (buf->rect.y1 - buf->rect.y0) / tan_angle[Z] );
+
+        if (!scaled && (zlinenum % empspacing) == 0) {
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, empcolor);
         } else {
-            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, grid->color);
+            sp_caxonomgrid_drawline (buf, x0, y0, x1, y1, color);
         }
     }
-    
 }
 
-static void
-sp_caxonomgrid_update (SPCanvasItem *item, NR::Matrix const &affine, unsigned int flags)
+
+
+
+
+
+
+
+
+
+
+
+/**
+ * \return x rounded to the nearest multiple of c1 plus c0.
+ *
+ * \note
+ * If c1==0 (and c0 is finite), then returns +/-inf.  This makes grid spacing of zero
+ * mean "ignore the grid in this dimention".  We're currently discussing "good" semantics
+ * for guide/grid snapping.
+ */
+
+/* FIXME: move this somewhere else, perhaps */
+static double round_to_nearest_multiple_plus(double x, double const c1, double const c0)
 {
-    SPCAxonomGrid *grid = SP_CAXONOMGRID (item);
+    return floor((x - c0) / c1 + .5) * c1 + c0;
+}
 
-    if (parent_class->update)
-        (* parent_class->update) (item, affine, flags);
+CanvasAxonomGridSnapper::CanvasAxonomGridSnapper(CanvasAxonomGrid *grid, SPNamedView const *nv, NR::Coord const d) : LineSnapper(nv, d)
+{
+    this->grid = grid;
+}
 
-    grid->ow = grid->origin * affine;
-    grid->sw = NR::Point(fabs(affine[0]),fabs(affine[3]));
-    
-    for(int dim = 0; dim < 2; dim++) {
-        gint scaling_factor = grid->empspacing;
+LineSnapper::LineList
+CanvasAxonomGridSnapper::_getSnapLines(NR::Point const &p) const
+{
+    LineList s;
 
-        if (scaling_factor <= 1)
-            scaling_factor = 5;
+    if ( grid == NULL ) {
+        return s;
+    }
 
-        grid->scaled = FALSE;
-        while (grid->sw[dim] < 8.0) {
-            grid->scaled = TRUE;
-            grid->sw[dim] *= scaling_factor;
-            // First pass, go up to the major line spacing, then
-            // keep increasing by two.
-            scaling_factor = 2;
+    for (unsigned int i = 0; i < 2; ++i) {
+
+        /* This is to make sure we snap to only visible grid lines */
+        double scaled_spacing = grid->sw[i]; // this is spacing of visible lines if screen pixels
+
+        // convert screen pixels to px
+        // FIXME: after we switch to snapping dist in screen pixels, this will be unnecessary
+        if (SP_ACTIVE_DESKTOP) {
+            scaled_spacing /= SP_ACTIVE_DESKTOP->current_zoom();
         }
-    }
 
-    grid->spacing_ylines = grid->sw[NR::X] * grid->lengthy  /(grid->tan_angle[X] + grid->tan_angle[Z]);
-    grid->lyw            = grid->lengthy * grid->sw[NR::Y];
-    grid->lxw_x          = (grid->lengthy / grid->tan_angle[X]) * grid->sw[NR::X];
-    grid->lxw_z          = (grid->lengthy / grid->tan_angle[Z]) * grid->sw[NR::X];
+        NR::Coord const rounded = round_to_nearest_multiple_plus(p[i],
+                                                                 scaled_spacing,
+                                                                 grid->origin[i]);
 
-    if (grid->empspacing == 0) {
-        grid->scaled = TRUE;
+        s.push_back(std::make_pair(NR::Dim2(i), rounded));
     }
 
-    sp_canvas_request_redraw (item->canvas,
-                     -1000000, -1000000,
-                     1000000, 1000000);
-                     
-    item->x1 = item->y1 = -1000000;
-    item->x2 = item->y2 = 1000000;
+    return s;
 }
 
+
+}; // namespace Inkscape
+
+
 /*
   Local Variables:
   mode:c++
index 79de72b43340abe7adf65eba7980445207a3f723..d560dad7c9ca49dabf79a5a54a8e99215dffa1c1 100644 (file)
-#ifndef SP_CANVAS_AXONOMGRID_H
-#define SP_CANVAS_AXONOMGRID_H
+#ifndef CANVAS_AXONOMGRID_H
+#define CANVAS_AXONOMGRID_H
 
 /*
- * SPCAxonomGrid
+ * Copyright (C) 2006-2007 Johan Engelen  <johan@shouraizou.nl>
  *
- * Generic (and quite unintelligent) modified copy of the grid item for gnome canvas
- *
- * Copyright (C) 2006 Johan Engelen  <johan@shouraizou.nl>
- * Copyright (C) 2000 Lauris Kaplinski 2000
- *
- */
+  */
 
 #include <display/sp-canvas.h>
 #include <libnr/nr-coord.h>
+#include "xml/repr.h"
+#include <gtkmm/box.h>
+
+#include <gtkmm.h>
+#include "ui/widget/color-picker.h"
+#include "ui/widget/scalar-unit.h"
+
+#include "ui/widget/registered-widget.h"
+#include "ui/widget/registry.h"
+#include "ui/widget/tolerance-slider.h"
+
+#include "xml/node-event-vector.h"
+
+#include "snapper.h"
+#include "line-snapper.h"
+
+#include "canvas-grid.h"
+
+struct SPDesktop;
+struct SPNamedView;
+
+namespace Inkscape {
 
+class CanvasAxonomGrid : public CanvasGrid {
+public:
+    CanvasAxonomGrid(SPDesktop *desktop, Inkscape::XML::Node * in_repr);
+    ~CanvasAxonomGrid();
 
-#define SP_TYPE_CAXONOMGRID            (sp_caxonomgrid_get_type ())
-#define SP_CAXONOMGRID(obj)            (GTK_CHECK_CAST ((obj), SP_TYPE_CAXONOMGRID, SPCAxonomGrid))
-#define SP_CAXONOMGRID_CLASS(klass)    (GTK_CHECK_CLASS_CAST ((klass), SP_TYPE_CAXONOMGRID, SPCAxonomGridClass))
-#define SP_IS_CAXONOMGRID(obj)         (GTK_CHECK_TYPE ((obj), SP_TYPE_CAXONOMGRID))
-#define SP_IS_CAXONOMGRID_CLASS(klass) (GTK_CHECK_CLASS_TYPE ((klass), SP_TYPE_CAXONOMGRID))
-
-
-/** \brief  All the variables that are tracked for a axonometric grid specific
-            canvas item. */
-struct SPCAxonomGrid : public SPCanvasItem{
-       NR::Point origin;     /**< Origin of the grid */
-       double lengthy;       /**< The lengths of the primary y-axis */
-       double angle_deg[3];  /**< Angle of each axis (note that angle[2] == 0) */
-       double angle_rad[3];  /**< Angle of each axis (note that angle[2] == 0) */
-       double tan_angle[3];  /**< tan(angle[.]) */
-       guint32 color;        /**< Color for normal lines */
-       guint32 empcolor;     /**< Color for emphasis lines */
-       gint empspacing;      /**< Spacing between emphasis lines */
-       bool scaled;          /**< Whether the grid is in scaled mode */
-       
-       NR::Point ow;         /**< Transformed origin by the affine for the zoom */
-       double lyw;           /**< Transformed length y by the affine for the zoom */
-       double lxw_x;
-       double lxw_z;
-       double spacing_ylines;
+    void Update (NR::Matrix const &affine, unsigned int flags);
+    void Render (SPCanvasBuf *buf);
+    
+    void readRepr();
+    void onReprAttrChanged (Inkscape::XML::Node * repr, const gchar *key, const gchar *oldval, const gchar *newval, bool is_interactive);
+    
+    Gtk::Widget & getWidget();
+
+    SPUnit const* gridunit;
+
+    NR::Point origin;     /**< Origin of the grid */
+    double lengthy;       /**< The lengths of the primary y-axis */
+    double angle_deg[3];  /**< Angle of each axis (note that angle[2] == 0) */
+    double angle_rad[3];  /**< Angle of each axis (note that angle[2] == 0) */
+    double tan_angle[3];  /**< tan(angle[.]) */
+    guint32 color;        /**< Color for normal lines */
+    guint32 empcolor;     /**< Color for emphasis lines */
+    gint empspacing;      /**< Spacing between emphasis lines */
+    bool scaled;          /**< Whether the grid is in scaled mode */
+
+    NR::Point ow;         /**< Transformed origin by the affine for the zoom */
+    double lyw;           /**< Transformed length y by the affine for the zoom */
+    double lxw_x;
+    double lxw_z;
+    double spacing_ylines;
                           
     NR::Point sw;          /**< the scaling factors of the affine transform */
-};
+    
+    
+private:
+    CanvasAxonomGrid(const CanvasAxonomGrid&);
+    CanvasAxonomGrid& operator=(const CanvasAxonomGrid&);
+    
+    void updateWidgets();
+
+    Gtk::Table table;
+    
+    Inkscape::UI::Widget::RegisteredCheckButton _rcbgrid, _rcbsnbb, _rcbsnnod;
+    Inkscape::UI::Widget::RegisteredUnitMenu    _rumg, _rums;
+    Inkscape::UI::Widget::RegisteredScalarUnit  _rsu_ox, _rsu_oy, _rsu_sy, _rsu_ax, _rsu_az;
+    Inkscape::UI::Widget::RegisteredColorPicker _rcp_gcol, _rcp_gmcol;
+    Inkscape::UI::Widget::RegisteredSuffixedInteger _rsi;
+    
+    Inkscape::UI::Widget::Registry _wr;
 
-struct SPCAxonomGridClass {
-       SPCanvasItemClass parent_class;
 };
 
 
-/* Standard Gtk function */
-GtkType sp_caxonomgrid_get_type (void);
+
+class CanvasAxonomGridSnapper : public LineSnapper
+{
+public:
+    CanvasAxonomGridSnapper(CanvasAxonomGrid *grid, SPNamedView const *nv, NR::Coord const d);
+
+private:    
+    LineList _getSnapLines(NR::Point const &p) const;
+    
+    CanvasAxonomGrid *grid;
+}; 
+
+
+}; //namespace Inkscape
 
 
 
index 853fc011e522371b0c6907c248b0a870ca312c3b..643ffccdc1aac7b9ab80b77e785caec59386283a 100644 (file)
@@ -9,7 +9,6 @@
 
 
 #include "sp-canvas-util.h"
-#include "canvas-grid.h"
 #include "display-forward.h"
 #include <libnr/nr-pixops.h>
 #include "desktop-handles.h"
 #include "sp-namedview.h"
 #include "inkscape.h"
 #include "desktop.h"
-#include "display/canvas-grid.h"
-#include "display/canvas-axonomgrid.h"
-#include "../document.h"
 
+#include "../document.h"
 
+#include "canvas-grid.h"
+#include "canvas-axonomgrid.h"
 
 namespace Inkscape {
 
@@ -184,9 +183,7 @@ CanvasGrid::writeNewGridToRepr(Inkscape::XML::Node * repr, const char * gridtype
     Inkscape::XML::Document *xml_doc = sp_document_repr_doc(sp_desktop_document(SP_ACTIVE_DESKTOP));
     Inkscape::XML::Node *newnode;
     newnode = xml_doc->createElement("inkscape:grid");
-    if (!strcmp(gridtype,"xygrid")) {
-        newnode->setAttribute("type","xygrid");
-    }
+    newnode->setAttribute("type",gridtype);
     
     repr->appendChild(newnode);
 
@@ -207,6 +204,8 @@ CanvasGrid::NewGrid(SPDesktop *desktop, Inkscape::XML::Node * in_repr, const cha
     
     if (!strcmp(gridtype,"xygrid")) {
         return (CanvasGrid*) new CanvasXYGrid(desktop, in_repr);
+    } else if (!strcmp(gridtype,"axonometric")) {
+        return (CanvasGrid*) new CanvasAxonomGrid(desktop, in_repr);
     }
     
     return NULL;
@@ -331,7 +330,6 @@ CanvasXYGrid::CanvasXYGrid (SPDesktop *desktop, Inkscape::XML::Node * in_repr)
     const Gtk::Widget* widget_array[] =
     {
         0,                  _rcbgrid._button,
-        0,                  _rrb_gridtype._hbox,
         _rumg._label,       _rumg._sel,
         0,                  _rsu_ox.getSU(),
         0,                  _rsu_oy.getSU(),
index f92c731cdfb761518e844b0ce4245d7639831525..c501ab3f38a9fbdc50391b3ac6b33336c48d933b 100644 (file)
@@ -131,9 +131,8 @@ private:
     Gtk::Table table;
     
     Inkscape::UI::Widget::RegisteredCheckButton _rcbgrid, _rcbsnbb, _rcbsnnod;
-    Inkscape::UI::Widget::RegisteredRadioButtonPair _rrb_gridtype;
     Inkscape::UI::Widget::RegisteredUnitMenu    _rumg, _rums;
-    Inkscape::UI::Widget::RegisteredScalarUnit  _rsu_ox, _rsu_oy, _rsu_sx, _rsu_sy, _rsu_ax, _rsu_az;
+    Inkscape::UI::Widget::RegisteredScalarUnit  _rsu_ox, _rsu_oy, _rsu_sx, _rsu_sy;
     Inkscape::UI::Widget::RegisteredColorPicker _rcp_gcol, _rcp_gmcol;
     Inkscape::UI::Widget::RegisteredSuffixedInteger _rsi;
     
index c6a802a54aecae1f9b055b7bfc078c021617497d..818f0abd9c2751af2c3e869a8920cf3ebe69051d 100644 (file)
@@ -83,66 +83,6 @@ Inkscape::GridSnapper::_getSnapLines(NR::Point const &p) const
     return s;
 }
 
-
-
-Inkscape::AxonomGridSnapper::AxonomGridSnapper(SPNamedView const *nv, NR::Coord const d) : LineSnapper(nv, d)
-{
-
-}
-
-
-Inkscape::LineSnapper::LineList 
-Inkscape::AxonomGridSnapper::_getSnapLines(NR::Point const &p) const
-{
-    LineList s;
-
-    if ( NULL == _named_view ) {
-        return s;
-    }
-
-    SPCAxonomGrid *griditem = NULL;
-    for (GSList *l = _named_view->gridviews; l != NULL; l = l->next) {
-        // FIXME : this is a hack since there is only one view for now
-        //                 but when we'll handle multiple views, snapping should
-        //                 must be rethought and maybe only the current view
-        //                 should give back it's SHOWN lines to snap to
-        //                 For now, the last SPCAxonomGrid in _named_view->gridviews will be used.
-        if ( SP_IS_CAXONOMGRID(GTK_OBJECT(l->data)) ) {
-            griditem = SP_CAXONOMGRID(l->data);
-        }
-    }
-
-    g_assert(griditem != NULL);
-
-    // add vertical line.       
-    
-
-    // This is to make sure we snap to only visible grid lines
-    double scaled_spacing = griditem->spacing_ylines; // this is spacing of visible lines if screen pixels
-    // convert screen pixels to px
-    // FIXME: after we switch to snapping dist in screen pixels, this will be unnecessary
-    if (SP_ACTIVE_DESKTOP) {
-        scaled_spacing /= SP_ACTIVE_DESKTOP->current_zoom();
-    }
-
-    NR::Coord const rounded = round_to_nearest_multiple_plus(p[0], scaled_spacing, griditem->origin[0]);
-    
-    /*
-    int a = round(scaled_spacing);
-    int b = round(p[0]);
-    int c = round(rounded);
-    
-    g_message("hier %d; %d; %d",a,b,c);
-    */
-
-    s.push_back(std::make_pair(NR::Dim2(0), rounded));
-    
-
-    return s;
-}
-
-
-
 /*
   Local Variables:
   mode:c++
index 1a3e6d3a8619203dfaffd6eb8ef2176daf08dc61..1a704dd4e8c965646bea3f89c1d610825fdf21df 100644 (file)
@@ -31,18 +31,6 @@ private:
     LineList _getSnapLines(NR::Point const &p) const;
 };           
 
-
-class AxonomGridSnapper : public LineSnapper
-{
-public:
-    AxonomGridSnapper(SPNamedView const *nv, NR::Coord const d);
-
-private:    
-    LineList _getSnapLines(NR::Point const &p) const;
-};           
-
-
-
 }
 
 #endif
index c5381f403c8c1a3527dfe49100ab7720bf1da6a8..c40d0c7da324f8f8e7aae3c9067b1d6881a08f31 100644 (file)
@@ -131,8 +131,7 @@ static char const preferences_skeleton[] =
 "    <group id=\"xml\"/>\n"
 "    <group id=\"find\"/>\n"
 "    <group \n"
-"      id=\"documentoptions\"\n"
-"      axonomgrid_enabled=\"0\" />\n"
+"      id=\"documentoptions\" />\n"
 "    <group id=\"preferences\"/>\n"
 "    <group id=\"gradienteditor\"/>\n"
 "    <group id=\"object\"/>\n"
index ff05858fd05f3b097ca17e60136f4458f3216e69..e9bdb70636405eb5789baca619086d9c4d8c7bfc 100644 (file)
@@ -31,7 +31,6 @@
 
 SnapManager::SnapManager(SPNamedView const *v) :
     grid(v, 0),
-    axonomgrid(v, 0),
     guide(v, 0),
     object(v, 0),
     _named_view(v)
@@ -47,11 +46,7 @@ SnapManager::SnapManager(SPNamedView const *v) :
 SnapManager::SnapperList SnapManager::getSnappers() const
 {
     SnapManager::SnapperList s;
-    if (_named_view->gridtype == 0) {
-      s.push_back(&grid);
-    } else {
-      s.push_back(&axonomgrid);
-    }
+    s.push_back(&grid);
     s.push_back(&guide);
     s.push_back(&object);
 
index dc2d0c7883c25d980479a20e4fffa19c87380740..09c527c749114552f59e85d4ab51699cf337345f 100644 (file)
@@ -101,7 +101,6 @@ public:
                                             NR::Dim2 d) const;
 
     Inkscape::GridSnapper grid;        ///< grid snapper
-    Inkscape::AxonomGridSnapper axonomgrid;  ///< axonometricgrid snapper
     Inkscape::GuideSnapper guide;      ///< guide snapper
     Inkscape::ObjectSnapper object;    ///< snapper to other objects
 
index c2b8aba10da4f48e5409ef067d9ce805bab46e40..a220d8401c2cb259a9039208592c8a0f8ac1132f 100644 (file)
@@ -17,7 +17,6 @@
 #include "config.h"
 
 #include "display/canvas-grid.h"
-#include "display/canvas-axonomgrid.h"
 #include "helper/units.h"
 #include "svg/svg-color.h"
 #include "xml/repr.h"
@@ -109,7 +108,6 @@ static void sp_namedview_init(SPNamedView *nv)
 {
     nv->editable = TRUE;
     nv->showgrid = FALSE;
-    nv->gridtype = 0;
     nv->showguides = TRUE;
     nv->showborder = TRUE;
     nv->showpageshadow = TRUE;
@@ -117,7 +115,7 @@ static void sp_namedview_init(SPNamedView *nv)
     nv->guides = NULL;
     nv->viewcount = 0;
     nv->grids = NULL;
-    
+
     nv->default_layer_id = 0;
 
     nv->connector_spacing = defaultConnSpacing;
@@ -137,7 +135,6 @@ static void sp_namedview_build(SPObject *object, SPDocument *document, Inkscape:
     sp_object_read_attr(object, "inkscape:document-units");
     sp_object_read_attr(object, "viewonly");
     sp_object_read_attr(object, "showgrid");
-    sp_object_read_attr(object, "gridtype");
     sp_object_read_attr(object, "showguides");
     sp_object_read_attr(object, "gridtolerance");
     sp_object_read_attr(object, "guidetolerance");
@@ -147,8 +144,6 @@ static void sp_namedview_build(SPObject *object, SPDocument *document, Inkscape:
     sp_object_read_attr(object, "gridoriginy");
     sp_object_read_attr(object, "gridspacingx");
     sp_object_read_attr(object, "gridspacingy");
-    sp_object_read_attr(object, "gridanglex");
-    sp_object_read_attr(object, "gridanglez");
     sp_object_read_attr(object, "gridempspacing");
     sp_object_read_attr(object, "gridcolor");
     sp_object_read_attr(object, "gridempcolor");
@@ -215,7 +210,7 @@ static void sp_namedview_release(SPObject *object)
         delete gr;
         namedview->grids = g_slist_remove_link(namedview->grids, namedview->grids);
     }
-    
+
     if (((SPObjectClass *) parent_class)->release) {
         ((SPObjectClass *) parent_class)->release(object);
     }
@@ -238,12 +233,6 @@ static void sp_namedview_set(SPObject *object, unsigned int key, const gchar *va
             sp_namedview_setup_grid(nv);
             /* Disable grid snaps if the grid is turned off */
             nv->snap_manager.grid.setEnabled(nv->showgrid);
-            nv->snap_manager.axonomgrid.setEnabled(nv->showgrid);
-            object->requestModified(SP_OBJECT_MODIFIED_FLAG);
-            break;
-       case SP_ATTR_GRIDTYPE:
-            nv->gridtype = sp_str_to_bool(value);
-            sp_namedview_setup_grid(nv);
             object->requestModified(SP_OBJECT_MODIFIED_FLAG);
             break;
        case SP_ATTR_SHOWGUIDES:
@@ -314,19 +303,7 @@ static void sp_namedview_set(SPObject *object, unsigned int key, const gchar *va
             object->requestModified(SP_OBJECT_MODIFIED_FLAG);
             break;
        }
-       case SP_ATTR_GRIDANGLEX:
-       case SP_ATTR_GRIDANGLEZ:
-       {
-            unsigned const d = (key == SP_ATTR_GRIDANGLEZ); // 0=X  1=Z
-            nv->gridangle[d] = 30; // 30 deg default
-            if (value) {
-                nv->gridangle[d] = g_ascii_strtod(value, NULL);
-            }
-            sp_namedview_setup_grid(nv);
-            object->requestModified(SP_OBJECT_MODIFIED_FLAG);
-            break;
-       }
-       case SP_ATTR_GRIDCOLOR:
+    case SP_ATTR_GRIDCOLOR:
             nv->gridcolor = (nv->gridcolor & 0xff) | (DEFAULTGRIDCOLOR & 0xffffff00);
             if (value) {
                 nv->gridcolor = (nv->gridcolor & 0xff) | sp_svg_read_color(value, nv->gridcolor);
@@ -468,12 +445,10 @@ static void sp_namedview_set(SPObject *object, unsigned int key, const gchar *va
             break;
        case SP_ATTR_INKSCAPE_GRID_BBOX:
             nv->snap_manager.grid.setSnapTo(Inkscape::Snapper::BBOX_POINT, value ? sp_str_to_bool(value) : TRUE);
-            nv->snap_manager.axonomgrid.setSnapTo(Inkscape::Snapper::BBOX_POINT, value ? sp_str_to_bool(value) : TRUE);
             object->requestModified(SP_OBJECT_MODIFIED_FLAG);
             break;
        case SP_ATTR_INKSCAPE_GRID_POINTS:
             nv->snap_manager.grid.setSnapTo(Inkscape::Snapper::SNAP_POINT, value ? sp_str_to_bool(value) : FALSE);
-            nv->snap_manager.axonomgrid.setSnapTo(Inkscape::Snapper::SNAP_POINT, value ? sp_str_to_bool(value) : FALSE);
             object->requestModified(SP_OBJECT_MODIFIED_FLAG);
             break;
        case SP_ATTR_INKSCAPE_GUIDE_BBOX:
@@ -577,7 +552,7 @@ g_message("named view:: child added");
             gridtype = "xygrid"; // use this as default gridtype when none is specified
             child->setAttribute("type", gridtype);
         }
-        addedgrid = Inkscape::CanvasXYGrid::NewGrid( (SPDesktop*) nv->views->data, child, gridtype);
+        addedgrid = Inkscape::CanvasGrid::NewGrid( (SPDesktop*) nv->views->data, child, gridtype);
         if (addedgrid) {
             nv->grids = g_slist_append(nv->grids, addedgrid);
             addedgrid->show();
@@ -685,11 +660,6 @@ void SPNamedView::show(SPDesktop *desktop)
     gtk_object_ref(GTK_OBJECT(item));
     gridviews = g_slist_prepend(gridviews, item);
 
-    item = sp_canvas_item_new(sp_desktop_grid(desktop), SP_TYPE_CAXONOMGRID, NULL);
-    // since we're keeping a copy, we need to bump up the ref count
-    gtk_object_ref(GTK_OBJECT(item));
-    gridviews = g_slist_prepend(gridviews, item);
-    
     // generate grids specified in SVG:
     Inkscape::XML::Node *repr = SP_OBJECT_REPR(this);
     if (repr) {
@@ -704,7 +674,7 @@ void SPNamedView::show(SPDesktop *desktop)
             }
         }
     }
-    
+
 
     sp_namedview_setup_grid(this);
 }
@@ -833,7 +803,7 @@ void SPNamedView::hide(SPDesktop const *desktop)
             gridviews = g_slist_remove(gridviews, l->data);
         }
     }
-    
+
     // delete grids:
     while ( grids ) {
         Inkscape::CanvasGrid *gr = (Inkscape::CanvasGrid *)grids->data;
@@ -903,17 +873,6 @@ void sp_namedview_toggle_grid(SPDocument *doc, Inkscape::XML::Node *repr)
     sp_document_set_undo_sensitive(doc, saved);
 }
 
-void sp_namedview_set_gridtype(unsigned int type, SPDocument *doc, Inkscape::XML::Node *repr)
-{
-    bool saved = sp_document_get_undo_sensitive(doc);
-    sp_document_set_undo_sensitive(doc, false);
-
-    sp_repr_set_int(repr, "gridtype", (gint)type);
-
-    doc->rroot->setAttribute("sodipodi:modified", "true");
-    sp_document_set_undo_sensitive(doc, saved);
-}
-
 static void sp_namedview_setup_grid(SPNamedView *nv)
 {
     for (GSList *l = nv->gridviews; l != NULL; l = l->next) {
@@ -923,34 +882,21 @@ static void sp_namedview_setup_grid(SPNamedView *nv)
 
 static void sp_namedview_setup_grid_item(SPNamedView *nv, SPCanvasItem *item)
 {
-    bool btype = SP_IS_CAXONOMGRID(GTK_OBJECT(item));
-
-    if ( nv->showgrid && (nv->gridtype == btype) ) {
+    if ( nv->showgrid ) {
         sp_canvas_item_show(item);
     } else {
         sp_canvas_item_hide(item);
     }
-    
+
     sp_canvas_item_set((GtkObject *) item,
                        "color", nv->gridcolor,
                        "originx", nv->gridorigin[NR::X],
                        "originy", nv->gridorigin[NR::Y],
+                       "spacingx", nv->gridspacing[NR::X],
                        "spacingy", nv->gridspacing[NR::Y],
                        "empcolor", nv->gridempcolor,
                        "empspacing", nv->gridempspacing,
                        NULL);
-    if (!btype){
-        // CXYGRID
-        sp_canvas_item_set((GtkObject *) item,
-                           "spacingx", nv->gridspacing[NR::X],
-                           NULL);
-    } else {
-        // CAXONOMGRID
-        sp_canvas_item_set((GtkObject *) item,
-                           "anglex", nv->gridangle[0],
-                           "anglez", nv->gridangle[1],
-                           NULL);
-    }
 }
 
 
index f3bb960e824fc741a959ec75824be86f57d5e1df..be7f1a541e91d9ca49442bd5130a3de135f8415b 100644 (file)
@@ -37,7 +37,6 @@ enum {
 struct SPNamedView : public SPObjectGroup {
     unsigned int editable : 1;
     unsigned int showgrid : 1;
-    unsigned int gridtype : 1; // 0=normal 1=axonometric
     unsigned int showguides : 1;
     unsigned int showborder : 1;
     unsigned int showpageshadow : 1;
@@ -59,7 +58,6 @@ struct SPNamedView : public SPObjectGroup {
     /* Grid data is in points regardless of unit */
     NR::Point gridorigin;
     gdouble gridspacing[2];
-    gdouble gridangle[2];   // for axonometric grid
     gint gridempspacing;
 
     SPUnit const *doc_units;
index 4f38eb3d9d472805d99912ec420c8d7fdb9b39ab..f4257206228ac39f8d2da49773b5d92a8961b4e7 100644 (file)
@@ -136,11 +136,6 @@ DocumentProperties::init()
                      G_CALLBACK(on_deactivate_desktop), 0);
 
     show_all_children();
-    if (prefs_get_int_attribute("dialogs.documentoptions", "axonomgrid_enabled", 0) != 1) {
-        _rrb_gridtype._hbox->hide();
-        _rsu_ax.getSU()->hide();
-        _rsu_az.getSU()->hide();
-    }
 
     present();
 }
@@ -250,10 +245,6 @@ DocumentProperties::build_grid()
     /// Dissenting view: you want snapping without grid.
 
     _rcbgrid.init (_("_Show grid"), _("Show or hide grid"), "showgrid", _wr);
-    _rrb_gridtype.init (_("Grid type:"), _("Normal (2D)"), _("Axonometric (3D)"),
-                _("The normal grid with vertical and horizontal lines."),
-                _("A grid with vertical lines and two diagonal line groups, each representing the projection of a primary axis."),
-                "gridtype", _wr);
 
     _rumg.init (_("Grid _units:"), "grid_units", _wr);
     _rsu_ox.init (_("_Origin X:"), _("X coordinate of grid origin"),
@@ -264,10 +255,6 @@ DocumentProperties::build_grid()
                   "gridspacingx", _rumg, _wr);
     _rsu_sy.init (_("Spacing _Y:"), _("Distance between horizontal grid lines"),
                   "gridspacingy", _rumg, _wr);
-    _rsu_ax.init (_("Angle X:"), _("Angle of x-axis of axonometric grid"),
-                  "gridanglex", _rumg, _wr);
-    _rsu_az.init (_("Angle Z:"), _("Angle of z-axis of axonometric grid"),
-                  "gridanglez", _rumg, _wr);
     _rcp_gcol.init (_("Grid line _color:"), _("Grid line color"),
                     _("Color of grid lines"), "gridcolor", "gridopacity", _wr);
     _rcp_gmcol.init (_("Ma_jor grid line color:"), _("Major grid line color"),
@@ -289,14 +276,11 @@ DocumentProperties::build_grid()
     {
         label_grid,         0,
         0,                  _rcbgrid._button,
-        0,                  _rrb_gridtype._hbox,
         _rumg._label,       _rumg._sel,
         0,                  _rsu_ox.getSU(),
         0,                  _rsu_oy.getSU(),
         0,                  _rsu_sx.getSU(),
         0,                  _rsu_sy.getSU(),
-        0,                  _rsu_ax.getSU(),
-        0,                  _rsu_az.getSU(),
         _rcp_gcol._label,   _rcp_gcol._cp,
         0,                  0,
         _rcp_gmcol._label,  _rcp_gmcol._cp,
@@ -433,7 +417,9 @@ DocumentProperties::build_gridspage()
     Gtk::Label* label_crea_type = manage (new Gtk::Label);
     label_crea_type->set_markup (_("Gridtype"));
     
-    _grids_entry_gridtype.set_text(Glib::ustring("xygrid"));
+    _grids_combo_gridtype.append_text(Glib::ustring("xygrid"));
+    _grids_combo_gridtype.append_text(Glib::ustring("axonometric"));
+    _grids_combo_gridtype.set_active_text(Glib::ustring("xygrid"));
     
     Gtk::Label* label_def = manage (new Gtk::Label);
     label_def->set_markup (_("<b>Defined grids</b>"));
@@ -446,7 +432,7 @@ DocumentProperties::build_gridspage()
     const Gtk::Widget* widget_array[] =
     {
         label_crea, 0,
-        label_crea_type, (Gtk::Widget*) &_grids_entry_gridtype,
+        label_crea_type, (Gtk::Widget*) &_grids_combo_gridtype,
         (Gtk::Widget*) &_grids_button_new,         (Gtk::Widget*) &_grids_button_remove, 
         label_def,         0,
         (Gtk::Widget*) &_grids_notebook, 0
@@ -487,7 +473,6 @@ DocumentProperties::update()
 
     //-----------------------------------------------------------grid page
     _rcbgrid.setActive (nv->showgrid);
-    _rrb_gridtype.setValue (nv->gridtype);
     _rumg.setUnit (nv->gridunit);
 
     gdouble val;
@@ -504,11 +489,6 @@ DocumentProperties::update()
     double gridy = sp_pixels_get_units (val, *(nv->gridunit));
     _rsu_sy.setValue (gridy);
 
-    val = nv->gridangle[0];
-    _rsu_ax.setValue (val);
-    val = nv->gridangle[1];
-    _rsu_az.setValue (val);
-
     _rcp_gcol.setRgba32 (nv->gridcolor);
     _rcp_gmcol.setRgba32 (nv->gridempcolor);
     _rsi.setValue (nv->gridempspacing);
@@ -646,7 +626,7 @@ DocumentProperties::onNewGrid()
 {
     Inkscape::XML::Node *repr = SP_OBJECT_REPR(sp_desktop_namedview(SP_ACTIVE_DESKTOP));
 
-    Glib::ustring typestring = _grids_entry_gridtype.get_text();
+    Glib::ustring typestring = _grids_combo_gridtype.get_active_text();
     CanvasGrid::writeNewGridToRepr(repr, typestring.c_str()); // FIXME ofcourse user should supply choice for gridtype 
 }
 
@@ -655,7 +635,12 @@ void
 DocumentProperties::onRemoveGrid()
 {
     gint pagenum = _grids_notebook.get_current_page();
+    if (pagenum == -1) // no pages
+      return;
+      
     Gtk::Widget *page = _grids_notebook.get_nth_page(pagenum);
+    if (!page) return;
+    
     Glib::ustring tabtext = _grids_notebook.get_tab_label_text(*page);
     
     // find the grid with name tabtext (it's id) and delete that one.
index b42c0860910c0989e3814dbf8411932dc8721321..b9928914c1ed0e43bb25c0d64e5556f93d12b789 100644 (file)
@@ -67,9 +67,8 @@ protected:
     PageSizer             _page_sizer;
     //---------------------------------------------------------------
     RegisteredCheckButton _rcbgrid, _rcbsnbb, _rcbsnnod;
-    RegisteredRadioButtonPair _rrb_gridtype;
     RegisteredUnitMenu    _rumg, _rums;
-    RegisteredScalarUnit  _rsu_ox, _rsu_oy, _rsu_sx, _rsu_sy, _rsu_ax, _rsu_az;
+    RegisteredScalarUnit  _rsu_ox, _rsu_oy, _rsu_sx, _rsu_sy;
     RegisteredColorPicker _rcp_gcol, _rcp_gmcol;
     RegisteredSuffixedInteger _rsi;
     //---------------------------------------------------------------
@@ -86,7 +85,7 @@ protected:
     Gtk::Notebook   _grids_notebook;
     Gtk::Button     _grids_button_new;
     Gtk::Button     _grids_button_remove;
-    Gtk::Entry      _grids_entry_gridtype;
+    Gtk::ComboBoxText _grids_combo_gridtype;
     //---------------------------------------------------------------
 
     gchar * _prefs_path;