]> de.git.xonotic.org Git - xonotic/netradiant.git/blobdiff - radiant/patchdialog.cpp
default texture is notex, not textures/ dir
[xonotic/netradiant.git] / radiant / patchdialog.cpp
index 6765ceeb2df68c7ec0adeab70aa428d658e08222..ba5349185878fcfe478fe8494f5a4bb4fc4352d4 100644 (file)
@@ -1,23 +1,23 @@
 /*
-Copyright (C) 1999-2006 Id Software, Inc. and contributors.
-For a list of contributors, see the accompanying CONTRIBUTORS file.
+   Copyright (C) 1999-2006 Id Software, Inc. and contributors.
+   For a list of contributors, see the accompanying CONTRIBUTORS file.
 
-This file is part of GtkRadiant.
+   This file is part of GtkRadiant.
 
-GtkRadiant is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   GtkRadiant is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-GtkRadiant is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   GtkRadiant is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with GtkRadiant; if not, write to the Free Software
-Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
-*/
+   You should have received a copy of the GNU General Public License
+   along with GtkRadiant; if not, write to the Free Software
+   Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ */
 
 //
 // Patch Dialog
@@ -27,20 +27,12 @@ Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 
 #include "patchdialog.h"
 
+#include <gtk/gtk.h>
+
 #include "itexdef.h"
 
 #include "debugging/debugging.h"
 
-#include <gtk/gtkvbox.h>
-#include <gtk/gtkhbox.h>
-#include <gtk/gtkframe.h>
-#include <gtk/gtklabel.h>
-#include <gtk/gtktable.h>
-#include <gtk/gtkcombobox.h>
-#include <gtk/gtkbutton.h>
-#include <gtk/gtkspinbutton.h>
-#include <gtk/gtkcheckbutton.h>
-
 #include "gtkutil/idledraw.h"
 #include "gtkutil/entry.h"
 #include "gtkutil/button.h"
@@ -58,291 +50,280 @@ Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 #include <gdk/gdkkeysyms.h>
 
 // the increment we are using for the patch inspector (this is saved in the prefs)
