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 762137 - GtkPlacesSidebar: row selection jumps around
GtkPlacesSidebar: row selection jumps around
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Sidebar
3.19.x
Other Linux
: Normal normal
: 3.20
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-02-16 12:58 UTC by Allan Day
Modified: 2016-02-24 10:23 UTC
See Also:
GNOME target: 3.20
GNOME version: ---


Attachments
screencast of the issue (225.79 KB, application/octet-stream)
2016-02-16 12:58 UTC, Allan Day
  Details
window: use slot location instead of loading (1.75 KB, patch)
2016-02-22 10:01 UTC, Carlos Soriano
committed Details | Review

Description Allan Day 2016-02-16 12:58:12 UTC
Created attachment 321366 [details]
screencast of the issue

Observed in an up to date continuous image today: if you select a row in the places sidebar, the selection sometimes jumps to another row, and then back to the one you selected.

You can see the issue on the attached screencast.
Comment 1 Carlos Soriano 2016-02-22 10:01:05 UTC
Created attachment 321817 [details] [review]
window: use slot location instead of loading

We were changing the location of the sidebar every time the slot started
loading.
However, that's dangerous since the slot can start loading few times
even with the old location.

Instead of using the loading property, use the location property and
change the sidebar only when the location actually changes on the slot.

This fixes the sidebar jumping rows with gtk+ 3.20.
Comment 2 Carlos Soriano 2016-02-24 10:23:35 UTC
Attachment 321817 [details] pushed as 9da55e1 - window: use slot location instead of loading