| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Co-authored: Sabri Ünal <libreajans@gmail.com>
|
|
|
|
| |
Co-authored: Sabri Ünal <libreajans@gmail.com>
|
| |
|
| |
|
|
|
|
|
|
| |
Instead of letting the front-end handle magic numbers for no subtitle,
or automatic lang selection, add those directly to the backend, and
export an opaque "id" for the front-end to use.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of having the front-end keep track of track lists, and compare
them, keep them inside the video widget. This also fixes the old
totem-menu code that didn't check for title equality when comparing
lists.
Note that this updates the subtitles and languages menu too often,
as we're *always* updating the menu even when it hasn't changed.
Fixes: becbe7f8ef61f51ccbbb228f248b85b02e1eeab9
|
|
|
|
|
|
|
|
|
|
|
| |
Due to recent changes in gst-plugins-base, some languages without a 2
letter-code are now getting mapped to languages using
gst_tag_get_language_name(). Which isn't a problem, except that we were
relying on gst_tag_get_language_name("und") returning NULL.
Check for "und" manually so that track menus get named properly.
See https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/-/merge_requests/785
|
|
|
|
| |
Update Bugzilla links for GitLab links when the bugs are still opened.
|
|
|
|
| |
An enum is much easier to understand.
|
|
|
|
| |
Commit d628a08c5c fixed it half-way, now also fix the menu name.
|
|
|
|
|
| |
This was what get_label_for_type() did, make sure not to lose that
support.
|
|
|
|
|
| |
To make sure that the display of language names and codecs never
regresses.
|
|
|
|
|
|
|
|
| |
If we have 2 audio tracks for the same language, include the codec
information to try and differentiate them. If the language/codec
information is also used for 2 tracks, fallback to numbering the tracks.
Closes: #214
|
|
|
|
|
|
|
|
|
|
| |
Instead of simply returning a list of language codes for subtitles and
audio tracks, return a struct contains both the language code and the
audio codec. We will be able to use this to differentiate tracks better
in the menus.
There are no functional or UI changes in this commit, just an API
extension.
|
|
|
|
|
|
| |
"und" means "Defaut languague" in Matroska containers:
https://matroska.org/technical/specs/tagging/index.html
so assume this means English, as it does in VLC.
|
|
|
|
|
| |
"None" only appears for subtitles selection, not for audio track
selection.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Move items from the app menu to a menu button in the main window. This
new hamburger menu will show in the content navigation/selection view
and include the same items that were available in the app menu.
As we do not want 2 hamburger menus in the player view, duplicate the
preferences and keyboard shortcuts menu items in the player menu.
See:
https://gitlab.gnome.org/GNOME/Initiatives/wikis/App-Menu-Retirement
Closes: #265
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
duplicate ‘const’ declaration specifier [-Wduplicate-decl-specifier]
The warning was added in:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43651
just for this purpose.
"
int foo(const char const *data);
^~~~~~
1 warning generated.
[...] In that case it was clearly a mistake, because the author meant
'const char *const data' [...]
"
|
| |
|
|
|
|
|
|
| |
It doesn't include shortcuts from optional plugins, yet.
http://bugzilla.gnome.org/show_bug.cgi?id=757831
|
|
|
|
|
|
|
|
|
| |
Instead of implementing command-line parsing ourselves, use
the "handle-local-options" signal along with
g_application_add_main_option_entries() to parse command-line options.
Command-line options that work with a remote primary instance will be
passed over D-Bus with the "remote-command" action.
|
|
|
|
|
|
| |
Register the GActions early, so that they are available even when
the main window isn't. This is necessary to make remote commands
available on D-Bus.
|
|
|
|
| |
Which will be available on the bus for us to use.
|
|
|
|
| |
Plenty of deprecated code gone.
|
| |
|
|
|
|
| |
It's already setup in the properties plugin.
|
|
|
|
| |
Instead of add_accelerator() and remove_accelerator().
|
| |
|
| |
|
|
|
|
|
| |
We now have a shuffle button in the grilo plugin, to allow
shuffling the selection.
|
|
|
|
| |
Helps: https://bugzilla.gnome.org/show_bug.cgi?id=626399
|
| |
|
| |
|
|
|
|
| |
We have a single menu path to handle now.
|
|
|
|
|
| |
The cogwheel menu is visible in fullscreen and will contain
pretty much everything the popup did.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
The volume up/down menu items are gone.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|