diff options
| author | Martin Owens <doctormo@gmail.com> | 2018-09-27 16:12:37 +0000 |
|---|---|---|
| committer | Martin Owens <doctormo@gmail.com> | 2018-09-27 16:12:37 +0000 |
| commit | 705b3c16ee384d7bd04cbafa933a94545318d0fc (patch) | |
| tree | f63d4e864727fca99a918aa083af0c20fe07086b /src | |
| parent | Merge branch 'master' of gitlab.com:inkscape/inkscape (diff) | |
| download | inkscape-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.cpp | 10 |
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(); } } } |
