GNOME Bugzilla – Bug 397776
Scrolling dropdown in sidebar does not switch entries
Last modified: 2007-06-28 22:28:50 UTC
This bug was filed in the Debian BTS. http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=407046 "When the sidebar is enabled and you scroll the Playlist/Properties dropdown element, totem skips through the currently played song. But it should switch between the entries of this dropdown box instead since this is the default and expected GTK behaviour for hovered dropdown/combobox elements." I'm not sure if this is an issue with Totem or GTK+ as at least Nautilus has a similar issue. (The path selection button in the lower left of spatial Nautilus doesn't respond to scrolling either.)
Created attachment 87599 [details] [review] Use a default GtkComboBox instead of the custom button menu combination
Patch looks OK to me, although I could quibble over the new padding between the combobox and the playlist box, which looks a little much to me.
Created attachment 90827 [details] [review] v2: Rework padding/border
I don't think we need padding on the left (inside) side of sidebar, as we already have the paned doing the separation. Looks good otherwise.
Created attachment 90838 [details] [review] v3: use 6 6 6 0 border
Looks great. Thanks for the patch.
2007-06-29 Jan Arne Petersen <jpetersen@jpetersen.org> * data/playlist.glade: * src/ev-sidebar.c: (ev_sidebar_destroy), (ev_sidebar_combobox_changed_cb), (ev_sidebar_init), (ev_sidebar_get_current_page), (ev_sidebar_get_iter_for_page_id), (ev_sidebar_set_current_page), (ev_sidebar_add_page), (ev_sidebar_remove_page): Use a default GtkComboBox instead of the custom button menu combination (Closes #397776)