Merge gitlab.com:inkscape/inkscape

9 files changed, 467 insertions, 26 deletions

diff --git a/src/ui/CMakeLists.txt b/src/ui/CMakeLists.txt
index 789378e0c..efdb279b4 100644
--- a/src/ui/CMakeLists.txt
+++ b/src/ui/CMakeLists.txt
@@ -147,6 +147,7 @@ set(ui_SRC
 	widget/licensor.cpp
 	widget/notebook-page.cpp
 	widget/object-composite-settings.cpp
+	widget/ink-select-one-action.cpp
 	widget/page-sizer.cpp
 	widget/panel.cpp
 	widget/point.cpp
@@ -339,6 +340,7 @@ set(ui_SRC
 	widget/licensor.h
 	widget/notebook-page.h
 	widget/object-composite-settings.h
+	widget/ink-select-one-action.h
 	widget/page-sizer.h
 	widget/panel.h
 	widget/point.h
diff --git a/src/ui/dialog/document-properties.cpp b/src/ui/dialog/document-properties.cpp
index 7d51653db..2765e63f4 100644
--- a/src/ui/dialog/document-properties.cpp
+++ b/src/ui/dialog/document-properties.cpp
@@ -207,7 +207,7 @@ DocumentProperties::~DocumentProperties()
  * 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.
  */
-inline void attach_all(Gtk::Grid &table, Gtk::Widget *const arr[], unsigned const n, int start = 0, int docum_prop_flag = 0)
+inline void attach_all(Gtk::Grid &table, Gtk::Widget *const arr[], unsigned const n, int start = 0, int docum_prop_flag = 0, bool indent = false)
 {
     for (unsigned i = 0, r = start; i < n; i += 2) {
         if (arr[i] && arr[i+1]) {
@@ -226,7 +226,7 @@ inline void attach_all(Gtk::Grid &table, Gtk::Widget *const arr[], unsigned cons
                 }
                 if (docum_prop_flag) {
                     // this sets the padding for subordinate widgets on the "Page" page
-                    if( i==(n-8) || i==(n-10) ) {
+                    if( indent && (i==(n-6) || i==(n-4) || i==(n-2)) ) {
                         arr[i+1]->set_hexpand();
 #if WITH_GTKMM_3_12
                         arr[i+1]->set_margin_start(20);
@@ -305,29 +305,47 @@ void DocumentProperties::build_page()
 
     Gtk::Widget *const widget_array[] =
     {
-        label_gen,         0,
-        0,                 &_rum_deflt,
-        //label_col,         0,
-        //_rcp_bg._label,    &_rcp_bg,
-        0,                 0,
-        label_for,         0,
-        0,                 &_page_sizer,
-        0,                 0,
-        label_bkg,         0,
-        0,                 &_rcb_checkerboard,
-        _rcp_bg._label,    &_rcp_bg,
-        label_bdr,         0,
-        0,                 &_rcb_canb,
-        0,                 &_rcb_bord,
-        0,                 &_rcb_shad,
-        _rcp_bord._label,  &_rcp_bord,
-        label_dsp,         0,
-        0,                 &_rcb_antialias,
+        label_gen,            0,
+        0,                    &_rum_deflt,
+        //label_col,          0,
+        //_rcp_bg._label,     &_rcp_bg,
+        0,                    0,
+        label_for,            0,
+        0,                    &_page_sizer,
+        0,                    0,
+        &_rcb_doc_props_left, &_rcb_doc_props_right,
     };
 
+    attach_all(_page_page->table(), widget_array, G_N_ELEMENTS(widget_array),0,1);
+    _rcp_bg_col.pack_start(*_rcp_bg._label, false, false, 2);
+    _rcp_bg_col.pack_start(_rcp_bg, false, false, 2);
+    Gtk::Widget *const widget_array_left[] =
+    {
+        label_bkg,        0,
+        0,                &_rcb_checkerboard,
+        0,                &_rcp_bg_col,
+        label_dsp,        0,
+        0,                &_rcb_antialias,
+    };
+
+    attach_all(_rcb_doc_props_left, widget_array_left, G_N_ELEMENTS(widget_array_left),0,1);
+    _rcp_bord_col.pack_start(*_rcp_bord._label, false, false, 2);
+    _rcp_bord_col.pack_start(_rcp_bord, false, false, 2);
+    Gtk::Widget *const widget_array_right[] =
+    {
+        label_bdr,        0,
+        0,                &_rcb_canb,
+        0,                &_rcb_bord,
+        0,                &_rcb_shad,
+        0,                &_rcp_bord_col,
+    };
+    
+    attach_all(_rcb_doc_props_right, widget_array_right, G_N_ELEMENTS(widget_array_right),0,1, true);
+
     std::list<Gtk::Widget*> _slaveList;
     _slaveList.push_back(&_rcb_bord);
     _slaveList.push_back(&_rcb_shad);
+    _slaveList.push_back(&_rcp_bord_col);
     _rcb_canb.setSlaveWidgets(_slaveList);
 
     attach_all(_page_page->table(), widget_array, G_N_ELEMENTS(widget_array),0,1);
diff --git a/src/ui/dialog/document-properties.h b/src/ui/dialog/document-properties.h
index 8d1c6b38a..fe0e1864e 100644
--- a/src/ui/dialog/document-properties.h
+++ b/src/ui/dialog/document-properties.h
@@ -117,6 +117,10 @@ protected:
 
     UI::Widget::Registry _wr;
     //---------------------------------------------------------------
+    Gtk::Grid            _rcb_doc_props_left;
+    Gtk::Grid            _rcb_doc_props_right;
+    Gtk::Box             _rcp_bg_col;
+    Gtk::Box             _rcp_bord_col;
     UI::Widget::RegisteredCheckButton _rcb_antialias;
     UI::Widget::RegisteredCheckButton _rcb_checkerboard;
     UI::Widget::RegisteredCheckButton _rcb_canb;
diff --git a/src/ui/interface.cpp b/src/ui/interface.cpp
index a0ee5c72b..bbbd71ab8 100644
--- a/src/ui/interface.cpp
+++ b/src/ui/interface.cpp
@@ -481,7 +481,12 @@ static GtkWidget *sp_ui_menu_append_item_from_verb(GtkMenu                  *men
         GtkWidget *label = gtk_accel_label_new(action->name);
         gtk_label_set_markup_with_mnemonic( GTK_LABEL(label), action->name);
         gtk_label_set_use_underline(GTK_LABEL(label), true);
+
+#if GTK_CHECK_VERSION(3,16,0)
         gtk_label_set_xalign(GTK_LABEL(label), 0.0);
+#else
+        gtk_misc_set_alignment(GTK_MISC(label), 0.0, 0.5);
+#endif
 
         GtkAccelGroup *accel_group = sp_shortcut_get_accel_group();
         gtk_menu_set_accel_group(menu, accel_group);
diff --git a/src/ui/object-edit.cpp b/src/ui/object-edit.cpp
index 81f0b92c0..a2b6a2de0 100644
--- a/src/ui/object-edit.cpp
+++ b/src/ui/object-edit.cpp
@@ -1293,6 +1293,12 @@ public:
     virtual void knot_set(Geom::Point const &p, Geom::Point const &origin, unsigned int state);
 };
 
+class SpiralKnotHolderEntityCenter : public KnotHolderEntity {
+public:
+    virtual Geom::Point knot_get() const;
+    virtual void knot_set(Geom::Point const &p, Geom::Point const &origin, unsigned int state);
+};
+
 
 /*
  * set attributes via inner (t=t0) knot point:
@@ -1425,6 +1431,20 @@ SpiralKnotHolderEntityOuter::knot_set(Geom::Point const &p, Geom::Point const &/
     spiral->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG);
 }
 
+void
+SpiralKnotHolderEntityCenter::knot_set(Geom::Point const &p, Geom::Point const &/*origin*/, unsigned int state)
+{
+    SPSpiral *spiral = dynamic_cast<SPSpiral *>(item);
+    g_assert(spiral != NULL);
+
+    Geom::Point const s = snap_knot_position(p, state);
+
+    spiral->cx = s[Geom::X];
+    spiral->cy = s[Geom::Y];
+
+    spiral->requestDisplayUpdate(SP_OBJECT_MODIFIED_FLAG);
+}
+
 Geom::Point
 SpiralKnotHolderEntityInner::knot_get() const
 {
@@ -1443,6 +1463,15 @@ SpiralKnotHolderEntityOuter::knot_get() const
     return spiral->getXY(1.0);
 }
 
+Geom::Point
+SpiralKnotHolderEntityCenter::knot_get() const
+{
+    SPSpiral const *spiral = dynamic_cast<SPSpiral const *>(item);
+    g_assert(spiral != NULL);
+
+    return Geom::Point(spiral->cx, spiral->cy);
+}
+
 void
 SpiralKnotHolderEntityInner::knot_click(unsigned int state)
 {
@@ -1461,9 +1490,26 @@ SpiralKnotHolderEntityInner::knot_click(unsigned int state)
 SpiralKnotHolder::SpiralKnotHolder(SPDesktop *desktop, SPItem *item, SPKnotHolderReleasedFunc relhandler) :
     KnotHolder(desktop, item, relhandler)
 {
+    SpiralKnotHolderEntityCenter *entity_center = new SpiralKnotHolderEntityCenter();
     SpiralKnotHolderEntityInner *entity_inner = new SpiralKnotHolderEntityInner();
     SpiralKnotHolderEntityOuter *entity_outer = new SpiralKnotHolderEntityOuter();
 
+    // NOTE: entity_central and entity_inner can overlap.
+    //
+    // In that case it would be a problem if the center control point was ON
+    // TOP because it would steal the mouse focus and the user would loose the
+    // ability to access the inner control point using only the mouse.
+    //
+    // However if the inner control point is ON TOP, taking focus, the
+    // situation is a lot better: the user can still move the inner control
+    // point with the mouse to regain access to the center control point.
+    //
+    // So, create entity_inner AFTER entity_center; this ensures that
+    // entity_inner gets rendered ON TOP.
+    entity_center->create(desktop, item, this, Inkscape::CTRL_TYPE_POINT,
+                          _("Drag to move the spiral"),
+                          SP_KNOT_SHAPE_CROSS);
+
     entity_inner->create(desktop, item, this, Inkscape::CTRL_TYPE_SHAPER,
                          _("Roll/unroll the spiral from <b>inside</b>; with <b>Ctrl</b> to snap angle; "
                            "with <b>Alt</b> to converge/diverge"));
@@ -1472,6 +1518,7 @@ SpiralKnotHolder::SpiralKnotHolder(SPDesktop *desktop, SPItem *item, SPKnotHolde
                          _("Roll/unroll the spiral from <b>outside</b>; with <b>Ctrl</b> to snap angle; "
                            "with <b>Shift</b> to scale/rotate; with <b>Alt</b> to lock radius"));
 
+    entity.push_back(entity_center);
     entity.push_back(entity_inner);
     entity.push_back(entity_outer);
 
diff --git a/src/ui/widget/ink-select-one-action.cpp b/src/ui/widget/ink-select-one-action.cpp
new file mode 100644
index 000000000..40bafecc2
--- /dev/null
+++ b/src/ui/widget/ink-select-one-action.cpp
@@ -0,0 +1,239 @@
+/*
+ * Authors:
+ *   Tavmjong Bah <tavmjong@free.fr>
+ *
+ * Copyright  (C) 2017 Tavmjong Bah
+ *
+ * Released under GNU GPL, read the file 'COPYING' for more information
+ */
+
+
+/** \file
+    An action (menu/toolbar item) that allows selecting one choice out of many.
+
+    The choices may be displayed as:
+
+    1. A group of items in a toolbar with labels and/or icons.
+    2. As a drop-down menu with a labels and/or icons.
+*/
+
+#include "ink-select-one-action.h"
+
+#include <iostream>
+#include <gtkmm/toolitem.h>
+#include <gtkmm/menuitem.h>
+#include <gtkmm/radioaction.h>
+#include <gtkmm/radiomenuitem.h>
+#include <gtkmm/combobox.h>
+#include <gtkmm/menu.h>
+#include <gtkmm/box.h>
+#include <gtkmm/label.h>
+#include <gtkmm/image.h>
+
+InkSelectOneAction* InkSelectOneAction::create(const Glib::ustring &name,
+                                               const Glib::ustring &label,
+                                               const Glib::ustring &tooltip,
+                                               const Glib::ustring &stock_id,
+                                               Glib::RefPtr<Gtk::ListStore> store ) {
+
+    return new InkSelectOneAction(name, label, tooltip, stock_id, store);
+}
+
+InkSelectOneAction::InkSelectOneAction (const Glib::ustring &name,
+                                        const Glib::ustring &label,
+                                        const Glib::ustring &tooltip,
+                                        const Glib::ustring &stock_id,
+                                        Glib::RefPtr<Gtk::ListStore> store ) :
+    Gtk::Action(name, stock_id, label, tooltip),
+    _name( name ),
+    _label( label ),
+    _tooltip( tooltip ),
+    _stock_id( stock_id ),
+    _store (store),
+    _use_radio (true),
+    _use_label (true),
+    _use_icon  (true),
+    _icon_size ( Gtk::ICON_SIZE_LARGE_TOOLBAR ),
+    _combobox (nullptr),
+    _radioaction (nullptr),
+    _menuitem (nullptr)
+{
+}
+
+void InkSelectOneAction::set_active (gint active) {
+
+    if (_active != active) {
+
+        _active = active;
+
+        if (_combobox) {
+            _combobox->set_active (active);
+        }
+
+        if (_radioaction) {
+            _radioaction->set_current_value (active);
+        }
+
+        if (active < _radiomenuitems.size()) {
+            _radiomenuitems[ active ]->set_active();
+        }
+    }
+}
+
+Gtk::Widget* InkSelectOneAction::create_menu_item_vfunc() {
+
+    if (_menuitem == nullptr) {
+
+        _menuitem = Gtk::manage (new Gtk::MenuItem);
+        Gtk::Menu *menu = Gtk::manage (new Gtk::Menu);
+
+        Gtk::RadioButton::Group group;
+        int index = 0;
+        auto children = _store->children();
+        for (auto iter = children.begin(); iter != children.end(); ++iter) {
+            Gtk::TreeModel::Row row = *iter;
+
+            InkSelectOneActionColumns columns;
+            Glib::ustring label     = row[columns.col_label     ];
+            Glib::ustring icon      = row[columns.col_icon      ];
+            Glib::ustring tooltip   = row[columns.col_tooltip   ];
+            bool          sensitive = row[columns.col_sensitive ];
+
+            Gtk::RadioMenuItem* button = Gtk::manage(new Gtk::RadioMenuItem(group));
+            button->set_label (label);
+            button->set_tooltip_text( tooltip );
+            button->set_sensitive( sensitive );
+
+            button->signal_toggled().connect( sigc::bind<0>(
+              sigc::mem_fun(*this, &InkSelectOneAction::on_toggled_radiomenu), index++)
+                );
+
+            menu->add (*button);
+
+            _radiomenuitems.push_back( button );
+        }
+
+        if ( _active < _radiomenuitems.size()) {
+            _radiomenuitems[ _active ]->set_active();
+        }
+   
+        _menuitem->set_submenu (*menu);
+        _menuitem->show_all();
+    }
+
+    return _menuitem;
+}
+
+
+Gtk::Widget* InkSelectOneAction::create_tool_item_vfunc() {
+    // Either a group of radio actions or a combobox with labels and/or icons.
+
+    Gtk::ToolItem *tool_item = new Gtk::ToolItem;
+
+    if (_use_radio) {
+        // Create radio actions (note: these are not radio buttons).
+
+        Gtk::Box* box = Gtk::manage(new Gtk::Box());
+        tool_item->add (*box);  
+
+        Gtk::RadioAction::Group group;
+        int index = 0;
+        auto children = _store->children();
+        for (auto iter = children.begin(); iter != children.end(); ++iter) {
+            Gtk::TreeModel::Row row = *iter;
+
+            InkSelectOneActionColumns columns;
+            Glib::ustring label     = row[columns.col_label    ];
+            Glib::ustring icon      = row[columns.col_icon     ];
+            Glib::ustring tooltip   = row[columns.col_tooltip  ];
+            bool          sensitive = row[columns.col_sensitive];
+            Glib::RefPtr<Gtk::RadioAction> action;
+            if (_use_icon) {
+                action =
+                    Gtk::RadioAction::create_with_icon_name (group, "Anonymous", icon, label, tooltip);
+            } else {
+                action =
+                    Gtk::RadioAction::create (group, "Anonymous", label, tooltip);
+            }
+            action->set_property( "value", index++ ); // To identify uniquely each radioaction.
+            action->set_sensitive( sensitive );
+
+            // Save first action for use in setting/getting active value.
+            if (!_radioaction) {
+                _radioaction = action;
+            }
+
+            Gtk::ToolItem* item = action->create_tool_item(); 
+            box->add (*item);
+        }
+
+        _radioaction->signal_changed().connect( sigc::mem_fun(*this, &InkSelectOneAction::on_changed_radioaction));
+
+    } else {
+        // Create combobox
+
+        _combobox = Gtk::manage (new Gtk::ComboBox());
+        _combobox->set_model(_store);
+
+        InkSelectOneActionColumns columns;
+        if (_use_icon ) {
+            Gtk::CellRendererPixbuf *renderer = new Gtk::CellRendererPixbuf;
+            renderer->set_property ("stock_size", Gtk::ICON_SIZE_LARGE_TOOLBAR);
+            _combobox->pack_start (*renderer, false);
+            _combobox->add_attribute (*renderer, "icon_name", columns.col_icon   );
+        }
+  
+        if (_use_label) {
+            _combobox->pack_start(columns.col_label);
+        }
+
+        std::vector<Gtk::CellRenderer*> cells = _combobox->get_cells();
+        for (auto iter = cells.begin(); iter!= cells.end(); ++iter) {
+            _combobox->add_attribute (**iter, "sensitive", columns.col_sensitive);
+        }
+
+        _combobox->set_active (_active);
+
+        _combobox->signal_changed().connect(
+            sigc::mem_fun(*this, &InkSelectOneAction::on_changed_combobox));
+
+        tool_item->add (*_combobox);
+    }
+
+    tool_item->show_all();
+
+    return tool_item;
+}
+
+void InkSelectOneAction::on_changed_combobox() {
+
+    set_active( _combobox->get_active_row_number() );
+     _changed.emit (_active);
+}
+
+void InkSelectOneAction::on_changed_radioaction(const Glib::RefPtr<Gtk::RadioAction>& current) {
+
+    set_active( current->get_current_value() );
+    _changed.emit (_active);
+}
+
+void InkSelectOneAction::on_toggled_radiomenu(int n) {
+
+    // toggled emitted twice, first for button toggled off, second for button toggled on.
+    // We want to react only to the button turned on.
+    if ( n < _radiomenuitems.size() &&_radiomenuitems[ n ]->get_active()) {
+        set_active ( n );
+        _changed.emit (_active);
+    }
+}
+
+/*
+  Local Variables:
+  mode:c++
+  c-file-style:"stroustrup"
+  c-file-offsets:((innamespace . 0)(inline-open . 0)(case-label . +))
+  indent-tabs-mode:nil
+  fill-column:99
+  End:
+*/
+// vim: filetype=cpp:expandtab:shiftwidth=4:tabstop=8:softtabstop=4:fileencoding=utf-8:textwidth=99 :
diff --git a/src/ui/widget/ink-select-one-action.h b/src/ui/widget/ink-select-one-action.h
new file mode 100644
index 000000000..194ca8027
--- /dev/null
+++ b/src/ui/widget/ink-select-one-action.h
@@ -0,0 +1,131 @@
+#ifndef SEEN_INK_SELECT_ONE_ACTION
+#define SEEN_INK_SELECT_ONE_ACTION
+
+/*
+ * Authors:
+ *   Tavmjong Bah <tavmjong@free.fr>
+ *
+ * Copyright  (C) 2017 Tavmjong Bah
+ *
+ * Released under GNU GPL, read the file 'COPYING' for more information
+ */
+
+/**
+   An action (menu/toolbar item) that allows selecting one choice out of many.
+
+   The choices may be displayed as:
+
+   1. A group of items in a toolbar with labels and/or icons.
+   2. As a drop-down menu with a labels and/or icons.
+*/
+
+#include <gtkmm/action.h>
+#include <gtkmm/liststore.h>
+#include <sigc++/sigc++.h>
+#include <vector>
+
+namespace Gtk {
+class ComboBox;
+class RadioAction;
+class MenuItem;
+class RadioMenuItem;
+}
+
+class InkSelectOneActionColumns : public Gtk::TreeModel::ColumnRecord {
+
+public:
+    InkSelectOneActionColumns() {
+        add (col_label);
+        add (col_icon);
+        add (col_tooltip);
+        add (col_sensitive);
+    }
+    Gtk::TreeModelColumn<Glib::ustring> col_label;
+    Gtk::TreeModelColumn<Glib::ustring> col_icon;
+    Gtk::TreeModelColumn<Glib::ustring> col_tooltip;
+    Gtk::TreeModelColumn<bool>          col_sensitive;
+};
+
+
+class InkSelectOneAction : public Gtk::Action {
+
+public:
+
+    static InkSelectOneAction* create(const Glib::ustring &name,
+                                      const Glib::ustring &label,
+                                      const Glib::ustring &tooltip,
+                                      const Glib::ustring &stock_id,
+                                      Glib::RefPtr<Gtk::ListStore> store );
+
+    /* Style of action */
+    void use_radio( bool use_radio ) { _use_radio = use_radio; }
+    void use_label( bool use_label ) { _use_label = use_label; }
+    void use_icon(  bool use_icon  ) { _use_icon  = use_icon;  }
+  
+    gint get_active() { return _active; }
+    void set_active( gint active );
+    void set_icon_size( Gtk::BuiltinIconSize size ) { _icon_size = size; }
+
+    Glib::RefPtr<Gtk::ListStore> get_store() { return _store; }
+
+    sigc::signal<void, int> signal_changed() { return _changed; }
+
+protected:
+
+    virtual Gtk::Widget* create_menu_item_vfunc();
+    virtual Gtk::Widget* create_tool_item_vfunc();
+
+    /* Signals */
+    sigc::signal<void, int> _changed;
+
+private:
+
+    Glib::ustring _name;
+    Glib::ustring _label;
+    Glib::ustring _tooltip;
+    Glib::ustring _stock_id;
+    Glib::RefPtr<Gtk::ListStore> _store;
+
+    gint _active;  /* Active menu item/button */
+
+    /* Style */
+    bool _use_radio;  // Applies to tool item only
+    bool _use_label;
+    bool _use_icon;   // Applies to menu item only
+    Gtk::BuiltinIconSize _icon_size;
+
+    /* Combobox in tool */
+    Gtk::ComboBox* _combobox;
+  
+    /* Need to track one action to get active action. */
+    Glib::RefPtr<Gtk::RadioAction> _radioaction;
+
+    Gtk::MenuItem* _menuitem;
+    std::vector<Gtk::RadioMenuItem*> _radiomenuitems;
+
+    /* Internal Callbacks */
+    void on_changed_combobox();
+    void on_changed_radioaction(const Glib::RefPtr<Gtk::RadioAction>& current);
+    void on_toggled_radiomenu(int n);
+
+    InkSelectOneAction (const Glib::ustring &name,
+                        const Glib::ustring &label,
+                        const Glib::ustring &tooltip,
+                        const Glib::ustring &stock_id,
+                        Glib::RefPtr<Gtk::ListStore> store );
+
+};
+
+
+#endif /* SEEN_INK_SELECT_ONE_ACTION */
+
+/*
+  Local Variables:
+  mode:c++
+  c-file-style:"stroustrup"
+  c-file-offsets:((innamespace . 0)(inline-open . 0)(case-label . +))
+  indent-tabs-mode:nil
+  fill-column:99
+  End:
+*/
+// vim: filetype=cpp:expandtab:shiftwidth=4:tabstop=8:softtabstop=4:fileencoding=utf-8:textwidth=99 :
diff --git a/src/ui/widget/page-sizer.cpp b/src/ui/widget/page-sizer.cpp
index bb88536d5..3ec6a2b9f 100644
--- a/src/ui/widget/page-sizer.cpp
+++ b/src/ui/widget/page-sizer.cpp
@@ -225,7 +225,7 @@ PageSizer::PageSizer(Registry & _wr)
       _marginBottom( _("Botto_m:"), _("Bottom margin"), "fit-margin-bottom", _wr),
       _lockMarginUpdate(false),
       _scaleX(_("Scale _x:"), _("Scale X"), "scale-x", _wr),
-      _scaleY(_("Scale _y:"), _("Scale Y"), "scale-y", _wr),
+      _scaleY(_("Scale _y:"), _("While SVG allows non-uniform scaling it is recommended to use only uniform scaling in Inkscape. To set a non-uniform scaling, set the 'viewBox' directly."), "scale-y", _wr),
       _lockScaleUpdate(false),
       _viewboxX(_("X:"),      _("X"),      "viewbox-x", _wr),
       _viewboxY(_("Y:"),      _("Y"),      "viewbox-y", _wr),
@@ -416,13 +416,9 @@ PageSizer::PageSizer(Registry & _wr)
     _scaleTable.attach(_scaleY,        1, 0, 1, 1);
 
     _scaleTable.attach(_scaleLabel,    2, 0, 1, 1);
-    _scaleTable.attach(_scaleWarning,  0, 1, 2, 1);
     _viewboxExpander.set_hexpand();
     _viewboxExpander.set_vexpand();
     _scaleTable.attach(_viewboxExpander, 0, 2, 2, 1);
-    
-    _scaleWarning.set_label(_("While SVG allows non-uniform scaling it is recommended to use only uniform scaling in Inkscape. To set a non-uniform scaling, set the 'viewBox' directly."));
-    _scaleWarning.set_line_wrap( true );
 
     _viewboxExpander.set_use_underline();
     _viewboxExpander.set_label(_("_Viewbox..."));
diff --git a/src/ui/widget/page-sizer.h b/src/ui/widget/page-sizer.h
index d0655fb0e..f84f96782 100644
--- a/src/ui/widget/page-sizer.h
+++ b/src/ui/widget/page-sizer.h
@@ -238,7 +238,6 @@ protected:
     Gtk::Grid            _scaleTable;
 
     Gtk::Label           _scaleLabel;
-    Gtk::Label           _scaleWarning;
     RegisteredScalar     _scaleX;
     RegisteredScalar     _scaleY;
     bool                 _lockScaleUpdate;