GNOME Bugzilla – Bug 164036
Having a "DVD menu" button or something similar in fullscreen mode would be nice
Last modified: 2008-09-21 07:35:50 UTC
Having a "DVD menu" button or something similar in fullscreen mode would be nice. This for changing subtitles or audio on the fly without having to leave fullscreen mode.
I don really like the idea of cluttering the fullscreen UI. Would quickkeys (the same ones as assigned to the menu items) be OK?
The menu items that have shortcuts already work in fullscreen. We could add some sub-menus to the context menu in fullscreen to switch subtitles/audio tracks.
That'd be nice. Another idea I had, not sure how good or bad, would be to show the menubar in fullscreen when the mouse moves, just like the bottom bar.
Jorn, hmm, wouldn't pressing 'M' be enough actually?
For me, yes. For most people? No, as most never use keyboard shortcuts.
It should actually be really straight forward to do now that we use GtkAction on trunk. I would show "Menu" and the subtitles and languages sub-menus when a DVD is playing.
Created attachment 103237 [details] [review] not-yet-working patch So this patch will get the menu to show... but if you click on it, it disappears. Not sure why this is happening/how to fix it.
(In reply to comment #7) > Created an attachment (id=103237) [edit] > not-yet-working patch > > So this patch will get the menu to show... but if you click on it, it > disappears. Not sure why this is happening/how to fix it. Don't use another popup, but rather add subtitles and audio tracks actions to the right-click popup menu.
Created attachment 103379 [details] [review] yet another non-working patch Gets a segfault...
Created attachment 117228 [details] [review] Working patch This should work. Please test, and I'll commit it for 2.26.
2008-09-21 Bastien Nocera <hadess@hadess.net> * data/totem.ui: * src/totem-menu.c (add_lang_action), (create_lang_actions), (totem_languages_update), (totem_subtitles_update): Add the subtitle and languages menus to the fullscreen popup as well (Closes: #164036)