Switched circle toobar to stock GTK+ toolbars

Working on bug #146955

diff --git a/share/icons/icons.svg b/share/icons/icons.svg
index b4cd0bdbf150f3cc1aee4008155e5f3c1558a7c2..3007737153ce7985fe03bae613c6c54dc8447df1 100644 (file)
--- a/share/icons/icons.svg
+++ b/share/icons/icons.svg
@@ -353,8 +353,8 @@
     <linearGradient
        id="linearGradient5899">
       <stop
-         style="stop-color:#ffffff;stop-opacity:1.0000000;"
-         offset="0.0000000"
+         style="stop-color:#ffffff;stop-opacity:1;"
+         offset="0"
          id="stop5901" />
       <stop
          style="stop-color:#ffacb5;stop-opacity:1;"
@@ -4145,6 +4145,36 @@
        y2="308.5"
        gradientUnits="userSpaceOnUse"
        gradientTransform="matrix(1.0621969,0,0,0.9973142,-1261.9238,1.5933312)" />
+    <linearGradient
+       inkscape:collect="always"
+       xlink:href="#linearGradient5704"
+       id="linearGradient4328"
+       gradientUnits="userSpaceOnUse"
+       x1="113.7436"
+       y1="149.37389"
+       x2="123.97673"
+       y2="167.55257"
+       gradientTransform="translate(-60.34214,0)" />
+    <linearGradient
+       inkscape:collect="always"
+       xlink:href="#linearGradient5899"
+       id="linearGradient4330"
+       gradientUnits="userSpaceOnUse"
+       x1="113.17024"
+       y1="154.33151"
+       x2="129.23041"
+       y2="172.62581"
+       gradientTransform="translate(-67.877238,0)" />
+    <linearGradient
+       inkscape:collect="always"
+       xlink:href="#linearGradient5899"
+       id="linearGradient4332"
+       gradientUnits="userSpaceOnUse"
+       x1="115.92701"
+       y1="156.459"
+       x2="137.9319"
+       y2="180.11198"
+       gradientTransform="translate(-67.877238,0)" />
   </defs>
   <sodipodi:namedview
      inkscape:guide-bbox="true"
@@ -4160,7 +4190,7 @@
      inkscape:window-width="1021"
      inkscape:cy="860.52734"
      inkscape:cx="563.9888"
-     inkscape:zoom="0.72148239"
+     inkscape:zoom="0.71599583"
      gridtolerance="6"
      snaptogrid="false"
      showgrid="true"
@@ -14712,4 +14742,106 @@ http://www.inkscape.org/</dc:description>
          style="color:#000000;fill:none;fill-opacity:0.70196078;fill-rule:evenodd;stroke:#000000;stroke-width:2;stroke-linecap:butt;stroke-linejoin:miter;marker:none;marker-start:none;marker-mid:none;marker-end:none;stroke-miterlimit:1.5;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1;display:inline" />
     </g>
   </g>