-struct pi_globals_t
-{
-  float shift[2];
-  float scale[2];
-  float rotate;
-
-  pi_globals_t()
-  {
-    shift[0] = 8.0f;
-    shift[1] = 8.0f;
-    scale[0] = 0.5f;
-    scale[1] = 0.5f;
-    rotate = 45.0f;
-  }
+struct pi_globals_t {
+    float shift[2];
+    float scale[2];
+    float rotate;
+
+    pi_globals_t()
+    {
+        shift[0] = 8.0f;
+        shift[1] = 8.0f;
+        scale[0] = 0.5f;
+        scale[1] = 0.5f;
+        rotate = 45.0f;
+    }
 };
 
 pi_globals_t g_pi_globals;
 
-class PatchFixedSubdivisions
-{
+class PatchFixedSubdivisions {
 public:
-  PatchFixedSubdivisions() : m_enabled(false), m_x(0), m_y(0)
-  {
-  }
-  PatchFixedSubdivisions(bool enabled, std::size_t x, std::size_t y) : m_enabled(enabled), m_x(x), m_y(y)
-  {
-  }
-  bool m_enabled;
-  std::size_t m_x;
-  std::size_t m_y;
+    PatchFixedSubdivisions() : m_enabled(false), m_x(0), m_y(0)
+    {
+    }
+
+    PatchFixedSubdivisions(bool enabled, std::size_t x, std::size_t y) : m_enabled(enabled), m_x(x), m_y(y)
+    {
+    }
+
+    bool m_enabled;
+    std::size_t m_x;
+    std::size_t m_y;
 };
 
-void Patch_getFixedSubdivisions(const Patch& patch, PatchFixedSubdivisions& subdivisions)
+void Patch_getFixedSubdivisions(const Patch &patch, PatchFixedSubdivisions &subdivisions)
 {
-  subdivisions.m_enabled = patch.m_patchDef3;
-  subdivisions.m_x = patch.m_subdivisions_x;
-  subdivisions.m_y = patch.m_subdivisions_y;
+    subdivisions.m_enabled = patch.m_patchDef3;
+    subdivisions.m_x = patch.m_subdivisions_x;
+    subdivisions.m_y = patch.m_subdivisions_y;
 }
 
 const std::size_t MAX_PATCH_SUBDIVISIONS = 32;
 
-void Patch_setFixedSubdivisions(Patch& patch, const PatchFixedSubdivisions& subdivisions)
+void Patch_setFixedSubdivisions(Patch &patch, const PatchFixedSubdivisions &subdivisions)
 {
-  patch.undoSave();
-
-  patch.m_patchDef3 = subdivisions.m_enabled;
-  patch.m_subdivisions_x = subdivisions.m_x;
-  patch.m_subdivisions_y = subdivisions.m_y;
-
-  if(patch.m_subdivisions_x == 0)
-  {
-    patch.m_subdivisions_x = 4;
-  }
-  else if(patch.m_subdivisions_x > MAX_PATCH_SUBDIVISIONS)
-  {
-    patch.m_subdivisions_x = MAX_PATCH_SUBDIVISIONS;
-  }
-  if(patch.m_subdivisions_y == 0)
-  {
-    patch.m_subdivisions_y = 4;
-  }
-  else if(patch.m_subdivisions_y > MAX_PATCH_SUBDIVISIONS)
-  {
-    patch.m_subdivisions_y = MAX_PATCH_SUBDIVISIONS;
-  }
-
-  SceneChangeNotify();
-  Patch_textureChanged();
-  patch.controlPointsChanged();
+    patch.undoSave();
+
+    patch.m_patchDef3 = subdivisions.m_enabled;
+    patch.m_subdivisions_x = subdivisions.m_x;
+    patch.m_subdivisions_y = subdivisions.m_y;
+
+    if (patch.m_subdivisions_x == 0) {
+        patch.m_subdivisions_x = 4;
+    } else if (patch.m_subdivisions_x > MAX_PATCH_SUBDIVISIONS) {
+        patch.m_subdivisions_x = MAX_PATCH_SUBDIVISIONS;
+    }
+    if (patch.m_subdivisions_y == 0) {
+        patch.m_subdivisions_y = 4;
+    } else if (patch.m_subdivisions_y > MAX_PATCH_SUBDIVISIONS) {
+        patch.m_subdivisions_y = MAX_PATCH_SUBDIVISIONS;
+    }
+
+    SceneChangeNotify();
+    Patch_textureChanged();
+    patch.controlPointsChanged();
 }
 
-class PatchGetFixedSubdivisions
-{
-  PatchFixedSubdivisions& m_subdivisions;
+class PatchGetFixedSubdivisions {
+    PatchFixedSubdivisions &m_subdivisions;
 public:
-  PatchGetFixedSubdivisions(PatchFixedSubdivisions& subdivisions) : m_subdivisions(subdivisions)
-  {
-  }
-  void operator()(Patch& patch)
-  {
-    Patch_getFixedSubdivisions(patch, m_subdivisions);
-    SceneChangeNotify();
-  }
+    PatchGetFixedSubdivisions(PatchFixedSubdivisions &subdivisions) : m_subdivisions(subdivisions)
+    {
+    }
+
+    void operator()(Patch &patch)
+    {
+        Patch_getFixedSubdivisions(patch, m_subdivisions);
+        SceneChangeNotify();
+    }
 };
 
-void Scene_PatchGetFixedSubdivisions(PatchFixedSubdivisionssubdivisions)
+void Scene_PatchGetFixedSubdivisions(PatchFixedSubdivisions &subdivisions)
 {
 #if 1
-  if(GlobalSelectionSystem().countSelected() != 0)
-  {
-    Patch* patch = Node_getPatch(GlobalSelectionSystem().ultimateSelected().path().top());
-    if(patch != 0)
-    {
-      Patch_getFixedSubdivisions(*patch, subdivisions);
+    if (GlobalSelectionSystem().countSelected() != 0) {
+        Patch *patch = Node_getPatch(GlobalSelectionSystem().ultimateSelected().path().top());
+        if (patch != 0) {
+            Patch_getFixedSubdivisions(*patch, subdivisions);
+        }
     }
-  }
 #else
-  Scene_forEachVisibleSelectedPatch(PatchGetFixedSubdivisions(subdivisions));
+    Scene_forEachVisibleSelectedPatch( PatchGetFixedSubdivisions( subdivisions ) );
 #endif
 }
 
-class PatchSetFixedSubdivisions
-{
-  const PatchFixedSubdivisions& m_subdivisions;
-public:
-  PatchSetFixedSubdivisions(const PatchFixedSubdivisions& subdivisions) : m_subdivisions(subdivisions)
-  {
-  }
-  void operator()(Patch& patch) const
-  {
-    Patch_setFixedSubdivisions(patch, m_subdivisions);
-  }
-};
-
-void Scene_PatchSetFixedSubdivisions(const PatchFixedSubdivisions& subdivisions)
+void Scene_PatchSetFixedSubdivisions(const PatchFixedSubdivisions &subdivisions)
 {
-  UndoableCommand command("patchSetFixedSubdivisions");
-  Scene_forEachVisibleSelectedPatch(PatchSetFixedSubdivisions(subdivisions));
+    UndoableCommand command("patchSetFixedSubdivisions");
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        Patch_setFixedSubdivisions(patch, subdivisions);
+    });
 }
 
-typedef struct _GtkCheckButton GtkCheckButton;
 
-class Subdivisions
-{
+class Subdivisions {
 public:
-  GtkCheckButton* m_enabled;
-  GtkEntry* m_horizontal;
-  GtkEntry* m_vertical;
-  Subdivisions() : m_enabled(0), m_horizontal(0), m_vertical(0)
-  {
-  }
-  void update()
-  {
-    PatchFixedSubdivisions subdivisions;
-    Scene_PatchGetFixedSubdivisions(subdivisions);
-
-    toggle_button_set_active_no_signal(GTK_TOGGLE_BUTTON(m_enabled), subdivisions.m_enabled);
-
-    if(subdivisions.m_enabled)
+    ui::CheckButton m_enabled;
+    ui::Entry m_horizontal;
+    ui::Entry m_vertical;
+
+    Subdivisions() : m_enabled(ui::null), m_horizontal(ui::null), m_vertical(ui::null)
+    {
+    }
+
+    void update()
     {
-      entry_set_int(m_horizontal, static_cast<int>(subdivisions.m_x));
-      entry_set_int(m_vertical, static_cast<int>(subdivisions.m_y));
-      gtk_widget_set_sensitive(GTK_WIDGET(m_horizontal), TRUE);
-      gtk_widget_set_sensitive(GTK_WIDGET(m_vertical), TRUE);
+        PatchFixedSubdivisions subdivisions;
+        Scene_PatchGetFixedSubdivisions(subdivisions);
+
+        toggle_button_set_active_no_signal(m_enabled, subdivisions.m_enabled);
+
+        if (subdivisions.m_enabled) {
+            entry_set_int(m_horizontal, static_cast<int>( subdivisions.m_x ));
+            entry_set_int(m_vertical, static_cast<int>( subdivisions.m_y ));
+            gtk_widget_set_sensitive(m_horizontal, TRUE);
+            gtk_widget_set_sensitive(m_vertical, TRUE);
+        } else {
+            m_horizontal.text("");
+            m_vertical.text("");
+            gtk_widget_set_sensitive(m_horizontal, FALSE);
+            gtk_widget_set_sensitive(m_vertical, FALSE);
+        }
+    }
+
+    void cancel()
+    {
+        update();
+    }
+
+    typedef MemberCaller<Subdivisions, void(), &Subdivisions::cancel> CancelCaller;
+
+    void apply()
+    {
+        Scene_PatchSetFixedSubdivisions(
+                PatchFixedSubdivisions(
+                        gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(m_enabled)) != FALSE,
+                        static_cast<std::size_t>( entry_get_int(m_horizontal)),
+                        static_cast<std::size_t>( entry_get_int(m_vertical))
+                )
+        );
     }
-    else
+
+    typedef MemberCaller<Subdivisions, void(), &Subdivisions::apply> ApplyCaller;
+
+    static void applyGtk(ui::ToggleButton toggle, Subdivisions *self)
     {
-      gtk_entry_set_text(m_horizontal, "");
-      gtk_entry_set_text(m_vertical, "");
-      gtk_widget_set_sensitive(GTK_WIDGET(m_horizontal), FALSE);
-      gtk_widget_set_sensitive(GTK_WIDGET(m_vertical), FALSE);
+        self->apply();
     }
-  }
-  void cancel()
-  {
-    update();
-  }
-  typedef MemberCaller<Subdivisions, &Subdivisions::cancel> CancelCaller;
-  void apply()
-  {
-    Scene_PatchSetFixedSubdivisions(
-      PatchFixedSubdivisions(
-        gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(m_enabled)) != FALSE,
-        static_cast<std::size_t>(entry_get_int(m_horizontal)),
-        static_cast<std::size_t>(entry_get_int(m_vertical))
-      )
-    );
-  }
-  typedef MemberCaller<Subdivisions, &Subdivisions::apply> ApplyCaller;
-  static void applyGtk(GtkToggleButton* toggle, Subdivisions* self)
-  {
-    self->apply();
-  }
 };
 
-class PatchInspector : public Dialog
-{
-  GtkWindow* BuildDialog();
-  Subdivisions m_subdivisions;
-  NonModalEntry m_horizontalSubdivisionsEntry;
-  NonModalEntry m_verticalSubdivisionsEntry;
+class PatchInspector : public Dialog {
+    ui::Window BuildDialog();
+
+    Subdivisions m_subdivisions;
+    NonModalEntry m_horizontalSubdivisionsEntry;
+    NonModalEntry m_verticalSubdivisionsEntry;
 public:
-  IdleDraw m_idleDraw;
-  WindowPositionTracker m_position_tracker;
+    IdleDraw m_idleDraw;
+    WindowPositionTracker m_position_tracker;
 
-  Patch *m_Patch;
+    Patch *m_Patch;
 
-  CopiedString m_strName;
-  float        m_fS;
-  float        m_fT;
-  float        m_fX;
-  float        m_fY;
-  float        m_fZ;
+    CopiedString m_strName;
+    float m_fS;
+    float m_fT;
+    float m_fX;
+    float m_fY;
+    float m_fZ;
 /*  float      m_fHScale;
-  float        m_fHShift;
-  float        m_fRotate;
-  float        m_fVScale;
-  float        m_fVShift; */
-  int   m_nCol;
-  int   m_nRow;
-  GtkComboBox *m_pRowCombo;
-  GtkComboBox *m_pColCombo;
-  std::size_t m_countRows;
-  std::size_t m_countCols;
-
-  // turn on/off processing of the "changed" "value_changed" messages
-  // (need to turn off when we are feeding data in)
-  // NOTE: much more simple than blocking signals
-  bool m_bListenChanged;
-
-  PatchInspector() :
-    m_horizontalSubdivisionsEntry(Subdivisions::ApplyCaller(m_subdivisions), Subdivisions::CancelCaller(m_subdivisions)),
-    m_verticalSubdivisionsEntry(Subdivisions::ApplyCaller(m_subdivisions), Subdivisions::CancelCaller(m_subdivisions)),
-    m_idleDraw(MemberCaller<PatchInspector, &PatchInspector::GetPatchInfo>(*this))
-  {
-    m_fS = 0.0f;
-    m_fT = 0.0f;
-    m_fX = 0.0f;
-    m_fY = 0.0f;
-    m_fZ = 0.0f;
-    m_nCol = 0;
-    m_nRow = 0;
-    m_countRows = 0;
-    m_countCols = 0;
-    m_Patch = 0;
-    m_bListenChanged = true;
-
-    m_position_tracker.setPosition(c_default_window_pos);
-  }
+   float       m_fHShift;
+   float       m_fRotate;
+   float       m_fVScale;
+   float       m_fVShift; */
+    int m_nCol;
+    int m_nRow;
+    ui::ComboBoxText m_pRowCombo{ui::null};
+    ui::ComboBoxText m_pColCombo{ui::null};
+    std::size_t m_countRows;
+    std::size_t m_countCols;
+
+// turn on/off processing of the "changed" "value_changed" messages
+// (need to turn off when we are feeding data in)
+// NOTE: much more simple than blocking signals
+    bool m_bListenChanged;
+
+    PatchInspector() :
+            m_horizontalSubdivisionsEntry(Subdivisions::ApplyCaller(m_subdivisions),
+                                          Subdivisions::CancelCaller(m_subdivisions)),
+            m_verticalSubdivisionsEntry(Subdivisions::ApplyCaller(m_subdivisions),
+                                        Subdivisions::CancelCaller(m_subdivisions)),
+            m_idleDraw(MemberCaller<PatchInspector, void(), &PatchInspector::GetPatchInfo>(*this))
+    {
+        m_fS = 0.0f;
+        m_fT = 0.0f;
+        m_fX = 0.0f;
+        m_fY = 0.0f;
+        m_fZ = 0.0f;
+        m_nCol = 0;
+        m_nRow = 0;
+        m_countRows = 0;
+        m_countCols = 0;
+        m_Patch = 0;
+        m_bListenChanged = true;
+
+        m_position_tracker.setPosition(c_default_window_pos);
+    }
 
-  bool visible()
-  {
-    return GTK_WIDGET_VISIBLE(GetWidget());
-  }
+    bool visible()
+    {
+        return GetWidget().visible();
+    }
 
 //  void UpdateInfo();
 //  void SetPatchInfo();
-  void GetPatchInfo();
-  void UpdateSpinners(bool bUp, int nID);
-  // read the current patch on map and initialize m_fX m_fY accordingly
-  void UpdateRowColInfo();
-  // sync the dialog our internal data structures
-  // depending on the flag it will read or write
-  // we use m_nCol m_nRow m_fX m_fY m_fZ m_fS m_fT m_strName
-  // (NOTE: this doesn't actually commit stuff to the map or read from it)
-  void importData();
-  void exportData();
+    void GetPatchInfo();
+
+    void UpdateSpinners(bool bUp, int nID);
+
+// read the current patch on map and initialize m_fX m_fY accordingly
+    void UpdateRowColInfo();
+
+// sync the dialog our internal data structures
+// depending on the flag it will read or write
+// we use m_nCol m_nRow m_fX m_fY m_fZ m_fS m_fT m_strName
+// (NOTE: this doesn't actually commit stuff to the map or read from it)
+    void importData();
+
+    void exportData();
 };
 
 PatchInspector g_PatchInspector;
 
-void PatchInspector_constructWindow(GtkWindow* main_window)
+void PatchInspector_constructWindow(ui::Window main_window)
 {
-  g_PatchInspector.m_parent = main_window;
-  g_PatchInspector.Create();
+    g_PatchInspector.m_parent = main_window;
+    g_PatchInspector.Create();
 }
+
 void PatchInspector_destroyWindow()
 {
-  g_PatchInspector.Destroy();
+    g_PatchInspector.Destroy();
 }
 
 void PatchInspector_queueDraw()
 {
-  if(g_PatchInspector.visible())
-  {
-    g_PatchInspector.m_idleDraw.queueDraw();
-  }
+    if (g_PatchInspector.visible()) {
+        g_PatchInspector.m_idleDraw.queueDraw();
+    }
 }
 
 void DoPatchInspector()
 {
-  g_PatchInspector.GetPatchInfo();
-  if (!g_PatchInspector.visible())
-    g_PatchInspector.ShowDlg();
+    g_PatchInspector.GetPatchInfo();
+    if (!g_PatchInspector.visible()) {
+        g_PatchInspector.ShowDlg();
+    }
 }
 
 void PatchInspector_toggleShown()
 {
-  if (g_PatchInspector.visible())
-  {
-    g_PatchInspector.m_Patch = 0;
-    g_PatchInspector.HideDlg();
-  }
-  else
-    DoPatchInspector();
+    if (g_PatchInspector.visible()) {
+        g_PatchInspector.m_Patch = 0;
+        g_PatchInspector.HideDlg();
+    } else {
+        DoPatchInspector();
+    }
 }
 
 
@@ -350,847 +331,744 @@ void PatchInspector_toggleShown()
 // static functions
 
 // memorize the current state (that is don't try to undo our do before changing something else)
-static void OnApply (GtkWidget *widget, gpointer data)
+static void OnApply(ui::Widget widget, gpointer data)
 {
-  g_PatchInspector.exportData();
-  if (g_PatchInspector.m_Patch != 0)
-  {
-    UndoableCommand command("patchSetTexture");
-    g_PatchInspector.m_Patch->undoSave();
-
-    if (!texdef_name_valid(g_PatchInspector.m_strName.c_str()))
-    {
-      globalErrorStream() << "invalid texture name '" << g_PatchInspector.m_strName.c_str() << "'\n";
-      g_PatchInspector.m_strName = texdef_name_default();
-    }
-    g_PatchInspector.m_Patch->SetShader(g_PatchInspector.m_strName.c_str());
-
-    std::size_t r = g_PatchInspector.m_nRow;
-    std::size_t c = g_PatchInspector.m_nCol;
-    if(r < g_PatchInspector.m_Patch->getHeight()
-      && c < g_PatchInspector.m_Patch->getWidth())
-    {            
-      PatchControl& p = g_PatchInspector.m_Patch->ctrlAt(r,c);
-      p.m_vertex[0] = g_PatchInspector.m_fX;
-      p.m_vertex[1] = g_PatchInspector.m_fY;
-      p.m_vertex[2] = g_PatchInspector.m_fZ;
-      p.m_texcoord[0] = g_PatchInspector.m_fS;
-      p.m_texcoord[1] = g_PatchInspector.m_fT;
-      g_PatchInspector.m_Patch->controlPointsChanged();
+    g_PatchInspector.exportData();
+    if (g_PatchInspector.m_Patch != 0) {
+        UndoableCommand command("patchSetTexture");
+        g_PatchInspector.m_Patch->undoSave();
+
+        if (!texdef_name_valid(g_PatchInspector.m_strName.c_str())) {
+            globalErrorStream() << "invalid texture name '" << g_PatchInspector.m_strName.c_str() << "'\n";
+            g_PatchInspector.m_strName = texdef_name_default();
+        }
+        g_PatchInspector.m_Patch->SetShader(g_PatchInspector.m_strName.c_str());
+
+        std::size_t r = g_PatchInspector.m_nRow;
+        std::size_t c = g_PatchInspector.m_nCol;
+        if (r < g_PatchInspector.m_Patch->getHeight()
+            && c < g_PatchInspector.m_Patch->getWidth()) {
+            PatchControl &p = g_PatchInspector.m_Patch->ctrlAt(r, c);
+            p.m_vertex[0] = g_PatchInspector.m_fX;
+            p.m_vertex[1] = g_PatchInspector.m_fY;
+            p.m_vertex[2] = g_PatchInspector.m_fZ;
+            p.m_texcoord[0] = g_PatchInspector.m_fS;
+            p.m_texcoord[1] = g_PatchInspector.m_fT;
+            g_PatchInspector.m_Patch->controlPointsChanged();
+        }
     }
-  }
 }
 
-static void OnSelchangeComboColRow (GtkWidget *widget, gpointer data)
+static void OnSelchangeComboColRow(ui::Widget widget, gpointer data)
 {
-  if (!g_PatchInspector.m_bListenChanged)
-    return;
-  // retrieve the current m_nRow and m_nCol, other params are not relevant
-  g_PatchInspector.exportData();
-  // read the changed values ourselves
-  g_PatchInspector.UpdateRowColInfo();
-  // now reflect our changes
-  g_PatchInspector.importData();
+    if (!g_PatchInspector.m_bListenChanged) {
+        return;
+    }
+    // retrieve the current m_nRow and m_nCol, other params are not relevant
+    g_PatchInspector.exportData();
+    // read the changed values ourselves
+    g_PatchInspector.UpdateRowColInfo();
+    // now reflect our changes
+    g_PatchInspector.importData();
 }
 
-class PatchSetTextureRepeat
+void Scene_PatchTileTexture_Selected(scene::Graph &graph, float s, float t)
 {
-  float m_s, m_t;
-public:
-  PatchSetTextureRepeat(float s, float t) : m_s(s), m_t(t)
-  {
-  }
-  void operator()(Patch& patch) const
-  {
-    patch.SetTextureRepeat(m_s, m_t);
-  }
-};
-
-void Scene_PatchTileTexture_Selected(scene::Graph& graph, float s, float t)
-{
-  Scene_forEachVisibleSelectedPatch(PatchSetTextureRepeat(s, t));
-  SceneChangeNotify();
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.SetTextureRepeat(s, t);
+    });
+    SceneChangeNotify();
 }
 
-static void OnBtnPatchdetails (GtkWidget *widget, gpointer data)
+static void OnBtnPatchdetails(ui::Widget widget, gpointer data)
 {
-  UndoableCommand command("patchCapTexture");
-
-  Scene_PatchCapTexture_Selected(GlobalSceneGraph());
+    Patch_CapTexture();
 }
 
-static void OnBtnPatchfit (GtkWidget *widget, gpointer data)
+static void OnBtnPatchfit(ui::Widget widget, gpointer data)
 {
-  UndoableCommand command("patchFitTexture");
-
-  Scene_PatchTileTexture_Selected(GlobalSceneGraph(), 1, 1);
+    Patch_FitTexture();
 }
 
-static void OnBtnPatchnatural (GtkWidget *widget, gpointer data)
+static void OnBtnPatchnatural(ui::Widget widget, gpointer data)
 {
-  UndoableCommand command("patchNaturalTexture");
-
-  Scene_PatchNaturalTexture_Selected(GlobalSceneGraph());
+    Patch_NaturalTexture();
 }
 
-static void OnBtnPatchreset (GtkWidget *widget, gpointer data)
+static void OnBtnPatchreset(ui::Widget widget, gpointer data)
 {
-  float fx, fy;
-  if (DoTextureLayout (&fx, &fy) == eIDOK)
-  {
-    UndoableCommand command("patchTileTexture");
-    Scene_PatchTileTexture_Selected(GlobalSceneGraph(), fx, fy);
-  }
+    Patch_ResetTexture();
 }
 
-struct PatchRotateTexture
+static void OnBtnPatchFlipX(ui::Widget widget, gpointer data)
 {
-  float m_angle;
-public:
-  PatchRotateTexture(float angle) : m_angle(angle)
-  {
-  }
-  void operator()(Patch& patch) const
-  {
-    patch.RotateTexture(m_angle);
-  }
-};
+    Patch_FlipTextureX();
+}
 
