[mythtv] [PATCH] Playlist UI Updated

Chris Pinkham cpinkham at bc2va.org
Fri Jan 28 00:32:43 EST 2005


> This patch updates the playlist UI with feedback from Chris Pinkham.  It 
> does the following:

Thanks for taking care of this.  Looks great.

I've applied the patch to current CVS after a couple minor mods.  I had
to change the argument to ScheduledRecording::signalChange() which should
be a number and you were passing in a QSqlDatabase::database().  Not sure
how that even compiled. :)  I also changed a few lines that were over 80
characters long, and I added a togglePlayListTitle(void) method.  With all
the work you did, I noticed one last thing that could be confusing to the
user.  If you hit INFO while on the left hand title box and selected
"Add To Playlist", the info displaying was for a specific recording, but
the whole title/group/etc. would be added to the playlist since
togglePlayListItem(void) was just checking "if (inTitle)" when it determined
whether to add the individual recording or the whole group.  It wasn't
checking to see where it was called from.  So, I made a new SLOT that is
called for the buttons that say they are going to affect all items in
the title/group and used the original togglePlayListItem slot for the
INFO popup menu and for the MENU popup when the focus is on a specific
recording using the same checks you have to display different text
on the button depending on the values of inTitle and titleView.

You'll probably have to un-apply your patch before syncing with CVS since
I changed a few things.  Can you re-update and make sure you think the
changes for the INFO popup's "Add to Playlist"/"Remove from Playlist"
button makes sense?  I just thought it was kind of confusing when I was
on a Title and hit INFO and it popped up the INFO menu, but when I hit
"Add to Playlist", instead of adding just the recording who's information
was on the INFO menu, it instead added all recordings for that title.

-- 

Chris



More information about the mythtv-dev mailing list