Skip to content
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

[video] Video Versions/Extras: More smaller cleanup and fixes #24356

Merged
merged 4 commits into from Dec 29, 2023

Conversation

ksooo
Copy link
Member

@ksooo ksooo commented Dec 29, 2023

Some smaller cleanups and fixes for Video Versions and Extras:

1) Movie window -> node "Versions" does now only show what it says, versions, no extras anymore.

Before:
screenshot00000

After:
screenshot00002

2) Fix: When playing an extra, the OSD contained the information of the movie the extra was attached to, which makes no sense, as the extra is not the movie itself. What is correct fpor versions, is not for extras.

3) Make the labels for video conversion more user friendly:

screenshot00005
screenshot00006

4) Swapped position of context menu item "Add as version to..." and "Manage versions". Now, "Manage movie sets" and "Manage versions" are located next to each other, followed by "Add as version to...". See screen shot above.

@ksooo ksooo added Type: Fix non-breaking change which fixes an issue Type: Improvement non-breaking change which improves existing functionality Component: Video v21 Omega Feature: Video Versions/Extras labels Dec 29, 2023
@ksooo ksooo added this to the Omega 21.0 Beta 3 milestone Dec 29, 2023
Copy link
Member

@enen92 enen92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but if someone else could take a look into the dB changes it would be better

@CrystalP
Copy link
Contributor

CrystalP commented Dec 29, 2023

I know that I suggested "Add as version to..." but after thinking about it, it doesn't convey the idea that the movie will be removed from the movies list.

Alternative: "Convert to version of...". It implies that the current item will be transformed.
The difference with the current message is the addition of "of...", which I think clarifies things enough.

But I'll go with the consensus.

@atrugiel69
Copy link

I wrote this in the development discussion: https://forum.kodi.tv/showthread.php?tid=337992&pid=3177462#pid3177462

And it was written several days ago. Why is this proceeding with two different menu entries? I remain convinced it's unneeded and confusing for end users.

@ksooo ksooo merged commit f89e88a into xbmc:master Dec 29, 2023
2 checks passed
@ksooo ksooo deleted the video-versions-extras-stuff branch December 29, 2023 18:10
@ksooo
Copy link
Member Author

ksooo commented Dec 29, 2023

And it was written several days ago. Why is this proceeding with two different menu entries? I remain convinced it's unneeded and confusing for end users.

@atrugiel69 I do agree, but I'm not in a position to decide this without having the okay from the team.

@CrystalP
Copy link
Contributor

CrystalP commented Dec 29, 2023

Technically both paths accomplish the same thing in the end.
It's a matter of personal preference and mindset. Start with the movie to attach (context menu), or with the movie to be attached to (manage dialog).

@da-anda
Copy link
Member

da-anda commented Dec 29, 2023

I'm fine with dropping the additional context menu entry. So far I just found it at least a bit more straight forward than the manage dialog where you end up in a file browser instead of a movie list of your library (it's fine for extras, but unexpected for movies). If we think we could fix this behavior for Omega, then I'm more than fine to get rid of the second context menu entry. If we don't want to push RC another month just for the video versions, I might be in favor of keeping it, at least for Omega, and then ditch it in P* along with more improvements/refactoring. But no hard feelings either way.

PS: getting rid of the file browse dialog also will prevent a NAS from having to spin up just for converting library entries into versions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Video Feature: Video Versions/Extras Type: Fix non-breaking change which fixes an issue Type: Improvement non-breaking change which improves existing functionality v21 Omega
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants