summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMartin Owens <doctormo@gmail.com>2018-09-27 16:12:37 +0000
committerMartin Owens <doctormo@gmail.com>2018-09-27 16:12:37 +0000
commit705b3c16ee384d7bd04cbafa933a94545318d0fc (patch)
treef63d4e864727fca99a918aa083af0c20fe07086b /src
parentMerge branch 'master' of gitlab.com:inkscape/inkscape (diff)
downloadinkscape-705b3c16ee384d7bd04cbafa933a94545318d0fc.tar.gz
inkscape-705b3c16ee384d7bd04cbafa933a94545318d0fc.zip
Comment out a crash creating signal and remove some comments
Diffstat (limited to 'src')
-rw-r--r--src/ui/dialog/attrdialog.cpp10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/ui/dialog/attrdialog.cpp b/src/ui/dialog/attrdialog.cpp
index bb4f241b8..65bdf9942 100644
--- a/src/ui/dialog/attrdialog.cpp
+++ b/src/ui/dialog/attrdialog.cpp
@@ -82,7 +82,6 @@ AttrDialog::AttrDialog():
// isn't in this exact way, the onAttrDelete is called when the header lines are pressed.
button->signal_button_release_event().connect(sigc::mem_fun(*this, &AttrDialog::onAttrCreate), false);
}
- //_treeView.signal_button_release_event().connect(sigc::mem_fun(*this, &AttrDialog::onAttrDelete));
addRenderer->signal_activated().connect(sigc::mem_fun(*this, &AttrDialog::onAttrDelete));
_treeView.signal_key_press_event().connect(sigc::mem_fun(*this, &AttrDialog::onKeyPressed));
_treeView.set_search_column(-1);
@@ -215,7 +214,6 @@ bool AttrDialog::onAttrCreate(GdkEventButton *event)
* @return true
* Delete the attribute from the xml
*/
-//bool AttrDialog::onAttrDelete(GdkEventButton *event)
void AttrDialog::onAttrDelete(Glib::ustring path)
{
Gtk::TreeModel::Row row = *_store->get_iter(path);
@@ -284,9 +282,11 @@ void AttrDialog::nameEdited (const Glib::ustring& path, const Glib::ustring& nam
} else {
// Move to editing value, we set the name as a temporary store value
row[_attrColumns._attributeName] = name;
- Gtk::TreeModel::Path _path = (Gtk::TreeModel::Path)row;
- _treeView.set_cursor(_path, *_valueCol, true);
- grab_focus();
+ // This would be nice to have, but it causes a crash when treeview looses focus
+ // because signaling vs. focus is in some sort of conflict.
+ //Gtk::TreeModel::Path _path = (Gtk::TreeModel::Path)row;
+ //_treeView.set_cursor(_path, *_valueCol, true);
+ //grab_focus();
}
}
}