+  <g
+     transform="translate(340,-5)"
+     id="g4303">
+    <rect
+       style="opacity:1;color:#000000;fill:none;fill-opacity:1;fill-rule:evenodd;stroke:none;stroke-width:1.00000012;stroke-linecap:round;stroke-linejoin:round;marker:none;marker-start:none;marker-mid:none;marker-end:none;stroke-miterlimit:0;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1;display:inline"
+       id="rect4336"
+       width="24"
+       height="24"
+       x="110"
+       y="150" />
+    <path
+       sodipodi:type="arc"
+       style="opacity:1;color:#000000;fill:none;fill-opacity:1;fill-rule:evenodd;stroke:url(#linearGradient6574);stroke-width:1.25467694;stroke-linecap:round;stroke-linejoin:round;marker:none;marker-start:none;marker-mid:none;marker-end:none;stroke-miterlimit:0;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1;display:inline"
+       id="path4338"
+       sodipodi:cx="122"
+       sodipodi:cy="162"
+       sodipodi:rx="12"
+       sodipodi:ry="12"
+       d="M 134,162 A 12,12 0 1 1 110,162 A 12,12 0 1 1 134,162 z"
+       transform="matrix(0.795464,0,0,0.798577,24.99896,32.67435)" />
+  </g>
+  <g
+     id="reset_circle"
+     style="fill:none">
+    <rect
+       y="150"
+       x="500"
+       height="16"
+       width="16"
+       id="rect4298"
+       style="opacity:1;fill:none;fill-opacity:1;fill-rule:nonzero;stroke:none;stroke-width:1;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:1.50000000000000000;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1" />
+    <path
+       sodipodi:type="arc"
+       style="opacity:1;fill:#b0c5da;fill-opacity:1;fill-rule:nonzero;stroke:none;stroke-width:1;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:1.50000000000000000;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4888"
+       sodipodi:cx="508"
+       sodipodi:cy="158"
+       sodipodi:rx="6"
+       sodipodi:ry="6"
+       d="M 514,158 A 6,6 0 1 1 502,158 A 6,6 0 1 1 514,158 z" />
+    <path
+       id="path5285"
+       style="color:#000000;fill:none;fill-opacity:1;fill-rule:evenodd;stroke:#c80000;stroke-width:1;stroke-linecap:butt;stroke-linejoin:round;marker:none;marker-start:none;marker-mid:none;marker-end:none;stroke-miterlimit:0;stroke-dasharray:1, 1;stroke-dashoffset:0;stroke-opacity:1;display:inline"
+       d="M 514,158 L 508,158 L 511,153"
+       sodipodi:nodetypes="ccc" />
+    <path
+       d="M 514,158 A 6,6 0 1 1 502,158 A 6,6 0 1 1 514,158 z"
+       sodipodi:ry="6"
+       sodipodi:rx="6"
+       sodipodi:cy="158"
+       sodipodi:cx="508"
+       id="path5292"
+       style="opacity:1;fill:none;fill-opacity:1;fill-rule:nonzero;stroke:#0000ff;stroke-width:1;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:1.50000000000000000;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       sodipodi:type="arc" />
+  </g>
+  <g
+     style="fill:none"
+     id="circle_closed_arc"
+     transform="translate(20,0)">
+    <rect
+       style="opacity:1;fill:none;fill-opacity:1;fill-rule:nonzero;stroke:none;stroke-width:1;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:1.5;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="rect4316"
+       width="16"
+       height="16"
+       x="500"
+       y="150" />
+    <path
+       d="M 514,158 A 6,6 0 1 1 505,152.80385 L 508,158 z"
+       sodipodi:ry="6"
+       sodipodi:rx="6"
+       sodipodi:cy="158"
+       sodipodi:cx="508"
+       id="path4318"
+       style="opacity:1;fill:#b0c5da;fill-opacity:1;fill-rule:nonzero;stroke:#0000ff;stroke-width:1;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:1.5;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       sodipodi:type="arc"
+       sodipodi:start="0"
+       sodipodi:end="4.1887903" />
+  </g>
+  <g
+     transform="translate(40,0)"
+     id="circle_open_arc"
+     style="fill:none">
+    <rect
+       y="150"
+       x="500"
+       height="16"
+       width="16"
+       id="rect4326"
+       style="opacity:1;fill:none;fill-opacity:1;fill-rule:nonzero;stroke:none;stroke-width:1;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:1.5;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1" />
+    <path
+       sodipodi:end="4.1887903"
+       sodipodi:start="0"
+       sodipodi:type="arc"
+       style="opacity:1;fill:#b0c5da;fill-opacity:1;fill-rule:nonzero;stroke:#0000ff;stroke-width:1;stroke-linecap:butt;stroke-linejoin:miter;stroke-miterlimit:1.5;stroke-dasharray:none;stroke-dashoffset:0;stroke-opacity:1"
+       id="path4328"
+       sodipodi:cx="508"
+       sodipodi:cy="158"
+       sodipodi:rx="6"
+       sodipodi:ry="6"
+       d="M 514,158 A 6,6 0 1 1 505,152.80385"
+       sodipodi:open="true" />
+  </g>
 </svg>

