Help Skin the Moving Pictures GUI Importer

Discussion in 'MediaPortal 1 Skins' started by fforde, January 10, 2013.

  1. fforde

    fforde Community Plugin Dev

    Joined:
    June 7, 2007
    Messages:
    2,666
    Likes Received:
    1,690
    Occupation:
    Software Engineer
    Location:
    Texas
    Ratings:
    +1,696 / 0
    Home Country:
    United States of America United States of America
    The next major release of Moving Pictures will have an interface in the MediaPortal GUI for managing files in the importer. This is still a work in progress and this build is only meant for skin designers but there is an alpha linked below. Be sure to download the unofficial Titan skin files as well, they are listed separate. Skin designers, if you want a skin property or button or anything that is not currently there, please ask.

    Many thanks to Edalex for his initial patch for this functionality. If you see him in this thread give him a like, if it were not for him this would not be happening.

    DOWNLOAD
    This build is meant for developers, if you are not a developer download this build at your own risk.
    ------------------------------------------------------------------------------------------------------------
    movpics-importer-01.png movpics-importer-02.png movpics-importer-03.png
    Edit: more screenshots

    EDIT:Additional changes are documented on page 2.

    Included Skins
    This build should work fine in either MediaPortal 1.2.3 or 1.3 Beta, but the included skin files are targeted at 1.3 Beta. The main download includes working skin files for DefaultWide only. There is an additional download that contains Titan skin files, if you are a skin designer or are just checking things out I highly recommend checking out the Titan skin files, these are of course a work in progress but more time has been given to these.

    Also worth noting, the included Titan skin files were not created by the Titan team. This is at best a reference implementation to help other skin designer get started/inspired.

    Window ID and Controls
    The importer is implemented as a new screen with ID# 96743. This can be launched either the context menu in Moving Pictures or a skin based hyperlink (in the DefaultWide and Titan skins this is on the slide out menu).



    The only control linked to any code on the Importer screen is the file listcontrol which has an ID of 310. Each list item includes three labels, the filename(s), the currently selected movie for this file, and an enum indicating the file's current status. The third will almost certainly never be used but I figured maybe it could be used for some clever skin based conditionals or something. Each list item also has a pinicon to indicate status. Now that I am typing this up I realize these filenames are not ideal so expect these to change in the future. Suggestions are welcome.
    Code (Text):
    1. Status Icons:
    2. -------------
    3. waiting to be processed:  [no icon]
    4. currently being processed: led_blue.png
    5. needs user input:         led_yellow.png
    6. successfully imported:  led_green.png
    EDIT: Well the CODE markup on the forums here sucks...

    Skin Properties
    The following properties are pushed to the skin. If you can think of anything you want but is not available please let me know. Unless otherwise indicated these skin properties are only reliably available on the Importer screen.

    EDIT: Don't feel obligated to use all of these. But be creative. And tell me what is missing. :)

    #MovingPictures.Importer.Status: A description of the current task the importer is working on. The CurrentTask properties below are directly related to this task description. This on occasion gets blanked out for some reason, I need to figure out why.

    #MovingPictures.Importer.TotalProgressPercent: Indicates how far along the importer is as a percentage. This would be useful for a progress bar. (0-100)

    #MovingPictures.Importer.IsActive: Indicates if the importer is enabled. This will almost always be set to true. If the user has intentionally disabled the importer in the GUI this will be false. Available on main Moving Pictures Screen as well.

    #MovingPictures.Importer.Waiting.Count: The number of files that are queued in the importer but are waiting to be processed.

    #MovingPictures.Importer.NeedInput.Count: The number of files requiring attention by the user. Available on main Moving Pictures Screen as well.

    #MovingPictures.Importer.Processing.Count: The number of files currently being processed. This is usually directly tied to the number of threads running in the importer. Maybe not so useful but maybe someone can do something creative with this.

    #MovingPictures.Importer.Done.Count: The number of movies that have been successfully imported since the last time the importer was (re)started.

    #MovingPictures.Importer.CurrentTask.Count: The number of completed units of work in the current task. This can refer to import paths scanned, files queued, etc depending on context. This combined with the total field below might be useful to append to the status string. For example, "Queuing Files... (23/75)"

    #MovingPictures.Importer.CurrentTask.Total: The total number of units of work for the current task.

    #MovingPictures.Importer.CurrentTask.Percentage: The percent complete for the current task (0-100).

    Current Issues / Possible Changes
    This is a very early build so things will probably change. I have tried to stabilize everything that is included right now but this is not a final build. Below are some things I jotted down to myself while working on this. Please feel free to comment or make additional suggestions.
    • Currently the importer is restarted when you enter the GUI Importer screen for the first time. This will be fixed in the next release, but just be aware of this.
    • No feedback is given when you ignore a file. This functions correctly under the hood I just need to figure out a way to indicate the file has been ignored to the user. Suggestions are welcome.
    • I will add a "Send to Importer" action for movies in the main Moving Pictures UI.
    • Most strings are not currently translated, this of course will change.
    • The status skin property sometimes gets blanked out, not sure why. Will try to fix.
    • MediaPortal currently crashes when the Importer skin file is missing.
    What do you think?
    Please post with your feedback and thoughts. Here are a few specific things I'd like feedback on but please feel free to post with whatever concerns/interests you.
    • I would like to add some filtering functionality to the list. Most people for example will not be interested in what was successfully imported several days ago. Or may just want to see a list of stuff that needs their attention. How do you imagine yourself using the importer?
    • Are there any launch parameters you guys would like to see?
    • The the Titan and DefaultWide skin the Importer link on the hidden menu is at the very bottom. This is probably not ideal and it really should be grouped with the other primary Moving Pictures functions higher in the list.
    • Should we add an optional "Importer" entry to the Category list? Or is the slide out menu and context menu enough? Bear in mind not all users are pro like you. ;)
    • Can those stupid numbers leading each dialog entry be removed? They look horrible and provide seemingly little function.
    • Are the current number of status icons enough?
    • Do we need column headers for the list?
     
    Last edited: January 22, 2013
    • Like Like x 20
  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. mbuzina
    • Premium Supporter

    mbuzina Retired Team Member

    Joined:
    April 11, 2005
    Messages:
    2,839
    Likes Received:
    627
    Location:
    Germany
    Ratings:
    +723 / 4
    Home Country:
    Germany Germany
    Show System Specs
    The numbers allow for quick selection of a menu item using the remote number keys.
     
  4. fforde

    fforde Community Plugin Dev

    Joined:
    June 7, 2007
    Messages:
    2,666
    Likes Received:
    1,690
    Occupation:
    Software Engineer
    Location:
    Texas
    Ratings:
    +1,696 / 0
    Home Country:
    United States of America United States of America
    Sure I know this. But in practice it makes the dialog look worse. It's a feature used by 4% of users that negatively affects the other 96% of users (pretend stats but you get my point). It's not even skinnable. We are here to talk about how to make more beautiful skins. The current implementation of the leading number in dialog selections is just a little inflexible and ugly.
     
    Last edited: January 10, 2013
  5. mbuzina
    • Premium Supporter

    mbuzina Retired Team Member

    Joined:
    April 11, 2005
    Messages:
    2,839
    Likes Received:
    627
    Location:
    Germany
    Ratings:
    +723 / 4
    Home Country:
    Germany Germany
    Show System Specs
    Great work. My small suggestions:
    1. Status Values should be:
    - Queued
    - In Process
    - Needs Input
    - Imported
    - Ignored
    Name the pin icons accordingly, so the skin designer can decide how it is displayed (it is not smart to change the color of led_yellow), allow translation of the status names
    2. Have a group by status view (so you can see al "needs user attention", "ignored", etc.)
    3. Having a 2 row list is probably better, as I can see some problems with longer file / movie names. You could also show the imdb id in the list then.
    4. Headers are not needed, this is clear enough
     
    • Like Like x 2
  6. emphatic
    • Team MediaPortal

    emphatic Design Group

    Joined:
    August 25, 2006
    Messages:
    3,586
    Likes Received:
    828
    Gender:
    Male
    Occupation:
    Surface treatment
    Location:
    Alingsås
    Ratings:
    +1,083 / 2
    Home Country:
    Sweden Sweden
    Show System Specs
    I'd like to have an optional pop-up message that's shown when you enter Moving Pictures (or even restrict it to Moving Pictures->Latest added movies) that says "Importer needs your attention for #items, open importer now? YES/NO". Default is NO, so it can be quickly dismissed for higher WAF. This is also why it should be optional.

    Emph
     
    • Like Like x 2
  7. mbuzina
    • Premium Supporter

    mbuzina Retired Team Member

    Joined:
    April 11, 2005
    Messages:
    2,839
    Likes Received:
    627
    Location:
    Germany
    Ratings:
    +723 / 4
    Home Country:
    Germany Germany
    Show System Specs
    I would prefer a Status line showing that there are N items requiring my Attention when entering Moving Pics. It should not Interrupt me in any way (I hate pop-ups!)
     
    • Like Like x 1
  8. Edalex
    • Premium Supporter

    Edalex Community Plugin Dev

    Joined:
    January 3, 2008
    Messages:
    2,904
    Likes Received:
    710
    Gender:
    Male
    Location:
    Saratov
    Ratings:
    +1,225 / 9
    Home Country:
    Russian Federation Russian Federation
    Show System Specs
    1. Statuses are curently named as you say and they exposed as GUIListItem.Label3. But I'm agree with your suggestion to change icons name to something unique. Original icons with that names was only ones which suited our purposes. But you should know that each icon is mapped to sevaral statuses now.
     
    • Like Like x 2
  9. mbuzina
    • Premium Supporter

    mbuzina Retired Team Member

    Joined:
    April 11, 2005
    Messages:
    2,839
    Likes Received:
    627
    Location:
    Germany
    Ratings:
    +723 / 4
    Home Country:
    Germany Germany
    Show System Specs
  10. Edalex
    • Premium Supporter

    Edalex Community Plugin Dev

    Joined:
    January 3, 2008
    Messages:
    2,904
    Likes Received:
    710
    Gender:
    Male
    Location:
    Saratov
    Ratings:
    +1,225 / 9
    Home Country:
    Russian Federation Russian Federation
    Show System Specs
    @mbuzina Sorry maybe I don't understand you well. You need to show all ignored files for all period or what?
    Currently status is showing as ignored if you push button "Ignore" in GUI Importer. After ignoring it files will not show in importer anymore until we add "view ignored" button or smth
     
    Last edited: January 12, 2013
  11. vuego
    • Team MediaPortal

    vuego Documentation Group

    Joined:
    August 5, 2006
    Messages:
    1,570
    Likes Received:
    660
    Occupation:
    Alchemist, Astrobiologist, Exotic dancer
    Location:
    Göteborg
    Ratings:
    +868 / 0
    Home Country:
    Sweden Sweden
    Show System Specs
    Just tested this to watch a movie getting auto approved and ignoring a file and it works well (y)

    I think mbuzina would like to view the previously ignored files in GUI, just like Manage Ignored files in the Configuration - Movie Importer tab.
     
Loading...

Users Viewing Thread (Users: 0, Guests: 0)

  1. This site uses cookies to help personalise content, tailor your experience and to keep you logged in if you register.
    By continuing to use this site, you are consenting to our use of cookies.
    Dismiss Notice
  • About The Project

    The vision of the MediaPortal project is to create a free open source media centre application, which supports all advanced media centre functions, and is accessible to all Windows users.

    In reaching this goal we are working every day to make sure our software is one of the best.

             

  • Support MediaPortal!

    The team works very hard to make sure the community is running the best HTPC-software. We give away MediaPortal for free but hosting and software is not for us.

    Care to support our work with a few bucks? We'd really appreciate it!