[mythtv] [PATCH] Playlist feature UI tweaking

Kevin Kuphal kuphal at dls.net
Sun Jan 23 14:24:46 EST 2005


Chris Pinkham wrote:

>The reason I did a 'alternate' INFO popup was that some things don't
>apply to a playlist.  I wanted to make it clear that when you start
>defining a playlist, the actions available were only for the playlist,
>not for the individual item you were currently on.  Do you think it
>is clear enough the way it is in your patch?  I wonder about users
>thinking "Hey, I hit INFO on a single recording and the popup menu
>was for that recording, but when I changed the recording group, it
>changed the group for the whole playlist.  The same goes for Delete.
>  
>
I believe I misunderstood the function of the alternate popup then.   
What exactly does "Change the recording group" do when a playlist is 
present?  The function without playlists present filters the view by the 
recording group selected.  How does that differ when acting on a 
playlist?  More comments below...

>Maybe a better layout would be something like this:
>
>MENU popup
>  - Playlist Actions (calls sub-popup)
>	- Play
>	- Play Random
>	- Delete All Items
>	- Change Recording Group
>	- Clear
>	- Add title/category/recgroup/etc.
>	(And eventually maybe)
>	- Save
>	- Load
>	- Edit
>
>INFO popup
>	- Add to Playlist
>	- Remove from Playlist
>
>The "Add title/category/recgroup/etc." would add all items in the
>current right hand box.  The text could/would change depending on
>what the current view was.  This would take advantage of
>your recent "Toggle recording group view in Watch Recordings"
>patch.
>
>If this was the layout, then the Delete, Change Recording
>Group, and Play methods would have to know whether they were supposed
>to be playing a playlist or not.  They could be broken up into
>separate methods instead of having the "if (playlist.count())
>blah else blah;" statements they have in them now.
>This layout would also provide a clear distinction between what
>actions were performed on the playlist as a whole and what actions
>could be performed on the individual recording currently selected.
>  
>
I like this layout with a single button on the MENU popup for Playlist 
Actions and then the Add/Remove button as the only change to the INFO 
menu.  This keeps the INFO button acting the same across the board and 
the MENU button gets more functionality that applies to the entire 
screen as it should.  This feels more intuitive to me than the original 
or my patch.

Did you say you had this in you tree already or is it something you'd 
like me to work up?

Kevin



More information about the mythtv-dev mailing list