Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#9849 closed Patch - Feature (Invalid)

[Feature Wish] Watch Recordings with A-Z pre-selector

Reported by: tobias@… Owned by: Janne Grunau
Priority: minor Milestone: unknown
Component: MythTV - Video Playback Version: 0.24
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Hello,

I have too many recordings (~ 700). At the moment it is very hard to scroll to a recording beginning with a letter in the middle of the alphabet as I have to scroll through all recordings. It would be great if one could add the possibility to first select a character e.g. "M" and then get only a listing of all recordings beginning with "M".

I have made a little mockup which should illustrate this.

This "pre-selector" should be a new column and not replace the recordings grouped by the same name.

Thank you for your great application! Tobias

Attachments (5)

mockup_mythtv.png (223.3 KB) - added by anonymous 8 years ago.
snapshot.png (176.4 KB) - added by tobias@… 8 years ago.
Preview of working a-z selector
azselector.patch (10.9 KB) - added by tobias@… 8 years ago.
Patch for playbackbox.cpp & .h
azselector-theme.patch (8.5 KB) - added by tobias@… 8 years ago.
Patch for theme (recordings-ui.xml)
reclist_background.png (8.6 KB) - added by tobias@… 8 years ago.
Background image for MythCenter?-wide

Download all attachments as: .zip

Change History (10)

Changed 8 years ago by anonymous

Attachment: mockup_mythtv.png added

comment:1 Changed 8 years ago by robertm

Resolution: Invalid
Status: newclosed

Please read the ticket howto, we do not accept feature requests.

Changed 8 years ago by tobias@…

Attachment: snapshot.png added

Preview of working a-z selector

comment:2 Changed 8 years ago by tobias@…

Hello robertm!

I have made two patches (one for playbackbox.cpp and playbackbox.h, the other one for recordings-ui.xml) and created a new reclist_background.png to get the above idea working.

I'll attach a preview of this and the three mentioned files.

What else do I have to do in order to get this (hopefully) merged in SVN?

Thank you Tobias

Changed 8 years ago by tobias@…

Attachment: azselector.patch added

Patch for playbackbox.cpp & .h

Changed 8 years ago by tobias@…

Attachment: azselector-theme.patch added

Patch for theme (recordings-ui.xml)

Changed 8 years ago by tobias@…

Attachment: reclist_background.png added

Background image for MythCenter?-wide

comment:3 Changed 8 years ago by jpoet

I just took a quick look at the azselector.patch, and here are some random thoughts:

It does not look like you are following myth's coding style: http://www.mythtv.org/wiki/Coding_Standards

Also, please create unified diffs. git will do that by default (e.g. "git diff > azselector.patch")

It looks like this patch would require all themes to support the new buttonlist. I can pretty much guaranty that that will not be an acceptable requirement.

It looks like it populates the new buttonlist with the entire Latin alphabet, even if there are no show titles that start with a given letter. If someone was just displaying a "News" recording group, they might only have a few shows in the list: CBS News, NBC News, etc. To get to NBC News, would the person have to scroll all the way through A,B,C,D,E,F,G,H,I,J,K.L.M,N to get there?

Even if the theme does support the new buttonlist, the user might want to see all the recordings in a single (ALL) list -- does this patch allow for that?

Is there any need to support diacritic letters, or are those handled under @ ?

comment:4 Changed 8 years ago by sphery

Tobias, thanks for taking the time to generate a patch. Unfortunately, there are a couple of problems with the patch (as mentioned above). However, I will be incorporating some new functionality which will allow the functionality you want (and more). The plan is to modify MythTV to allow tagging recordings with any number of user-specified tags, then allow filtering the Watch Recordings list by to show recordings in any number of tags with and/or relationships). At this point, you'll be able to tag your recordings with tags, A-Z, 0-9, @, and then tell MythTV to display recordings with any of those tags with the tag shown as the "category" (left hand side). This change will come after a change to the way we're doing our recording file schema, but it is on the list. :)

In the meantime, feel free to continue using your patch on your system. Thanks, again.

comment:5 Changed 8 years ago by tobias@…

Hi, jpoet, mdean!

JPoet thank you for taking a look at my patch. Your thoughts are completly right, it is my first time coding for a project that is not mine, so I expected to do something not the way it should be.

Mdean: Will these changes you are describing coming so fast that it does not make sense to develop this patch any longer?

If this is a long term plan I am willing to fix the problems jpoet stated above to get this patch included.

Thank you Tobias

Note: See TracTickets for help on using tickets.