[Martijn]

What i never understood is why seasons is done the way it is. On the one hand, yeah seasons is a step between show and episode. But i think it is better to send people straight to titles or episodes instead of seasons or a window to choose a genre. Most used = first place. To me it would make more sense to have the seasons list and episodes list in the same screen. In a dropdown, or side menu, like how Confluence starts PVR (directly visible), or a button with a popup. So seasons is just an extra on episodes. Just something that crossed my mind a few times.

please no.

[MassIV]

[ronie]

What i never understood is why seasons is done the way it is. On the one hand, yeah seasons is a step between show and episode. But i think it is better to send people straight to titles or episodes instead of seasons

you can skip the season list, if that's what you're after:
settings > videos > flatten tv shows

[MassIV]

No, i know that. A nice work around :p

[jingai]

It was intentional, primarily because it's easy to change tvshow stuff - much harder to do so for seasons (no season info).

Ok. Well, if it's ever a possibility in the future, please consider it. In flattened lists it's particularly nice to have.

BTW, really appreciate all the work on this. Now I won't have to answer questions about squashed images every other day

[Hitcher]

Working via the context menu but using the 'choose art' from video info sets the TV show images not season ones.

Thanks - there's no official "season" support in the info dialog, so it doesn't really surprise me if it doesn't necessarily show up correctly - the context menu stuff will stay in either way, so not a problem IMO.

In that case can button 10 be disabled at season level to avoid any user confusion as to what image they're actually setting?

I tried this at skin level but Container.Content(seasons) isn't valid from the video info dialog.

[Hitcher]

ListItem.Art(season.poster) and ListItem.Art(season.banner) stopped working for video info dialog at season level recently (not sure when yet) but they still work at episode level.

[jmarshall]

This was deliberate, yes. When you go to Video Info at the season level you actually just get the show information (it's labelled TV Show Information in the UI for this reason).

[Hitcher]

OK, but seeing as ListItem.Art(tvshow.poster) & ListItem.Art(tvshow.banner) are available at season level, and ListItem.Art(banner) & ListItem.Art(poster) show the actual season images, shouldn't it also be this way in the video info dialog?



As it is now ListItem.Art(banner) & ListItem.Art(poster) then show the tvshow images and ListItem.Art(tvshow.poster) and ListItem.Art(tvshow.banner) are empty.



Hope that makes sense.

Thanks.

[jmarshall]

The video info dialog does not support seasons. When you go to Information it's the TV Show Information. It's always been that way, just there was a bug that didn't correctly update the type of item from season->show. i.e. it worked by accident, but was inconsistent, in that the Choose Art stuff updated the art of the tvshow, not the season (while showing art for the season instead of show).

Cheers,
Jonathan

[nessus]

JM when you have the time can you give a look on this ?

Cheers
Nessus

[Hitcher]

@jonathan Understood, thanks.

@nessus As I said in that thread they're exactly the same and still working fine for me.

[`Black]

@nessus As I said in that thread they're exactly the same and still working fine for me.

Nessus is correct, LatestEpisode.*.ShowThumb is empty now which should not be the case I guess. Problem could be that tv shows have no thumb, so item->GetArt("tvshow.thumb") has to be item->GetArt("tvshow.poster").

[Hitcher]

Sorry, I thought he meant all images.

[jmarshall]

I'll add a fallback from tvshow.thumb -> tvshow.poster.