-
-
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
[gui] dialogvideoinfo: show selectdialog for directorbutton if amount of directors > 1 #10776
Conversation
pDlgSelect->Add(director); | ||
pDlgSelect->Open(); | ||
|
||
auto iItem = pDlgSelect->GetSelectedItem(); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
} | ||
else | ||
{ | ||
auto strDirector = StringUtils::Join(m_movieItem->GetVideoInfoTag()->m_director, g_advancedSettings.m_videoItemSeparator); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
if (iItem < 0) | ||
return true; | ||
OnSearch(directors[iItem]); | ||
|
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
nice, i like the idea +1 |
… of directors > 1
5957a88
to
3bc1377
Compare
@xhaggi updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
jenkins build this please. |
Jenkins doesnt listen.... |
Director button in videoinfo dialog doesnt work in case the amount of directors is > 1.
This fixes those cases by showing a selectdialog before the actual search which allows to choose between all available directors.
@ronie @xhaggi