[mythtv] [mythtv-commits] mythtv/master commit: 8e2982436 by Paul Harrison (paul-h)

[Paul Harrison]

On 07/01/12 23:29, Stuart Morgan wrote:
> On Saturday 07 Jan 2012 22:37:46 Paul Harrison wrote:
>>
>> The only way at the moment is with the  MARK keybinding (probably a bad
>> name for it now that I think about it) the default is the 'T' key which
>> is similar to what you did in MythGallery to mark images in the gallery
>> there. I did briefly try other more accessible keys like PLAY/PAUSE but
>> that quickly becomes annoying when you go to pause the music only to
>> find you've added something to the playlist instead and I know we are
>> trying to not overload actions like that anyway.
> OK that makes sense. There's a binding we're using in PBB for toggle recording 
> in a playlist (TOGGLE_PLAYLIST iirc). This seems like a natural choice. Since 
> we'd be using the functionality in at least three different places it could be 
> a global binding which would make it consistent. I'm just stuck on where it 
> would belong on a standard remote but I suppose that's why we're also making 
> things available through the MENU.
>
We seem to be reusing the "NEXTFAV" keybinding to toggle a playlist item
in PBB. I don't see anything like TOGGLE_PLAYLIST being used anywhere
otherwise I'd switch to using that?
 
>> I was contemplating a context menu like the one I put in the in the search
>> view to see how I liked it. I'm still undecided whether I like it or
>> not. I'm open to suggestions.
> I'll give it some thought.

I'm going to add a context menu. Let me know what you think after trying
it for a few days.

Paul H.