[Places] places applet shows more places than nautilus - don't show places that do not exist anymore

Bug #247610 reported by Kai Schroeder
2
Affects Status Importance Assigned to Milestone
Individual Screenlets
Fix Released
Medium
Papp Bence

Bug Description

I wondered why the places applet shows a lot more than the sidebar in nautilus. After some investigation I have found out that nautilus doesn't show places that do not exist anymore. Although I would prefer it if nautilus would still show them so that I can remove them manually, I think it is right to follow nautilus' behavior in the places applet. Note, that I have not looked exactly what nautilus does but at least in my configuration the attached patched modifies the applet in a way so that it returns the same amount of places as nautilus.

Related branches

Revision history for this message
Kai Schroeder (kai-schroeder) wrote :
Papp Bence (sclegnrbs)
affects: screenlets → indiv-screenlets
summary: - places applet shows more places than nautilus - don't show places that
- do not exist anymore
+ [Places] places applet shows more places than nautilus - don't show
+ places that do not exist anymore
Changed in indiv-screenlets:
status: New → Fix Committed
importance: Undecided → Medium
Papp Bence (sclegnrbs)
Changed in indiv-screenlets:
status: Fix Committed → In Progress
Revision history for this message
Papp Bence (sclegnrbs) wrote :

I checked this patch, but it is not good, because in the places screenlet there are duplicated places file.

See the screenshot: http://i.imgur.com/cY3Ns.png

Revision history for this message
Märt Põder (boamaod) wrote :

Can you also add ~/.gtk-bookmarks?

Revision history for this message
Papp Bence (sclegnrbs) wrote :

When I delete the 'test' directory, it is not deleted from the ~/.gtk-bookmarks. But from the nautilus-places it's deleted. So the nautilus storage this in other file maybe.

Revision history for this message
Papp Bence (sclegnrbs) wrote :

It was a small mistake, I think it's OK now.
Fix committed in #599.

Changed in indiv-screenlets:
status: In Progress → Fix Committed
assignee: nobody → Papp Bence (sclegnrbs)
Märt Põder (boamaod)
Changed in indiv-screenlets:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Patches

Remote bug watches

Bug watches keep track of this bug in other bug trackers.