From 67c3fc5586ae05506f75bb30fe46a071e20613d2 Mon Sep 17 00:00:00 2001
From: Tavmjong Bah <tavmjong@free.fr>
Date: Thu, 24 Apr 2014 14:04:31 +0200
Subject: Clean up of style code, removal of SPFontStyle. Step 2.

(bzr r13300)
---
 src/ui/dialog/font-substitution.cpp | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

(limited to 'src/ui')

diff --git a/src/ui/dialog/font-substitution.cpp b/src/ui/dialog/font-substitution.cpp
index bf9133086..6e9ecc3c8 100644
--- a/src/ui/dialog/font-substitution.cpp
+++ b/src/ui/dialog/font-substitution.cpp
@@ -200,16 +200,16 @@ GSList * FontSubstitution::getFontReplacedItems(SPDocument* doc, Glib::ustring *
              }
         }
 
-        if (style && style->text) {
+        if (style) {
             gchar const *style_font = NULL;
-            if (style->text->font_family.set)
-                style_font = style->text->font_family.value;
-            else if (style->text->font_specification.set)
-                style_font = style->text->font_specification.value;
-            else if (style->text->font_family.value)
-                style_font = style->text->font_family.value;
-            else if (style->text->font_specification.value)
-                style_font = style->text->font_specification.value;
+            if (style->font_family.set)
+                style_font = style->font_family.value;
+            else if (style->font_specification.set)
+                style_font = style->font_specification.value;
+            else if (style->font_family.value)
+                style_font = style->font_family.value;
+            else if (style->font_specification.value)
+                style_font = style->font_specification.value;
 
             if (style_font) {
                 if (has_visible_text(item)) {
-- 
cgit v1.2.3


From b9f02c6200865e24150ce14a1e480c4283b0472a Mon Sep 17 00:00:00 2001
From: "Liam P. White" <inkscapebrony@gmail.com>
Date: Fri, 25 Apr 2014 12:49:54 -0700
Subject: Improve keyboard navigation in LPE dialog.

Fixed bugs:
  - https://launchpad.net/bugs/1310688

(bzr r13305)
---
 src/ui/dialog/livepatheffect-add.cpp | 14 ++++++++++++++
 src/ui/dialog/livepatheffect-add.h   |  4 ++++
 2 files changed, 18 insertions(+)

(limited to 'src/ui')

diff --git a/src/ui/dialog/livepatheffect-add.cpp b/src/ui/dialog/livepatheffect-add.cpp
index b5f51d81d..c558eddaf 100644
--- a/src/ui/dialog/livepatheffect-add.cpp
+++ b/src/ui/dialog/livepatheffect-add.cpp
@@ -38,6 +38,7 @@ LivePathEffectAdd::LivePathEffectAdd() :
     scrolled_window.set_policy(Gtk::POLICY_AUTOMATIC, Gtk::POLICY_AUTOMATIC);
     scrolled_window.set_shadow_type(Gtk::SHADOW_IN);
     scrolled_window.set_size_request(250, 200);
+    scrolled_window.set_can_focus();
 
     /**
      * Effect Store and Tree
@@ -83,10 +84,12 @@ LivePathEffectAdd::LivePathEffectAdd() :
     add_action_widget(close_button, Gtk::RESPONSE_CLOSE);
     add_action_widget(add_button, Gtk::RESPONSE_APPLY);
 
+    
     /**
      * Signal handlers
      */
     effectlist_treeview.signal_button_press_event().connect_notify( sigc::mem_fun(*this, &LivePathEffectAdd::onButtonEvent) );
+    effectlist_treeview.signal_key_press_event().connect_notify(sigc::mem_fun(*this, &LivePathEffectAdd::onKeyEvent));
     close_button.signal_clicked().connect(sigc::mem_fun(*this, &LivePathEffectAdd::onClose));
     add_button.signal_clicked().connect(sigc::mem_fun(*this, &LivePathEffectAdd::onAdd));
     signal_delete_event().connect( sigc::bind_return(sigc::hide(sigc::mem_fun(*this, &LivePathEffectAdd::onClose)), true ) );
@@ -107,6 +110,16 @@ void LivePathEffectAdd::onClose()
     hide();
 }
 
+void LivePathEffectAdd::onKeyEvent(GdkEventKey* evt)
+{
+    if (evt->keyval == GDK_KEY_Return) {
+         onAdd();
+    }
+    if (evt->keyval == GDK_KEY_Escape) {
+         onClose();
+    }
+}
+
 void LivePathEffectAdd::onButtonEvent(GdkEventButton* evt)
 {
     // Double click on tree is same as clicking the add button
@@ -135,6 +148,7 @@ void LivePathEffectAdd::show(SPDesktop *desktop)
     dial.set_modal(true);
     desktop->setWindowTransient (dial.gobj());
     dial.property_destroy_with_parent() = true;
+    dial.effectlist_treeview.grab_focus();
     dial.run();
 }
 
diff --git a/src/ui/dialog/livepatheffect-add.h b/src/ui/dialog/livepatheffect-add.h
index 7fa766272..99ead878c 100644
--- a/src/ui/dialog/livepatheffect-add.h
+++ b/src/ui/dialog/livepatheffect-add.h
@@ -74,6 +74,10 @@ protected:
      */
     void onButtonEvent(GdkEventButton* evt);
 
+    /**
+     * Key event
+     */
+    void onKeyEvent(GdkEventKey* evt);
 private:
 
     Gtk::TreeView     effectlist_treeview;
-- 
cgit v1.2.3


From b9f4d7dfe7411f551192a98a3d8151e14149c5ae Mon Sep 17 00:00:00 2001
From: Tavmjong Bah <tavmjong@free.fr>
Date: Sat, 26 Apr 2014 10:17:28 +0200
Subject: Clean up of style code: Patch from suv: SPStyle: struct -> class

(bzr r13309)
---
 src/ui/widget/style-subject.h | 2 +-
 src/ui/widget/style-swatch.h  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/ui')

diff --git a/src/ui/widget/style-subject.h b/src/ui/widget/style-subject.h
index c2941d995..47da91732 100644
--- a/src/ui/widget/style-subject.h
+++ b/src/ui/widget/style-subject.h
@@ -20,7 +20,7 @@
 class SPDesktop;
 class SPObject;
 class SPCSSAttr;
-struct SPStyle;
+class SPStyle;
 
 namespace Inkscape {
 class Selection;
diff --git a/src/ui/widget/style-swatch.h b/src/ui/widget/style-swatch.h
index 23ecbdfda..557ca82e2 100644
--- a/src/ui/widget/style-swatch.h
+++ b/src/ui/widget/style-swatch.h
@@ -29,7 +29,7 @@
 #include "desktop.h"
 #include "preferences.h"
 
-struct SPStyle;
+class SPStyle;
 class SPCSSAttr;
 
 namespace Gtk {
-- 
cgit v1.2.3