-void Scene_PatchRotateTexture_Selected(scene::Graph& graph, float angle)
+static void OnBtnPatchFlipY(ui::Widget widget, gpointer data)
 {
-  Scene_forEachVisibleSelectedPatch(PatchRotateTexture(angle));
+    Patch_FlipTextureY();
 }
 
-class PatchScaleTexture
+void Scene_PatchRotateTexture_Selected(scene::Graph &graph, float angle)
 {
-  float m_s, m_t;
-public:
-  PatchScaleTexture(float s, float t) : m_s(s), m_t(t)
-  {
-  }
-  void operator()(Patch& patch) const
-  {
-    patch.ScaleTexture(m_s, m_t);
-  }
-};
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.RotateTexture(angle);
+    });
+}
 
 float Patch_convertScale(float scale)
 {
-  if(scale > 0)
-  {
-    return scale;
-  }
-  if(scale < 0)
-  {
-    return -1 / scale;
-  }
-  return 1;
+    if (scale > 0) {
+        return scale;
+    }
+    if (scale < 0) {
+        return -1 / scale;
+    }
+    return 1;
 }
 
-void Scene_PatchScaleTexture_Selected(scene::Graphgraph, float s, float t)
+void Scene_PatchScaleTexture_Selected(scene::Graph &graph, float s, float t)
 {
-  Scene_forEachVisibleSelectedPatch(PatchScaleTexture(Patch_convertScale(s), Patch_convertScale(t)));
+    s = Patch_convertScale(s);
+    t = Patch_convertScale(t);
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.ScaleTexture(s, t);
+    });
 }
 
