-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added option for disabling grouping movies in sets when browsing a tag #10609
Conversation
@aerandir92, thanks for your PR! By analyzing the history of the files in this pull request, we identified @xhaggi, @Montellese and @FernetMenta to be potential reviewers. |
msgctxt "#40000" | ||
msgid "Show movie sets in tags" | ||
msgstr "" | ||
|
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
@@ -1377,7 +1377,8 @@ void CGUIWindowVideoBase::GetGroupedItems(CFileItemList &items) | |||
VIDEODATABASEDIRECTORY::NODE_TYPE nodeType = CVideoDatabaseDirectory::GetDirectoryChildType(m_strFilterPath); | |||
if (items.GetContent() == "movies" && params.GetSetId() <= 0 && | |||
nodeType == NODE_TYPE_TITLE_MOVIES && | |||
(CSettings::GetInstance().GetBool(CSettings::SETTING_VIDEOLIBRARY_GROUPMOVIESETS) || (StringUtils::EqualsNoCase(group, "sets") && mixed))) | |||
(CSettings::GetInstance().GetBool(CSettings::SETTING_VIDEOLIBRARY_GROUPMOVIESETS) && !StringUtils::StartsWithNoCase(m_strFilterPath, "videodb://movies/tags/") || (StringUtils::EqualsNoCase(group, "sets") && mixed)) || | |||
(CSettings::GetInstance().GetBool(CSettings::SETTING_VIDEOLIBRARY_GROUPMOVIESETSINTAGS) && StringUtils::StartsWithNoCase(m_strFilterPath, "videodb://movies/tags/") || (StringUtils::EqualsNoCase(group, "tags") && mixed))) |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Sorry, but -1 for custom options for specific nodes. Either we change default behaviour or not. |
@phil65 can this be done in an addon in a good way then? This is kinda part 1 of my quest to make tags actually usable. Next step is to be able to list tags together with movies (so that they work mostly as custom collections). Having tags hidden away in small submenu is just bad. |
I would start by adding a "add tag..." context menu item to movies / tvshows..That would improve usability quite a lot. I am not sure if a mixed movie-tag listing is what we want. |
That's a great start, actually, didn't think about that. It just annoyes me that I have to navigate back if I say I decide "I want to watch a Disney movie", instead of scrolling to D to find the Disney tag, I have to go back to home and to tags, then Disney |
Can we close this one? |
Description
Added a option under Mediasettings -> Video to choose to group movies when browsing a tag.
This options behaves completely separately from the existing general option of grouping movies in sets.
I'm quite sure some of the things I've done in the code is quite bad, and I will add my own comments with questions.
Motivation and Context
The innability to disabled grouping of movies when viewing a tag has annoyed me from the first time I created a tag. I just don't feel it always makes sense. My best example is the MCU. I have a MCU tag, and it doesn't make sense to then group the movies, I would rather be able to sort them by year which would be the intented viewing order.
Optimally, it should be possible to manually sort movies in each tag, separately, but that's probably much more work than this
How Has This Been Tested?
I've manually tested this by running a build on my PC where I've have both set grouping settings on, both off, one on and the other off, and then checked that it behaved as it should.
Types of change
Checklist: