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 682174 - Toggle buttons for list view and icon view briefly jump back to 'not pressed' until view has changed
Toggle buttons for list view and icon view briefly jump back to 'not pressed'...
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Main Toolbar
3.5.x
Other Linux
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
: 634903 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2012-08-18 22:44 UTC by Volker Sobek (weld)
Modified: 2012-09-18 18:21 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Don't try to change the view buttons to the old view while loading (1022 bytes, patch)
2012-09-05 18:33 UTC, William Jon McCann 		committed Details | Review

Description Volker Sobek (weld) 2012-08-18 22:44:27 UTC 
If you click on the list view or icon view button and release the mouse button, the button changes from 'pressed' to 'not pressed' for a second (until the view has changed?) before it changes to 'pressed' again.

Nothing crucial, it just doesn't feel crunchy.
Comment 1 William Jon McCann 2012-09-05 18:33:21 UTC 
Created attachment 223574 [details] [review]
Don't try to change the view buttons to the old view while loading

The code tries to sync up the view type buttons while the new
view is loading but the code tries to sync the buttons up with
the last view instead of the pending view. I think it is just
better to skip the operation while the view is loading.
Comment 2 Cosimo Cecchi 2012-09-05 18:43:43 UTC 
Review of attachment 223574 [details] [review]:

Makes sense.
Comment 3 William Jon McCann 2012-09-05 19:06:36 UTC 
Attachment 223574 [details] pushed as 9546a16 - Don't try to change the view buttons to the old view while loading
Comment 4 William Jon McCann 2012-09-18 18:21:29 UTC 
*** Bug 634903 has been marked as a duplicate of this bug. ***