-class PatchTranslateTexture
+void Scene_PatchTranslateTexture_Selected(scene::Graph &graph, float s, float t)
 {
-  float m_s, m_t;
-public:
-  PatchTranslateTexture(float s, float t)
-    : m_s(s), m_t(t)
-  {
-  }
-  void operator()(Patch& patch) const
-  {
-    patch.TranslateTexture(m_s, m_t);
-  }
-};
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.TranslateTexture(s, t);
+    });
+}
 
-void Scene_PatchTranslateTexture_Selected(scene::Graph& graph, float s, float t)
+static void OnBtnPatchAutoCap(ui::Widget widget, gpointer data)
 {
-  Scene_forEachVisibleSelectedPatch(PatchTranslateTexture(s, t));
+    Patch_AutoCapTexture();
 }
 
-static void OnSpinChanged (GtkAdjustment *adj, gpointer data)
+static void OnSpinChanged(ui::Adjustment adj, gpointer data)
 {
-  texdef_t td;
-
-  td.rotate = 0;
-  td.scale[0] = td.scale[1] = 0;
-  td.shift[0] = td.shift[1] = 0;
-
-  if (adj->value == 0)
-    return;
-
-  if (adj == g_object_get_data (G_OBJECT (g_PatchInspector.GetWidget()), "hshift_adj"))
-  {
-    g_pi_globals.shift[0] = static_cast<float>(atof (gtk_entry_get_text (GTK_ENTRY (data))));
-
-    if (adj->value > 0)
-      td.shift[0] = g_pi_globals.shift[0];
-    else
-      td.shift[0] = -g_pi_globals.shift[0];
-  }
-  else if (adj == g_object_get_data (G_OBJECT (g_PatchInspector.GetWidget()), "vshift_adj"))
-  {
-    g_pi_globals.shift[1] = static_cast<float>(atof (gtk_entry_get_text (GTK_ENTRY (data))));
-
-    if (adj->value > 0)
-      td.shift[1] = g_pi_globals.shift[1];
-    else
-      td.shift[1] = -g_pi_globals.shift[1];
-  }
-  else if (adj == g_object_get_data (G_OBJECT (g_PatchInspector.GetWidget()), "hscale_adj"))
-  {
-    g_pi_globals.scale[0] = static_cast<float>(atof (gtk_entry_get_text (GTK_ENTRY (data))));
-         if (g_pi_globals.scale[0] == 0.0f)
-                 return;
-    if (adj->value > 0)
-      td.scale[0] = g_pi_globals.scale[0];
-    else
-      td.scale[0] = -g_pi_globals.scale[0];
-  }
-  else if (adj == g_object_get_data (G_OBJECT (g_PatchInspector.GetWidget()), "vscale_adj"))
-  {
-    g_pi_globals.scale[1] = static_cast<float>(atof (gtk_entry_get_text (GTK_ENTRY (data))));
-         if (g_pi_globals.scale[1] == 0.0f)
-                 return;
-    if (adj->value > 0)
-      td.scale[1] = g_pi_globals.scale[1];
-    else
-      td.scale[1] = -g_pi_globals.scale[1];
-  }
-  else if (adj == g_object_get_data (G_OBJECT (g_PatchInspector.GetWidget()), "rotate_adj"))
-  {
-    g_pi_globals.rotate = static_cast<float>(atof (gtk_entry_get_text (GTK_ENTRY (data))));
-
-    if (adj->value > 0)
-      td.rotate = g_pi_globals.rotate;
-    else
-      td.rotate = -g_pi_globals.rotate;
-  }
-
-  adj->value = 0;
-
-  // will scale shift rotate the patch accordingly
-
-
-  if (td.shift[0] || td.shift[1])
-  {
-    UndoableCommand command("patchTranslateTexture");
-    Scene_PatchTranslateTexture_Selected (GlobalSceneGraph(), td.shift[0], td.shift[1]);
-  }
-  else if (td.scale[0] || td.scale[1])
-  {
-    UndoableCommand command("patchScaleTexture");
-    Scene_PatchScaleTexture_Selected (GlobalSceneGraph(), td.scale[0], td.scale[1]);
-  }
-  else if (td.rotate)
-  {
-    UndoableCommand command("patchRotateTexture");
-    Scene_PatchRotateTexture_Selected (GlobalSceneGraph(), td.rotate);
-  }
-
-  // update the point-by-point view
-  OnSelchangeComboColRow(0,0);
+    texdef_t td;
+
+    td.rotate = 0;
+    td.scale[0] = td.scale[1] = 0;
+    td.shift[0] = td.shift[1] = 0;
+
+    if (gtk_adjustment_get_value(adj) == 0) {
+        return;
+    }
+
+    if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "hshift_adj")) {
+        g_pi_globals.shift[0] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.shift[0] = g_pi_globals.shift[0];
+        } else {
+            td.shift[0] = -g_pi_globals.shift[0];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "vshift_adj")) {
+        g_pi_globals.shift[1] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.shift[1] = g_pi_globals.shift[1];
+        } else {
+            td.shift[1] = -g_pi_globals.shift[1];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "hscale_adj")) {
+        g_pi_globals.scale[0] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+        if (g_pi_globals.scale[0] == 0.0f) {
+            return;
+        }
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.scale[0] = g_pi_globals.scale[0];
+        } else {
+            td.scale[0] = -g_pi_globals.scale[0];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "vscale_adj")) {
+        g_pi_globals.scale[1] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+        if (g_pi_globals.scale[1] == 0.0f) {
+            return;
+        }
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.scale[1] = g_pi_globals.scale[1];
+        } else {
+            td.scale[1] = -g_pi_globals.scale[1];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "rotate_adj")) {
+        g_pi_globals.rotate = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.rotate = g_pi_globals.rotate;
+        } else {
+            td.rotate = -g_pi_globals.rotate;
+        }
+    }
+
+    gtk_adjustment_set_value(adj, 0);
+
+    // will scale shift rotate the patch accordingly
+
+
+    if (td.shift[0] || td.shift[1]) {
+        UndoableCommand command("patchTranslateTexture");
+        Scene_PatchTranslateTexture_Selected(GlobalSceneGraph(), td.shift[0], td.shift[1]);
+    } else if (td.scale[0] || td.scale[1]) {
+        UndoableCommand command("patchScaleTexture");
+        Scene_PatchScaleTexture_Selected(GlobalSceneGraph(), td.scale[0], td.scale[1]);
+    } else if (td.rotate) {
+        UndoableCommand command("patchRotateTexture");
+        Scene_PatchRotateTexture_Selected(GlobalSceneGraph(), td.rotate);
+    }
+
+    // update the point-by-point view
+    OnSelchangeComboColRow(ui::root, 0);
 }
 
-static gint OnDialogKey (GtkWidget* widget, GdkEventKey* event, gpointer data)
+static gint OnDialogKey(ui::Widget widget, GdkEventKey *event, gpointer data)
 {
-  if (event->keyval == GDK_Return)
-  {
-    OnApply (0, 0);
-    return TRUE;
-  }
-  else if (event->keyval == GDK_Escape)
-  {
-    g_PatchInspector.GetPatchInfo();
-    return TRUE;
-  }
-  return FALSE;
+    if (event->keyval == GDK_KEY_Return) {
+        OnApply(ui::root, 0);
+        return TRUE;
+    } else if (event->keyval == GDK_KEY_Escape) {
+        g_PatchInspector.GetPatchInfo();
+        return TRUE;
+    }
+    return FALSE;
 }
 
 // =============================================================================
 // PatchInspector class
 
