After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 72043 - reordering rows emits bogus selection_changed
reordering rows emits bogus selection_changed
Status: RESOLVED WONTFIX
Product: gtk+
Classification: Platform
Component: Widget: GtkTreeView
1.3.x
Other other
: Normal normal
: Medium fix
Assigned To: gtktreeview-bugs
gtktreeview-bugs
: 158613 (view as bug list)
Depends on:
Blocks: 72723 157066
 
 
Reported: 2002-02-20 20:04 UTC by jacob berkman
Modified: 2006-05-28 13:04 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Description jacob berkman 2002-02-20 20:04:35 UTC 
if i have a tree view with selection set to BROWSE and i set it as
reorderable, when i do a dnd to reorder it, i get a selection changed
callback with no rows selected

this shouldn't happen in BROWSE mode.
Comment 1 Jonathan Blandford 2002-02-25 22:00:31 UTC 
After a bit of discussion, we decided to not make the tree reselect
items.  As a result, you need to make sure that BROWSE is always
selected.  The only thing BROWSE guarantees is that the user can't
unselect rows.
Comment 2 jacob berkman 2002-02-26 20:07:32 UTC 
i believe the discussion was for the case of an app removing a row
explicitely.

for dnd the same rows should still be selected before and after the dnd.

reopening.
Comment 3 Owen Taylor 2002-03-30 01:31:36 UTC 
Moving remaining 2.0.1 bugs to 2.0.2 milestone.
Comment 4 Matthias Clasen 2002-04-05 13:33:35 UTC 
Move open bugs from milestones 2.0.[012] -- > 2.0.3, since 2.0.2 is already out.
Comment 5 Kristian Rietveld 2002-04-24 13:46:27 UTC 
If you reorder a row, the DnD code will remove the row and then add it
at the new place. Because of this ::selected_changed gets emitted.

I dont see a way to get around this, as you cant just move the row and
emit ::rows_reordered in trees (as ::rows_reordered only covers single
levels), you can in lists though.
Comment 6 Kristian Rietveld 2002-05-02 19:41:04 UTC 
Reassigning bugs to new component owner. Sorry for the flood.
Comment 7 Owen Taylor 2002-07-30 21:11:30 UTC 
Moving some 2.0.4 bugs to 2.0.7
Comment 8 Matthias Clasen 2002-11-21 19:00:45 UTC 
Moving bugs from older 2.0.x milestones to 2.0.10.
Comment 9 Kristian Rietveld 2003-05-22 21:13:07 UTC 
Not sure what to do here, will ask jrb. Putting on future for now.
Comment 10 Matthias Clasen 2004-11-19 03:33:24 UTC 
*** Bug 158613 has been marked as a duplicate of this bug. ***
Comment 11 Kristian Rietveld 2005-06-17 22:44:23 UTC 
Bug 158613 was marked as a dup of this bug, because of the delete-insert pair
'problem' with DnD. I am not really sure if having rows_reordered handle
multiple levels of nodes (we would need a new signal for this) is a good idea.