linuxstb / pidvbip

tvheadend client for the Raspberry Pi
http://www.pidvbip.org
GNU General Public License v2.0
53 stars 22 forks source link

Onscreenchannellist #18

Closed andyb2000 closed 11 years ago

andyb2000 commented 11 years ago

Try again! This implements the on screen channellist.

eindano commented 11 years ago

I did an implematation of OSD channel list yesterday - just needs to be cleaned up a bit so 90% ready :-) But anyway, a few comments:

Will see if I clean-up my code or not :-)

linuxstb commented 11 years ago

eindano,

Happy to look at your code and take the best of both. Note that I've been discussing Andy's code with him on IRC and he's been fighting with git, so I don't this is his final PR.

eindano commented 11 years ago

Hi,

First, thanks for doing this - really good to have an inexpensive Tvheadend frontend :-) I will send you my code as soon I got time to get in a good state. If you want any help with some special please let me know - I can help out with testing and coding.

Andy - I hope you didn't take my comments in a negative way. As I just had made the same thing I was curious how you did the implementation and wanted to help you out.

//Daniel

andyb2000 commented 11 years ago

Hi @eindano no negativity taken at all, glad to have somebody else with some changes and suggestions, sounds good. You'll see I've discussed quite a bit with @linuxstb as we lurk on IRC with TVHeadend development too. Feel free to send a PR in for mods, chances are @linuxstb will merge this in shortly and then take mods/fixes as this implements a few functions we're after having.

Feel free to drop a node over sometime @eindano as would be happy to discuss :-)