diff --git a/src/widgets/toolbox.cpp b/src/widgets/toolbox.cpp
index 5521ab544f9ebbee6fdf23b5c4a4221d9de83964..6cac1fe0fff6446001fde5be2a1db4e31530af6e 100644 (file)
--- a/src/widgets/toolbox.cpp
+++ b/src/widgets/toolbox.cpp
@@ -2604,17 +2604,17 @@ sp_calligraphy_toolbox_new(SPDesktop *desktop)
 static void
 sp_arctb_sensitivize (GtkWidget *tbl, double v1, double v2)
 {
-    GtkWidget *ocb = (GtkWidget*) g_object_get_data(G_OBJECT(tbl), "open_checkbox");
-    GtkWidget *make_whole = (GtkWidget*) g_object_get_data(G_OBJECT(tbl), "make_whole");
+    GtkAction *ocb = GTK_ACTION( g_object_get_data(G_OBJECT(tbl), "open_action") );
+    GtkAction *make_whole = GTK_ACTION( g_object_get_data(G_OBJECT(tbl), "make_whole") );
 
     if (v1 == 0 && v2 == 0) {
         if (gtk_object_get_data(GTK_OBJECT(tbl), "single")) { // only for a single selected ellipse (for now)
-            gtk_widget_set_sensitive(GTK_WIDGET(ocb), FALSE);
-            gtk_widget_set_sensitive(GTK_WIDGET(make_whole), FALSE);
+            gtk_action_set_sensitive( ocb, FALSE );
+            gtk_action_set_sensitive( make_whole, FALSE );
         }
     } else {
-        gtk_widget_set_sensitive(GTK_WIDGET(ocb), TRUE);
-        gtk_widget_set_sensitive(GTK_WIDGET(make_whole), TRUE);
+        gtk_action_set_sensitive( ocb, TRUE );
+        gtk_action_set_sensitive( make_whole, TRUE );
     }
 }
 
@@ -2693,12 +2693,11 @@ sp_arctb_end_value_changed(GtkAdjustment *adj, GtkWidget *tbl)
 }
 
 static void
-sp_arctb_open_state_changed(GtkWidget *widget, GtkObject *tbl)
+sp_arctb_open_state_changed( EgeSelectOneAction *act, GtkObject *tbl )
 {
     SPDesktop *desktop = (SPDesktop *) gtk_object_get_data(GTK_OBJECT(tbl), "desktop");
-
     if (sp_document_get_undo_sensitive(sp_desktop_document(desktop))) {
-        if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widget))) {
+        if ( ege_select_one_action_get_active( act ) != 0 ) {
             prefs_set_string_attribute("tools.shapes.arc", "open", "true");
         } else {
             prefs_set_string_attribute("tools.shapes.arc", "open", NULL);
@@ -2715,7 +2714,7 @@ sp_arctb_open_state_changed(GtkWidget *widget, GtkObject *tbl)
 
     bool modmade = false;
 
-    if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(widget))) {
+    if ( ege_select_one_action_get_active(act) != 0 ) {
         for (GSList const *items = sp_desktop_selection(desktop)->itemList();
              items != NULL;
              items = items->next)
@@ -2794,12 +2793,12 @@ static void arc_tb_event_attr_changed(Inkscape::XML::Node *repr, gchar const *na
 
     char const *openstr = NULL;
     openstr = repr->attribute("sodipodi:open");
-    GtkWidget *ocb = (GtkWidget*) g_object_get_data(G_OBJECT(tbl), "open_checkbox");
+    EgeSelectOneAction *ocb = EGE_SELECT_ONE_ACTION( g_object_get_data(G_OBJECT(tbl), "open_action") );
 
     if (openstr) {
-        gtk_toggle_button_set_active( GTK_TOGGLE_BUTTON(ocb),  TRUE);
+        ege_select_one_action_set_active( ocb, 1 );
     } else {
-        gtk_toggle_button_set_active( GTK_TOGGLE_BUTTON(ocb),  FALSE);
+        ege_select_one_action_set_active( ocb, 0 );
     }
 
     g_object_set_data(G_OBJECT(tbl), "freeze", GINT_TO_POINTER(FALSE));
@@ -2831,14 +2830,14 @@ sp_arc_toolbox_selection_changed(Inkscape::Selection *selection, GtkObject *tbl)
         }
     }
 
-    GtkWidget *l = GTK_WIDGET(gtk_object_get_data(GTK_OBJECT(tbl), "mode_label"));
+    EgeOutputAction* act = EGE_OUTPUT_ACTION( gtk_object_get_data(GTK_OBJECT(tbl), "mode_action") );
 
     g_object_set_data(G_OBJECT(tbl), "single", GINT_TO_POINTER(FALSE));
     if (n_selected == 0) {
-        gtk_label_set_markup(GTK_LABEL(l), _("<b>New:</b>"));
+        g_object_set( G_OBJECT(act), "label", _("<b>New:</b>"), NULL );
     } else if (n_selected == 1) {
         g_object_set_data(G_OBJECT(tbl), "single", GINT_TO_POINTER(TRUE));
-        gtk_label_set_markup(GTK_LABEL(l), _("<b>Change:</b>"));
+        g_object_set( G_OBJECT(act), "label", _("<b>Change:</b>"), NULL );
 
         oldrepr = (Inkscape::XML::Node *) gtk_object_get_data(GTK_OBJECT(tbl), "repr");
 
@@ -2858,7 +2857,7 @@ sp_arc_toolbox_selection_changed(Inkscape::Selection *selection, GtkObject *tbl)
     } else {
         // FIXME: implement averaging of all parameters for multiple selected
         //gtk_label_set_markup(GTK_LABEL(l), _("<b>Average:</b>"));
-        gtk_label_set_markup(GTK_LABEL(l), _("<b>Change:</b>"));
+        g_object_set( G_OBJECT(act), "label", _("<b>Change:</b>"), NULL );
         sp_arctb_sensitivize (GTK_WIDGET(tbl), 1, 0);
     }
 }
