[feature req] pvr overview rework - not an issue

[howie-f]

in my naivety i'm saying that is not a big deal ;) though only a feature request.
can the pvr overview get a bit tuned, like it is faked here for recording no. 2 ? cheers

[wutschel]

That will be quite easy to implement as "art" is already requested (and provided by Kodi) for recordings. Just need to ensure this is not impacting other thumbviews, and check if this works for timers as well. Looks like this after a 3-lines-of-code change:

[wutschel]

Implemented and added to #158.

[howie-f]

thanks much for implementing this.
can we adjust it so the station logos for pvr recordings (left side) match in size for e.g. channel overview?

[wutschel]

Let me look into this.

[wutschel]

It´s quite hard to adapt the recording thumb sizes to the current channel list thumb sizes properly. What is easy to achieve is to adapt both thumbs to a similar look. The logos are using the complete cell but with a frame for better readability.

https://abload.de/img/simulatorscreenshot-iq7jti.png
https://abload.de/img/simulatorscreenshot-i3dkes.png
https://abload.de/img/simulatorscreenshot-iz5jdq.png

[howie-f]

It´s quite hard to adapt the recording thumb sizes to the current channel list thumb sizes properly.

sorry, didnt expect this 😏

[wutschel]

I find the solution from the PR visually also quite nice. But the PR is raised as draft, maybe I find a better solution. Problem is that the menu level (and logic) in the recordings view is different to the one in the channel list. With the straight forward solution of adding same sizes of the UIImageView I get side effects I do not understand yet. But of course there´s no need to hurry...

[wutschel]

Now I think I found a way. But I will ask kambala for some help on this. There is something I do not understand fully yet: To achieve this I need to stop using dequeueReusableCellWithIdentifier which I understood makes a performance impact.

Looks like this now:
https://abload.de/img/simulatorscreenshot-ig1k4h.png
https://abload.de/img/simulatorscreenshot-inyjn6.png