Opened 14 years ago
Closed 14 years ago
Last modified 14 years ago
#9849 closed Patch - Feature (Invalid)
[Feature Wish] Watch Recordings with A-Z pre-selector
Reported by: | 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)
Change History (10)
Changed 14 years ago by
Attachment: | mockup_mythtv.png added |
---|
comment:1 Changed 14 years ago by
Resolution: | → Invalid |
---|---|
Status: | new → closed |
comment:2 Changed 14 years ago by
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 14 years ago by
Attachment: | azselector-theme.patch added |
---|
Patch for theme (recordings-ui.xml)
Changed 14 years ago by
Attachment: | reclist_background.png added |
---|
Background image for MythCenter?-wide
comment:3 Changed 14 years ago by
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 14 years ago by
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 14 years ago by
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
Please read the ticket howto, we do not accept feature requests.