@@ -2867,91 +2866,146 @@ sp_arc_toolbox_selection_changed(Inkscape::Selection *selection, GtkObject *tbl)
 static GtkWidget *
 sp_arc_toolbox_new(SPDesktop *desktop)
 {
-    GtkWidget *tbl = gtk_hbox_new(FALSE, 0);
+    GtkWidget *holder = gtk_hbox_new(FALSE, 0);
+    gtk_object_set_data(GTK_OBJECT(holder), "dtw", desktop->canvas);
+    gtk_object_set_data(GTK_OBJECT(holder), "desktop", desktop);
 
-    gtk_object_set_data(GTK_OBJECT(tbl), "dtw", desktop->canvas);
-    gtk_object_set_data(GTK_OBJECT(tbl), "desktop", desktop);
+    gchar const * descr =
+        "<ui>"
+        "  <toolbar name='ArcToolbar'>"
+        "    <toolitem action='ArcStateAction' />"
+        "    <separator />"
+        "    <toolitem action='ArcStartAction' />"
+        "    <toolitem action='ArcEndAction' />"
+        "    <separator />"
+        "    <toolitem action='ArcOpenAction' />"
+        "    <separator />"
+        "    <toolitem action='ArcResetAction' />"
+        "    <separator />"
+        "  </toolbar>"
+        "</ui>";
+    GtkActionGroup* mainActions = gtk_action_group_new("main");
 
-    GtkTooltips *tt = gtk_tooltips_new();
+    EgeAdjustmentAction* eact = 0;
 
-    sp_toolbox_add_label(tbl, _("<b>New:</b>"));
+
+    {
+        EgeOutputAction* act = ege_output_action_new( "ArcStateAction", _("<b>New:</b>"), "", 0 );
+        ege_output_action_set_use_markup( act, TRUE );
+        gtk_action_group_add_action( mainActions, GTK_ACTION( act ) );
+        gtk_object_set_data( GTK_OBJECT(holder), "mode_action", act );
+    }
 
     /* Start */
     {
-        GtkWidget *hb = sp_tb_spinbutton(_("Start:"), _("The angle (in degrees) from the horizontal to the arc's start point"),
+        eact = create_adjustment_action( "ArcStartAction",
+                                         _("Start:"), _("The angle (in degrees) from the horizontal to the arc's start point"),
                                          "tools.shapes.arc", "start", 0.0,
-                                         NULL, tbl, TRUE, "altx-arc",
+                                         GTK_WIDGET(desktop->canvas), NULL/*us*/, holder, TRUE, "altx-arc",
                                          -360.0, 360.0, 1.0, 10.0,
+                                         0, 0, 0,
                                          sp_arctb_start_value_changed);
-        gtk_box_pack_start(GTK_BOX(tbl), hb, FALSE, FALSE, AUX_BETWEEN_BUTTON_GROUPS);
+        gtk_action_group_add_action( mainActions, GTK_ACTION(eact) );
     }
 
     /* End */
     {
-        GtkWidget *hb = sp_tb_spinbutton(_("End:"), _("The angle (in degrees) from the horizontal to the arc's end point"),
+        eact = create_adjustment_action( "ArcEndAction",
+                                         _("End:"), _("The angle (in degrees) from the horizontal to the arc's end point"),
                                          "tools.shapes.arc", "end", 0.0,
-                                         NULL, tbl, FALSE, NULL,
+                                         GTK_WIDGET(desktop->canvas), NULL/*us*/, holder, FALSE, NULL,
                                          -360.0, 360.0, 1.0, 10.0,
+                                         0, 0, 0,
                                          sp_arctb_end_value_changed);
-        gtk_box_pack_start(GTK_BOX(tbl), hb, FALSE, FALSE, AUX_BETWEEN_BUTTON_GROUPS);
+        gtk_action_group_add_action( mainActions, GTK_ACTION(eact) );
     }
 
     /* Segments / Pie checkbox */
     {
-        GtkWidget *hb = gtk_hbox_new(FALSE, 1);
-        GtkWidget *fscb = gtk_check_button_new_with_label(_("Open arc"));
-        gtk_tooltips_set_tip(tt, fscb, _("Switch between arc (unclosed shape) and segment (closed shape with two radii)"), NULL);
-
-        gchar const *openstr = NULL;
-        openstr = prefs_get_string_attribute("tools.shapes.arc", "open");
-        if (!openstr || (openstr && !strcmp(openstr, "false")))
-            gtk_toggle_button_set_active( GTK_TOGGLE_BUTTON(fscb),  FALSE);
-        else
-            gtk_toggle_button_set_active( GTK_TOGGLE_BUTTON(fscb),  TRUE);
-
-        gtk_widget_show(fscb);
-        gtk_object_set_data(GTK_OBJECT(tbl), "open_checkbox", fscb);
-        gtk_container_add(GTK_CONTAINER(hb), fscb);
-        g_signal_connect(G_OBJECT(fscb), "toggled", GTK_SIGNAL_FUNC(sp_arctb_open_state_changed ), tbl);
-        gtk_box_pack_start(GTK_BOX(tbl),hb, FALSE, FALSE, AUX_BETWEEN_BUTTON_GROUPS);
+        GtkListStore* model = gtk_list_store_new( 3, G_TYPE_STRING, G_TYPE_STRING, G_TYPE_STRING );
+
+        GtkTreeIter iter;
+        gtk_list_store_append( model, &iter );
+        gtk_list_store_set( model, &iter,
+                            0, _("Closed arc"),
+                            1, _("Switch to segment (closed shape with two radii)"),
+                            2, "circle_closed_arc",
+                            -1 );
+
+        gtk_list_store_append( model, &iter );
+        gtk_list_store_set( model, &iter,
+                            0, _("Open Arc"),
+                            1, _("Switch to arc (unclosed shape)"),
+                            2, "circle_open_arc",
+                            -1 );
+
+        EgeSelectOneAction* act = ege_select_one_action_new( "ArcOpenAction", _(""), _(""), NULL, GTK_TREE_MODEL(model) );
+        gtk_action_group_add_action( mainActions, GTK_ACTION(act) );
+        gtk_object_set_data( GTK_OBJECT(holder), "open_action", act );
+
+        ege_select_one_action_set_appearance( act, "full" );
+        ege_select_one_action_set_radio_action_type( act, INK_RADIO_ACTION_TYPE );
+        g_object_set( G_OBJECT(act), "icon-property", "iconId", NULL );
+        ege_select_one_action_set_icon_column( act, 2 );
+        ege_select_one_action_set_tooltip_column( act, 1  );
+
+        gchar const *openstr = prefs_get_string_attribute("tools.shapes.arc", "open");
+        bool isClosed = (!openstr || (openstr && !strcmp(openstr, "false")));
+        ege_select_one_action_set_active( act, isClosed ? 0 : 1 );
+        g_signal_connect_after( G_OBJECT(act), "changed", G_CALLBACK(sp_arctb_open_state_changed), holder);
     }
 
     /* Make Whole */
     {
-        GtkWidget *hb = gtk_hbox_new(FALSE, 1);
-        GtkWidget *b = gtk_button_new_with_label(_("Make whole"));
-        gtk_object_set_data(GTK_OBJECT(tbl), "make_whole", b);
-        gtk_tooltips_set_tip(tt, b, _("Make the shape a whole ellipse, not arc or segment"), NULL);
-        gtk_widget_show(b);
-        gtk_container_add(GTK_CONTAINER(hb), b);
-        gtk_signal_connect(GTK_OBJECT(b), "clicked", GTK_SIGNAL_FUNC(sp_arctb_defaults), tbl);
-        gtk_box_pack_start(GTK_BOX(tbl),hb, FALSE, FALSE, AUX_BETWEEN_BUTTON_GROUPS);
+        InkAction* inky = ink_action_new( "ArcResetAction",
+                                          _("Make whole"),
+                                          _("Make the shape a whole ellipse, not arc or segment"),
+                                          "reset_circle",
+                                          Inkscape::ICON_SIZE_SMALL_TOOLBAR );
+        g_signal_connect_after( G_OBJECT(inky), "activate", G_CALLBACK(sp_arctb_defaults), holder );
+        gtk_action_group_add_action( mainActions, GTK_ACTION(inky) );
+        gtk_action_set_sensitive( GTK_ACTION(inky), TRUE );
+        gtk_object_set_data( GTK_OBJECT(holder), "make_whole", inky );
     }
 
-    g_object_set_data(G_OBJECT(tbl), "single", GINT_TO_POINTER(TRUE));
+    g_object_set_data( G_OBJECT(holder), "single", GINT_TO_POINTER(TRUE) );
     // sensitivize make whole and open checkbox
     {
-        GtkAdjustment *adj1 = GTK_ADJUSTMENT(gtk_object_get_data(GTK_OBJECT(tbl), "start"));
-        GtkAdjustment *adj2 = GTK_ADJUSTMENT(gtk_object_get_data(GTK_OBJECT(tbl), "end"));
-        sp_arctb_sensitivize (tbl, adj1->value, adj2->value);
+        GtkAdjustment *adj1 = GTK_ADJUSTMENT(gtk_object_get_data(GTK_OBJECT(holder), "start"));
+        GtkAdjustment *adj2 = GTK_ADJUSTMENT(gtk_object_get_data(GTK_OBJECT(holder), "end"));
+        sp_arctb_sensitivize( holder, adj1->value, adj2->value );
     }
 
+
+    GtkUIManager* mgr = gtk_ui_manager_new();
+    GError* errVal = 0;
+
+    gtk_ui_manager_insert_action_group( mgr, mainActions, 0 );
+    gtk_ui_manager_add_ui_from_string( mgr, descr, -1, &errVal );
+
+    GtkWidget* toolBar = gtk_ui_manager_get_widget( mgr, "/ui/ArcToolbar" );
+    gtk_toolbar_set_style( GTK_TOOLBAR(toolBar), GTK_TOOLBAR_ICONS );
+    gtk_toolbar_set_icon_size( GTK_TOOLBAR(toolBar), GTK_ICON_SIZE_SMALL_TOOLBAR );
+
+
+    gtk_box_pack_start( GTK_BOX(holder), toolBar, TRUE, TRUE, 0 );
+
     sigc::connection *connection = new sigc::connection(
-        sp_desktop_selection(desktop)->connectChanged(sigc::bind(sigc::ptr_fun(sp_arc_toolbox_selection_changed), (GtkObject *)tbl))
+        sp_desktop_selection(desktop)->connectChanged(sigc::bind(sigc::ptr_fun(sp_arc_toolbox_selection_changed), (GtkObject *)holder))
         );
-    g_signal_connect(G_OBJECT(tbl), "destroy", G_CALLBACK(delete_connection), connection);
+    g_signal_connect(G_OBJECT(holder), "destroy", G_CALLBACK(delete_connection), connection);
 
     Inkscape::UI::Widget::StyleSwatch *swatch = new Inkscape::UI::Widget::StyleSwatch(NULL, _("Style of new ellipses"));
     swatch->setDesktop (desktop);
     swatch->setClickVerb (SP_VERB_CONTEXT_ARC_PREFS);
     swatch->setWatchedTool ("tools.shapes.arc", true);
     GtkWidget *swatch_ = GTK_WIDGET(swatch->gobj());
-    gtk_box_pack_end(GTK_BOX(tbl), swatch_, FALSE, FALSE, 0);
+    gtk_box_pack_end(GTK_BOX(holder), swatch_, FALSE, FALSE, 0);
 
-    gtk_widget_show_all(tbl);
-    sp_set_font_size_smaller (tbl);
+    gtk_widget_show_all( holder );
+    sp_set_font_size_smaller( holder );
 
-    return tbl;
+    return holder;
 }