-GtkWindow* PatchInspector::BuildDialog()
+ui::Window PatchInspector::BuildDialog()
 {
-  GtkWindow* window = create_floating_window("Patch Properties", m_parent);
+    ui::Window window = ui::Window(create_floating_window("Patch Properties", m_parent));
+
+    m_position_tracker.connect(window);
 
-  m_position_tracker.connect(window);
+    global_accel_connect_window(window);
 
-  global_accel_connect_window(window);
+    window_connect_focus_in_clear_focus_widget(window);
 
-  window_connect_focus_in_clear_focus_widget(window);
-  
 
-  {
-    GtkVBox* vbox = GTK_VBOX(gtk_vbox_new(FALSE, 5));
-    gtk_container_set_border_width(GTK_CONTAINER(vbox), 5); 
-    gtk_widget_show(GTK_WIDGET(vbox));
-    gtk_container_add(GTK_CONTAINER(window), GTK_WIDGET(vbox));
     {
-      GtkHBox* hbox = GTK_HBOX(gtk_hbox_new(FALSE, 5));
-      gtk_widget_show(GTK_WIDGET(hbox));
-      gtk_box_pack_start(GTK_BOX(vbox), GTK_WIDGET(hbox), TRUE, TRUE, 0);
-      {
-        GtkVBox* vbox2 = GTK_VBOX(gtk_vbox_new(FALSE, 0));
-        gtk_container_set_border_width(GTK_CONTAINER(vbox2), 0);
-        gtk_widget_show(GTK_WIDGET(vbox2));
-        gtk_box_pack_start(GTK_BOX(hbox), GTK_WIDGET(vbox2), TRUE, TRUE, 0);
+        auto vbox = ui::VBox(FALSE, 5);
+        gtk_container_set_border_width(GTK_CONTAINER(vbox), 5);
+        vbox.show();
+        window.add(vbox);
         {
-          GtkFrame* frame = GTK_FRAME(gtk_frame_new("Details"));
-          gtk_widget_show(GTK_WIDGET(frame));
-          gtk_box_pack_start(GTK_BOX(vbox2), GTK_WIDGET(frame), TRUE, TRUE, 0);
-          {
-            GtkVBox* vbox3 = GTK_VBOX(gtk_vbox_new(FALSE, 5));
-            gtk_container_set_border_width(GTK_CONTAINER(vbox3), 5);
-            gtk_widget_show(GTK_WIDGET(vbox3));
-            gtk_container_add(GTK_CONTAINER(frame), GTK_WIDGET(vbox3));
-            {
-              GtkTable* table = GTK_TABLE(gtk_table_new(2, 2, FALSE));
-              gtk_widget_show(GTK_WIDGET(table));
-              gtk_box_pack_start(GTK_BOX(vbox3), GTK_WIDGET(table), TRUE, TRUE, 0);
-              gtk_table_set_row_spacings(table, 5);
-              gtk_table_set_col_spacings(table, 5);
-              {
-                GtkLabel* label = GTK_LABEL(gtk_label_new("Row:"));
-                gtk_widget_show(GTK_WIDGET(label));
-                gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 0, 1,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-              }
-              {
-                GtkLabel* label = GTK_LABEL(gtk_label_new("Column:"));
-                gtk_widget_show(GTK_WIDGET(label));
-                gtk_table_attach(table, GTK_WIDGET(label), 1, 2, 0, 1,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-              }
-              {
-                GtkComboBox* combo = GTK_COMBO_BOX(gtk_combo_box_new_text());
-                g_signal_connect(G_OBJECT(combo), "changed", G_CALLBACK(OnSelchangeComboColRow), this);
-                AddDialogData(*combo, m_nRow);
-
-                gtk_widget_show(GTK_WIDGET(combo));
-                gtk_table_attach(table, GTK_WIDGET(combo), 0, 1, 1, 2,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-                gtk_widget_set_usize(GTK_WIDGET(combo), 60, -1);
-                m_pRowCombo = combo;
-              }
-
-              {
-                GtkComboBox* combo = GTK_COMBO_BOX(gtk_combo_box_new_text());
-                g_signal_connect(G_OBJECT(combo), "changed", G_CALLBACK(OnSelchangeComboColRow), this);
-                AddDialogData(*combo, m_nCol);
-
-                gtk_widget_show(GTK_WIDGET(combo));
-                gtk_table_attach(table, GTK_WIDGET(combo), 1, 2, 1, 2,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-                gtk_widget_set_usize(GTK_WIDGET(combo), 60, -1);
-                m_pColCombo = combo;
-              }
-            }
-            GtkTable* table = GTK_TABLE(gtk_table_new(5, 2, FALSE));
-            gtk_widget_show(GTK_WIDGET(table));
-            gtk_box_pack_start(GTK_BOX(vbox3), GTK_WIDGET(table), TRUE, TRUE, 0);
-            gtk_table_set_row_spacings(table, 5);
-            gtk_table_set_col_spacings(table, 5);
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("X:"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 0, 1,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-            }
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("Y:"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 1, 2,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-            }
+            auto hbox = ui::HBox(FALSE, 5);
+            hbox.show();
+            vbox.pack_start(hbox, TRUE, TRUE, 0);
             {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("Z:"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 2, 3,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
+                auto vbox2 = ui::VBox(FALSE, 0);
+                gtk_container_set_border_width(GTK_CONTAINER(vbox2), 0);
+                vbox2.show();
+                hbox.pack_start(vbox2, TRUE, TRUE, 0);
+                {
+                    auto frame = ui::Frame("Details");
+                    frame.show();
+                    vbox2.pack_start(frame, TRUE, TRUE, 0);
+                    {
+                        auto vbox3 = ui::VBox(FALSE, 5);
+                        gtk_container_set_border_width(GTK_CONTAINER(vbox3), 5);
+                        vbox3.show();
+                        frame.add(vbox3);
+                        {
+                            auto table = ui::Table(2, 2, FALSE);
+                            table.show();
+                            vbox3.pack_start(table, TRUE, TRUE, 0);
+                            gtk_table_set_row_spacings(table, 5);
+                            gtk_table_set_col_spacings(table, 5);
+                            {
+                                auto label = ui::Label("Row:");
+                                label.show();
+                                table.attach(label, {0, 1, 0, 1},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                            }
+                            {
+                                auto label = ui::Label("Column:");
+                                label.show();
+                                table.attach(label, {1, 2, 0, 1},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                            }
+                            {
+                                auto combo = ui::ComboBoxText(ui::New);
+                                combo.connect("changed", G_CALLBACK(OnSelchangeComboColRow), this);
+                                AddDialogData(combo, m_nRow);
+
+                                combo.show();
+                                table.attach(combo, {0, 1, 1, 2},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                                combo.dimensions(60, -1);
+                                m_pRowCombo = combo;
+                            }
+
+                            {
+                                auto combo = ui::ComboBoxText(ui::New);
+                                combo.connect("changed", G_CALLBACK(OnSelchangeComboColRow), this);
+                                AddDialogData(combo, m_nCol);
+
+                                combo.show();
+                                table.attach(combo, {1, 2, 1, 2},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                                combo.dimensions(60, -1);
+                                m_pColCombo = combo;
+                            }
+                        }
+                        auto table = ui::Table(5, 2, FALSE);
+                        table.show();
+                        vbox3.pack_start(table, TRUE, TRUE, 0);
+                        gtk_table_set_row_spacings(table, 5);
+                        gtk_table_set_col_spacings(table, 5);
+                        {
+                            auto label = ui::Label("X:");
+                            label.show();
+                            table.attach(label, {0, 1, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
+                        }
+                        {
+                            auto label = ui::Label("Y:");
+                            label.show();
+                            table.attach(label, {0, 1, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
+                        }
+                        {
+                            auto label = ui::Label("Z:");
+                            label.show();
+                            table.attach(label, {0, 1, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
+                        }
+                        {
+                            auto label = ui::Label("S:");
+                            label.show();
+                            table.attach(label, {0, 1, 3, 4}, {GTK_EXPAND | GTK_FILL, 0});
+                        }
+                        {
+                            auto label = ui::Label("T:");
+                            label.show();
+                            table.attach(label, {0, 1, 4, 5}, {GTK_EXPAND | GTK_FILL, 0});
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {1, 2, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
+                            AddDialogData(entry, m_fX);
+
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {1, 2, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
+                            AddDialogData(entry, m_fY);
+
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {1, 2, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
+                            AddDialogData(entry, m_fZ);
+
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {1, 2, 3, 4}, {GTK_EXPAND | GTK_FILL, 0});
+                            AddDialogData(entry, m_fS);
+
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {1, 2, 4, 5}, {GTK_EXPAND | GTK_FILL, 0});
+                            AddDialogData(entry, m_fT);
+
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                    }
+                }
+                if (g_pGameDescription->mGameType == "doom3") {
+                    auto frame = ui::Frame("Tesselation");
+                    frame.show();
+                    vbox2.pack_start(frame, TRUE, TRUE, 0);
+                    {
+                        auto vbox3 = ui::VBox(FALSE, 5);
+                        gtk_container_set_border_width(GTK_CONTAINER(vbox3), 5);
+                        vbox3.show();
+                        frame.add(vbox3);
+                        {
+                            auto table = ui::Table(3, 2, FALSE);
+                            table.show();
+                            vbox3.pack_start(table, TRUE, TRUE, 0);
+                            gtk_table_set_row_spacings(table, 5);
+                            gtk_table_set_col_spacings(table, 5);
+                            {
+                                auto label = ui::Label("Fixed");
+                                label.show();
+                                table.attach(label, {0, 1, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
+                            }
+                            {
+                                auto check = ui::CheckButton::from(gtk_check_button_new());
+                                check.show();
+                                table.attach(check, {1, 2, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
+                                m_subdivisions.m_enabled = check;
+                                guint handler_id = check.connect("toggled", G_CALLBACK(&Subdivisions::applyGtk),
+                                                                 &m_subdivisions);
+                                g_object_set_data(G_OBJECT(check), "handler", gint_to_pointer(handler_id));
+                            }
+                            {
+                                auto label = ui::Label("Horizontal");
+                                label.show();
+                                table.attach(label, {0, 1, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
+                            }
+                            {
+                                auto entry = ui::Entry(ui::New);
+                                entry.show();
+                                table.attach(entry, {1, 2, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
+                                m_subdivisions.m_horizontal = entry;
+                                m_horizontalSubdivisionsEntry.connect(entry);
+                            }
+                            {
+                                auto label = ui::Label("Vertical");
+                                label.show();
+                                table.attach(label, {0, 1, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
+                            }
+                            {
+                                auto entry = ui::Entry(ui::New);
+                                entry.show();
+                                table.attach(entry, {1, 2, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
+                                m_subdivisions.m_vertical = entry;
+                                m_verticalSubdivisionsEntry.connect(entry);
+                            }
+                        }
+                    }
+                }
             }
             {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("S:"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 3, 4,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
+                auto frame = ui::Frame("Texturing");
+                frame.show();
+                hbox.pack_start(frame, TRUE, TRUE, 0);
+                {
+                    auto vbox2 = ui::VBox(FALSE, 5);
+                    vbox2.show();
+                    frame.add(vbox2);
+                    gtk_container_set_border_width(GTK_CONTAINER(vbox2), 5);
+                    {
+                        auto label = ui::Label("Name:");
+                        label.show();
+                        vbox2.pack_start(label, TRUE, TRUE, 0);
+                        gtk_label_set_justify(label, GTK_JUSTIFY_LEFT);
+                        gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                    }
+                    {
+                        auto entry = ui::Entry(ui::New);
+                        //  gtk_editable_set_editable (GTK_ENTRY (entry), false);
+                        entry.show();
+                        vbox2.pack_start(entry, TRUE, TRUE, 0);
+                        AddDialogData(entry, m_strName);
+
+                        entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                    }
+                    {
+                        auto table = ui::Table(5, 4, FALSE);
+                        table.show();
+                        vbox2.pack_start(table, TRUE, TRUE, 0);
+                        gtk_table_set_row_spacings(table, 5);
+                        gtk_table_set_col_spacings(table, 5);
+                        {
+                            auto label = ui::Label("Horizontal Shift Step");
+                            label.show();
+                            table.attach(label, {2, 4, 0, 1}, {GTK_FILL | GTK_EXPAND, 0});
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto label = ui::Label("Vertical Shift Step");
+                            label.show();
+                            table.attach(label, {2, 4, 1, 2}, {GTK_FILL | GTK_EXPAND, 0});
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto label = ui::Label("Horizontal Stretch Step");
+                            label.show();
+                            table.attach(label, {2, 3, 2, 3}, {GTK_FILL | GTK_EXPAND, 0});
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto button = ui::Button("Flip");
+                            button.show();
+                            table.attach(button, {3, 4, 2, 3}, {GTK_FILL, 0});
+                            button.connect("clicked", G_CALLBACK(OnBtnPatchFlipX), 0);
+                            button.dimensions(60, -1);
+                        }
+                        {
+                            auto label = ui::Label("Vertical Stretch Step");
+                            label.show();
+                            table.attach(label, {2, 3, 3, 4}, {GTK_FILL | GTK_EXPAND, 0});
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto button = ui::Button("Flip");
+                            button.show();
+                            table.attach(button, {3, 4, 3, 4}, {GTK_FILL, 0});
+                            button.connect("clicked", G_CALLBACK(OnBtnPatchFlipY), 0);
+                            button.dimensions(60, -1);
+                        }
+                        {
+                            auto label = ui::Label("Rotate Step");
+                            label.show();
+                            table.attach(label, {2, 4, 4, 5}, {GTK_FILL | GTK_EXPAND, 0});
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {0, 1, 0, 1}, {GTK_FILL, 0});
+                            entry.dimensions(50, -1);
+                            g_object_set_data(G_OBJECT(window), "hshift_entry", (void *) entry);
+                            // we fill in this data, if no patch is selected the widgets are unmodified when the inspector is raised
+                            // so we need to have at least one initialisation somewhere
+                            entry_set_float(entry, g_pi_globals.shift[0]);
+
+                            auto adj = ui::Adjustment(0, -8192, 8192, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), (gpointer) entry);
+                            g_object_set_data(G_OBJECT(window), "hshift_adj", (gpointer) adj);
+
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
+                            table.attach(spin, {1, 2, 0, 1}, {0, 0});
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {0, 1, 1, 2}, {GTK_FILL, 0});
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.shift[1]);
+
+                            auto adj = ui::Adjustment(0, -8192, 8192, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "vshift_adj", (gpointer) adj);
+
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
+                            table.attach(spin, {1, 2, 1, 2}, {0, 0});
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {0, 1, 2, 3}, {GTK_FILL, 0});
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.scale[0]);
+
+                            auto adj = ui::Adjustment(0, -1000, 1000, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "hscale_adj", (gpointer) adj);
+
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
+                            table.attach(spin, {1, 2, 2, 3}, {0, 0});
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {0, 1, 3, 4}, {GTK_FILL, 0});
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.scale[1]);
+
+                            auto adj = ui::Adjustment(0, -1000, 1000, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "vscale_adj", (gpointer) adj);
+
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
+                            table.attach(spin, {1, 2, 3, 4}, {0, 0});
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
+                            table.attach(entry, {0, 1, 4, 5}, {GTK_FILL, 0});
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.rotate);
+
+                            auto adj = ui::Adjustment(0, -1000, 1000, 1, 1,
+                                                      0); // NOTE: Arnout - this really should be 360 but can't change it anymore as it could break existing maps
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "rotate_adj", (gpointer) adj);
+
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
+                            table.attach(spin, {1, 2, 4, 5}, {0, 0});
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                    }
+                    auto hbox2 = ui::HBox(TRUE, 5);
+                    hbox2.show();
+                    vbox2.pack_start(hbox2, TRUE, FALSE, 0);
+                    {
+                        auto button = ui::Button("Auto Cap");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchAutoCap), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("CAP");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchdetails), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("Set...");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchreset), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("Natural");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchnatural), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("Fit");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchfit), 0);
+                        button.dimensions(60, -1);
+                    }
+                }
             }
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("T:"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 4, 5,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 1, 2, 0, 1,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              AddDialogData(*entry, m_fX);
-
-              g_signal_connect(G_OBJECT(entry), "key_press_event", G_CALLBACK(OnDialogKey), 0);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 1, 2, 1, 2,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              AddDialogData(*entry, m_fY);
-
-              g_signal_connect(G_OBJECT(entry), "key_press_event", G_CALLBACK(OnDialogKey), 0);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 1, 2, 2, 3,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              AddDialogData(*entry, m_fZ);
-
-              g_signal_connect(G_OBJECT(entry), "key_press_event", G_CALLBACK(OnDialogKey), 0);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 1, 2, 3, 4,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              AddDialogData(*entry, m_fS);
-
-              g_signal_connect(G_OBJECT(entry), "key_press_event", G_CALLBACK(OnDialogKey), 0);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 1, 2, 4, 5,
-                                (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              AddDialogData(*entry, m_fT);
-
-              g_signal_connect(G_OBJECT(entry), "key_press_event", G_CALLBACK(OnDialogKey), 0);
-            }
-          }
         }
-        if(g_pGameDescription->mGameType == "doom3")
-        {
-          GtkFrame* frame = GTK_FRAME(gtk_frame_new("Tesselation"));
-          gtk_widget_show(GTK_WIDGET(frame));
-          gtk_box_pack_start(GTK_BOX(vbox2), GTK_WIDGET(frame), TRUE, TRUE, 0);
-          {
-            GtkVBox* vbox3 = GTK_VBOX(gtk_vbox_new(FALSE, 5));
-            gtk_container_set_border_width(GTK_CONTAINER(vbox3), 5);
-            gtk_widget_show(GTK_WIDGET(vbox3));
-            gtk_container_add(GTK_CONTAINER(frame), GTK_WIDGET(vbox3));
-            {
-              GtkTable* table = GTK_TABLE(gtk_table_new(3, 2, FALSE));
-              gtk_widget_show(GTK_WIDGET(table));
-              gtk_box_pack_start(GTK_BOX(vbox3), GTK_WIDGET(table), TRUE, TRUE, 0);
-              gtk_table_set_row_spacings(table, 5);
-              gtk_table_set_col_spacings(table, 5);
-              {
-                GtkLabel* label = GTK_LABEL(gtk_label_new("Fixed"));
-                gtk_widget_show(GTK_WIDGET(label));
-                gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 0, 1,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-              }
-              {
-                GtkCheckButton* check = GTK_CHECK_BUTTON(gtk_check_button_new());
-                gtk_widget_show(GTK_WIDGET(check));
-                gtk_table_attach(table, GTK_WIDGET(check), 1, 2, 0, 1,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-                m_subdivisions.m_enabled = check;
-                guint handler_id = g_signal_connect(G_OBJECT(check), "toggled", G_CALLBACK(&Subdivisions::applyGtk), &m_subdivisions);
-                g_object_set_data(G_OBJECT(check), "handler", gint_to_pointer(handler_id));
-              }
-              {
-                GtkLabel* label = GTK_LABEL(gtk_label_new("Horizontal"));
-                gtk_widget_show(GTK_WIDGET(label));
-                gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 1, 2,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-              }
-              {
-                GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-                gtk_widget_show(GTK_WIDGET(entry));
-                gtk_table_attach(table, GTK_WIDGET(entry), 1, 2, 1, 2,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-                m_subdivisions.m_horizontal = entry;
-                m_horizontalSubdivisionsEntry.connect(entry);
-              }
-              {
-                GtkLabel* label = GTK_LABEL(gtk_label_new("Vertical"));
-                gtk_widget_show(GTK_WIDGET(label));
-                gtk_table_attach(table, GTK_WIDGET(label), 0, 1, 2, 3,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-              }
-              {
-                GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-                gtk_widget_show(GTK_WIDGET(entry));
-                gtk_table_attach(table, GTK_WIDGET(entry), 1, 2, 2, 3,
-                                  (GtkAttachOptions)(GTK_EXPAND | GTK_FILL),
-                                  (GtkAttachOptions)(0), 0, 0);
-                m_subdivisions.m_vertical = entry;
-                m_verticalSubdivisionsEntry.connect(entry);
-              }
-            }
-          }
-        }
-      }
-      {
-        GtkFrame* frame = GTK_FRAME(gtk_frame_new("Texturing"));
-        gtk_widget_show(GTK_WIDGET(frame));
-        gtk_box_pack_start(GTK_BOX(hbox), GTK_WIDGET(frame), TRUE, TRUE, 0);
-        {
-          GtkVBox* vbox2 = GTK_VBOX(gtk_vbox_new(FALSE, 5));
-          gtk_widget_show(GTK_WIDGET(vbox2));
-          gtk_container_add(GTK_CONTAINER(frame), GTK_WIDGET(vbox2));
-          gtk_container_set_border_width(GTK_CONTAINER(vbox2), 5);
-          {
-            GtkLabel* label = GTK_LABEL(gtk_label_new("Name:"));
-            gtk_widget_show(GTK_WIDGET(label));
-            gtk_box_pack_start(GTK_BOX(vbox2), GTK_WIDGET(label), TRUE, TRUE, 0);
-            gtk_label_set_justify(label, GTK_JUSTIFY_LEFT);
-            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
-          }
-          {
-            GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-          //  gtk_entry_set_editable (GTK_ENTRY (entry), false);
-            gtk_widget_show(GTK_WIDGET(entry));
-            gtk_box_pack_start(GTK_BOX(vbox2), GTK_WIDGET(entry), TRUE, TRUE, 0);
-            AddDialogData(*entry, m_strName);
-
-            g_signal_connect(G_OBJECT(entry), "key_press_event", G_CALLBACK(OnDialogKey), 0);
-          }
-          {
-            GtkTable* table = GTK_TABLE(gtk_table_new(5, 3, FALSE));
-            gtk_widget_show(GTK_WIDGET(table));
-            gtk_box_pack_start(GTK_BOX(vbox2), GTK_WIDGET(table), TRUE, TRUE, 0);
-            gtk_table_set_row_spacings(table, 5);
-            gtk_table_set_col_spacings(table, 5);
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("Horizontal Shift Step"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 2, 3, 0, 1,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
-            }
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("Vertical Shift Step"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 2, 3, 1, 2,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
-            }
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("Horizontal Stretch Step"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 2, 3, 2, 3,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
-            }
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("Vertical Stretch Step"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 2, 3, 3, 4,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
-            }
-            {
-              GtkLabel* label = GTK_LABEL(gtk_label_new("Rotate Step"));
-              gtk_widget_show(GTK_WIDGET(label));
-              gtk_table_attach(table, GTK_WIDGET(label), 2, 3, 4, 5,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 0, 1, 0, 1,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(entry), 50, -2);
-              g_object_set_data(G_OBJECT(window), "hshift_entry", entry);
-              // we fill in this data, if no patch is selected the widgets are unmodified when the inspector is raised
-              // so we need to have at least one initialisation somewhere
-              entry_set_float(entry, g_pi_globals.shift[0]);
-
-              GtkAdjustment* adj = GTK_ADJUSTMENT(gtk_adjustment_new(0, -8192, 8192, 1, 1, 1));
-              g_signal_connect(G_OBJECT(adj), "value_changed", G_CALLBACK(OnSpinChanged), entry);
-              g_object_set_data(G_OBJECT(window), "hshift_adj", adj);
-
-              GtkSpinButton* spin = GTK_SPIN_BUTTON(gtk_spin_button_new(adj, 1, 0));
-              gtk_widget_show(GTK_WIDGET(spin));
-              gtk_table_attach(table, GTK_WIDGET(spin), 1, 2, 0, 1,
-                                (GtkAttachOptions)(0),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(spin), 10, -2);
-              GTK_WIDGET_UNSET_FLAGS(spin, GTK_CAN_FOCUS);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 0, 1, 1, 2,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(entry), 50, -2);
-              entry_set_float(entry, g_pi_globals.shift[1]);
-
-              GtkAdjustment* adj = GTK_ADJUSTMENT(gtk_adjustment_new(0, -8192, 8192, 1, 1, 1));
-              g_signal_connect(G_OBJECT(adj), "value_changed", G_CALLBACK(OnSpinChanged), entry);
-              g_object_set_data(G_OBJECT(window), "vshift_adj", adj);
-
-              GtkSpinButton* spin = GTK_SPIN_BUTTON(gtk_spin_button_new(adj, 1, 0));
-              gtk_widget_show(GTK_WIDGET(spin));
-              gtk_table_attach(table, GTK_WIDGET(spin), 1, 2, 1, 2,
-                                (GtkAttachOptions)(0),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(spin), 10, -2);
-              GTK_WIDGET_UNSET_FLAGS(spin, GTK_CAN_FOCUS);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 0, 1, 2, 3,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(entry), 50, -2);
-              entry_set_float(entry, g_pi_globals.scale[0]);
-
-              GtkAdjustment* adj = GTK_ADJUSTMENT(gtk_adjustment_new(0, -1000, 1000, 1, 1, 1));
-              g_signal_connect(G_OBJECT(adj), "value_changed", G_CALLBACK(OnSpinChanged), entry);
-              g_object_set_data(G_OBJECT(window), "hscale_adj", adj);
-
-              GtkSpinButton* spin = GTK_SPIN_BUTTON(gtk_spin_button_new(adj, 1, 0));
-              gtk_widget_show(GTK_WIDGET(spin));
-              gtk_table_attach(table, GTK_WIDGET(spin), 1, 2, 2, 3,
-                                (GtkAttachOptions)(0),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(spin), 10, -2);
-              GTK_WIDGET_UNSET_FLAGS(spin, GTK_CAN_FOCUS);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 0, 1, 3, 4,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(entry), 50, -2);
-              entry_set_float(entry, g_pi_globals.scale[1]);
-
-              GtkAdjustment* adj = GTK_ADJUSTMENT(gtk_adjustment_new(0, -1000, 1000, 1, 1, 1));
-              g_signal_connect(G_OBJECT(adj), "value_changed", G_CALLBACK(OnSpinChanged), entry);
-              g_object_set_data(G_OBJECT(window), "vscale_adj", adj);
-
-              GtkSpinButton* spin = GTK_SPIN_BUTTON(gtk_spin_button_new(adj, 1, 0));
-              gtk_widget_show(GTK_WIDGET(spin));
-              gtk_table_attach(table, GTK_WIDGET(spin), 1, 2, 3, 4,
-                                (GtkAttachOptions)(0),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(spin), 10, -2);
-              GTK_WIDGET_UNSET_FLAGS(spin, GTK_CAN_FOCUS);
-            }
-            {
-              GtkEntry* entry = GTK_ENTRY(gtk_entry_new());
-              gtk_widget_show(GTK_WIDGET(entry));
-              gtk_table_attach(table, GTK_WIDGET(entry), 0, 1, 4, 5,
-                                (GtkAttachOptions)(GTK_FILL),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(entry), 50, -2);
-              entry_set_float(entry, g_pi_globals.rotate);
-
-              GtkAdjustment* adj = GTK_ADJUSTMENT(gtk_adjustment_new(0, -1000, 1000, 1, 1, 1));        // NOTE: Arnout - this really should be 360 but can't change it anymore as it could break existing maps
-              g_signal_connect(G_OBJECT(adj), "value_changed", G_CALLBACK(OnSpinChanged), entry);
-              g_object_set_data(G_OBJECT(window), "rotate_adj", adj);
-
-              GtkSpinButton* spin = GTK_SPIN_BUTTON(gtk_spin_button_new(adj, 1, 0));
-              gtk_widget_show(GTK_WIDGET(spin));
-              gtk_table_attach(table, GTK_WIDGET(spin), 1, 2, 4, 5,
-                                (GtkAttachOptions)(0),
-                                (GtkAttachOptions)(0), 0, 0);
-              gtk_widget_set_usize(GTK_WIDGET(spin), 10, -2);
-              GTK_WIDGET_UNSET_FLAGS(spin, GTK_CAN_FOCUS);
-            }
-          }
-          GtkHBox* hbox2 = GTK_HBOX(gtk_hbox_new(TRUE, 5));
-          gtk_widget_show(GTK_WIDGET(hbox2));
-          gtk_box_pack_start(GTK_BOX(vbox2), GTK_WIDGET(hbox2), TRUE, FALSE, 0);
-          {
-            GtkButton* button = GTK_BUTTON(gtk_button_new_with_label("CAP"));
-            gtk_widget_show(GTK_WIDGET(button));
-            gtk_box_pack_end(GTK_BOX(hbox2), GTK_WIDGET(button), TRUE, FALSE, 0);
-            g_signal_connect(G_OBJECT(button), "clicked", G_CALLBACK(OnBtnPatchdetails), 0);
-            gtk_widget_set_usize(GTK_WIDGET(button), 60, -1);
-          }
-          {
-            GtkButton* button = GTK_BUTTON(gtk_button_new_with_label("Set..."));
-            gtk_widget_show(GTK_WIDGET(button));
-            gtk_box_pack_end(GTK_BOX(hbox2), GTK_WIDGET(button), TRUE, FALSE, 0);
-            g_signal_connect(G_OBJECT(button), "clicked", G_CALLBACK(OnBtnPatchreset), 0);
-            gtk_widget_set_usize(GTK_WIDGET(button), 60, -1);
-          }
-          {
-            GtkButton* button = GTK_BUTTON(gtk_button_new_with_label("Natural"));
-            gtk_widget_show(GTK_WIDGET(button));
-            gtk_box_pack_end(GTK_BOX(hbox2), GTK_WIDGET(button), TRUE, FALSE, 0);
-            g_signal_connect(G_OBJECT(button), "clicked", G_CALLBACK(OnBtnPatchnatural), 0);
-            gtk_widget_set_usize(GTK_WIDGET(button), 60, -1);
-          }
-          {
-            GtkButton* button = GTK_BUTTON(gtk_button_new_with_label("Fit"));
-            gtk_widget_show(GTK_WIDGET(button));
-            gtk_box_pack_end(GTK_BOX(hbox2), GTK_WIDGET(button), TRUE, FALSE, 0);
-            g_signal_connect(G_OBJECT(button), "clicked", G_CALLBACK(OnBtnPatchfit), 0);
-            gtk_widget_set_usize(GTK_WIDGET(button), 60, -1);
-          }
-        }
-      }
     }
-  }
 
-  return window;
+    return window;
 }
 
 // sync the dialog our internal data structures
 void PatchInspector::exportData()
 {
-  m_bListenChanged = false;
-  Dialog::exportData();
-  m_bListenChanged = true;
+    m_bListenChanged = false;
+    Dialog::exportData();
+    m_bListenChanged = true;
 }
+
 void PatchInspector::importData()
 {
-  m_bListenChanged = false;
-  Dialog::importData();
-  m_bListenChanged = true;
+    m_bListenChanged = false;
+    Dialog::importData();
+    m_bListenChanged = true;
 }
 
 // read the map and feed in the stuff to the dialog box
 void PatchInspector::GetPatchInfo()
 {
-  if(g_pGameDescription->mGameType == "doom3")
-  {
-    m_subdivisions.update();
-  }
-
-  if(GlobalSelectionSystem().countSelected() == 0)
-  {
-    m_Patch = 0;
-  }
-  else
-  {
-    m_Patch = Node_getPatch(GlobalSelectionSystem().ultimateSelected().path().top());
-  }
-
-  if (m_Patch != 0)
-  {
-    m_strName = m_Patch->GetShader();
-  
-    // fill in the numbers for Row / Col selection
-    m_bListenChanged = false;
+    if (g_pGameDescription->mGameType == "doom3") {
+        m_subdivisions.update();
+    }
 
-    {
-      gtk_combo_box_set_active(m_pRowCombo, 0);
-
-      for (std::size_t i = 0; i < m_countRows; ++i)
-      {
-        gtk_combo_box_remove_text(m_pRowCombo, gint(m_countRows - i - 1));
-      }
-
-      m_countRows = m_Patch->getHeight();
-      for (std::size_t i = 0; i < m_countRows; ++i)
-      {
-        char buffer[16];
-        sprintf(buffer, "%u", Unsigned(i));
-        gtk_combo_box_append_text(m_pRowCombo, buffer);
-      }
-
-      gtk_combo_box_set_active(m_pRowCombo, 0);
+    if (GlobalSelectionSystem().countSelected() == 0) {
+        m_Patch = 0;
+    } else {
+        m_Patch = Node_getPatch(GlobalSelectionSystem().ultimateSelected().path().top());
     }
 
-    {
-      gtk_combo_box_set_active(m_pColCombo, 0);
-
-      for (std::size_t i = 0; i < m_countCols; ++i)
-      {
-        gtk_combo_box_remove_text(m_pColCombo, gint(m_countCols - i - 1));
-      }
-
-      m_countCols = m_Patch->getWidth();
-      for (std::size_t i = 0; i < m_countCols; ++i)
-      {
-        char buffer[16];
-        sprintf(buffer, "%u", Unsigned(i));
-        gtk_combo_box_append_text(m_pColCombo, buffer);
-      }
-
-      gtk_combo_box_set_active(m_pColCombo, 0);
+    if (m_Patch != 0) {
+        m_strName = m_Patch->GetShader();
+
+        // fill in the numbers for Row / Col selection
+        m_bListenChanged = false;
+
+        {
+            gtk_combo_box_set_active(m_pRowCombo, 0);
+
+            for (std::size_t i = 0; i < m_countRows; ++i) {
+                gtk_combo_box_text_remove(m_pRowCombo, gint(m_countRows - i - 1));
+            }
+
+            m_countRows = m_Patch->getHeight();
+            for (std::size_t i = 0; i < m_countRows; ++i) {
+                char buffer[16];
+                sprintf(buffer, "%u", Unsigned(i));
+                gtk_combo_box_text_append_text(m_pRowCombo, buffer);
+            }
+
+            gtk_combo_box_set_active(m_pRowCombo, 0);
+        }
+
+        {
+            gtk_combo_box_set_active(m_pColCombo, 0);
+
+            for (std::size_t i = 0; i < m_countCols; ++i) {
+                gtk_combo_box_text_remove(m_pColCombo, gint(m_countCols - i - 1));
+            }
+
+            m_countCols = m_Patch->getWidth();
+            for (std::size_t i = 0; i < m_countCols; ++i) {
+                char buffer[16];
+                sprintf(buffer, "%u", Unsigned(i));
+                gtk_combo_box_text_append_text(m_pColCombo, buffer);
+            }
+
+            gtk_combo_box_set_active(m_pColCombo, 0);
+        }
+
+        m_bListenChanged = true;
+
+    } else {
+        //globalOutputStream() << "WARNING: no patch\n";
     }
-    
-    m_bListenChanged = true;
-    
-  }
-  else
-  {
-    //globalOutputStream() << "WARNING: no patch\n";
-  }
-  // fill in our internal structs
-  m_nRow = 0; m_nCol = 0;
-  UpdateRowColInfo();
-  // now update the dialog box
-  importData();
+    // fill in our internal structs
+    m_nRow = 0;
+    m_nCol = 0;
+    UpdateRowColInfo();
+    // now update the dialog box
+    importData();
 }
 
 // read the current patch on map and initialize m_fX m_fY accordingly
 // NOTE: don't call UpdateData in there, it's not meant for
 void PatchInspector::UpdateRowColInfo()
 {
-  m_fX = m_fY = m_fZ = m_fS = m_fT = 0.0;
-
-  if (m_Patch != 0)
-  {
-    // we rely on whatever active row/column has been set before we get called
-    std::size_t r = m_nRow;
-    std::size_t c = m_nCol;
-    if(r < m_Patch->getHeight()
-      && c < m_Patch->getWidth())
-    {
-      const PatchControl& p = m_Patch->ctrlAt(r,c);
-      m_fX = p.m_vertex[0];
-      m_fY = p.m_vertex[1];
-      m_fZ = p.m_vertex[2];
-      m_fS = p.m_texcoord[0];
-      m_fT = p.m_texcoord[1];
+    m_fX = m_fY = m_fZ = m_fS = m_fT = 0.0;
+
+    if (m_Patch != 0) {
+        // we rely on whatever active row/column has been set before we get called
+        std::size_t r = m_nRow;
+        std::size_t c = m_nCol;
+        if (r < m_Patch->getHeight()
+            && c < m_Patch->getWidth()) {
+            const PatchControl &p = m_Patch->ctrlAt(r, c);
+            m_fX = p.m_vertex[0];
+            m_fY = p.m_vertex[1];
+            m_fZ = p.m_vertex[2];
+            m_fS = p.m_texcoord[0];
+            m_fT = p.m_texcoord[1];
+        }
     }
-  }
 }
 
 
-void PatchInspector_SelectionChanged(const Selectableselectable)
+void PatchInspector_SelectionChanged(const Selectable &selectable)
 {
-  PatchInspector_queueDraw();
+    PatchInspector_queueDraw();
 }
 
 
@@ -1199,20 +1077,23 @@ void PatchInspector_SelectionChanged(const Selectable& selectable)
 
 void PatchInspector_Construct()
 {
-  GlobalCommands_insert("PatchInspector", FreeCaller<PatchInspector_toggleShown>(), Accelerator('S', (GdkModifierType)GDK_SHIFT_MASK));
-
-  GlobalPreferenceSystem().registerPreference("PatchWnd", WindowPositionTrackerImportStringCaller(g_PatchInspector.m_position_tracker), WindowPositionTrackerExportStringCaller(g_PatchInspector.m_position_tracker));
-  GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Scale1", FloatImportStringCaller(g_pi_globals.scale[0]), FloatExportStringCaller(g_pi_globals.scale[0]));      
-  GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Scale2", FloatImportStringCaller(g_pi_globals.scale[1]), FloatExportStringCaller(g_pi_globals.scale[1]));
-  GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Shift1", FloatImportStringCaller(g_pi_globals.shift[0]), FloatExportStringCaller(g_pi_globals.shift[0]));
-  GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Shift2", FloatImportStringCaller(g_pi_globals.shift[1]), FloatExportStringCaller(g_pi_globals.shift[1]));
-  GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Rotate", FloatImportStringCaller(g_pi_globals.rotate), FloatExportStringCaller(g_pi_globals.rotate));
-
-  typedef FreeCaller1<const Selectable&, PatchInspector_SelectionChanged> PatchInspectorSelectionChangedCaller;
-  GlobalSelectionSystem().addSelectionChangeCallback(PatchInspectorSelectionChangedCaller());
-  typedef FreeCaller<PatchInspector_queueDraw> PatchInspectorQueueDrawCaller;
-  Patch_addTextureChangedCallback(PatchInspectorQueueDrawCaller());
+    GlobalCommands_insert("PatchInspector", makeCallbackF(PatchInspector_toggleShown),
+                          Accelerator('S', (GdkModifierType) GDK_SHIFT_MASK));
+
+    GlobalPreferenceSystem().registerPreference("PatchWnd", make_property<WindowPositionTracker_String>(
+            g_PatchInspector.m_position_tracker));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Scale1", make_property_string(g_pi_globals.scale[0]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Scale2", make_property_string(g_pi_globals.scale[1]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Shift1", make_property_string(g_pi_globals.shift[0]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Shift2", make_property_string(g_pi_globals.shift[1]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Rotate", make_property_string(g_pi_globals.rotate));
+
+    typedef FreeCaller<void(const Selectable &), PatchInspector_SelectionChanged> PatchInspectorSelectionChangedCaller;
+    GlobalSelectionSystem().addSelectionChangeCallback(PatchInspectorSelectionChangedCaller());
+    typedef FreeCaller<void(), PatchInspector_queueDraw> PatchInspectorQueueDrawCaller;
+    Patch_addTextureChangedCallback(PatchInspectorQueueDrawCaller());
 }
+
 void PatchInspector_Destroy()
 {
 }