[Approved] GUIListControl, popup first letter when scrolling (1 Viewer)

[pilehave]

The purpose of this patch is already described in https://forum.team-mediaportal.com/...re-48/new-idea-scrolling-borrowed-xbmc-87938/ so I'll add the description from this thread.

The attached patch is a bit simpler than the original ones were.

Purpose:
Show the first letter of #selectedindex as an independent label when scrolling. The label is only shown when you hold the up or down key down:





This is a common feature in XBMC, where they use a property called "Container.Scrolling" that indicates if the user is scrolling. If you have tried XBMC and used the built-in skin Confluence you will know this:



Solution:
Patch GUIListControl.cs to output the firstcharacter of #selecteditem as #selecteditem.firstchar and the properties of #scrolling.up and/or #scrolling.down to values "yes" or "no"


The result of my patch is shown in the screenshots with Blue3Wide and Maya, and in my video on YouTube (with Maya as skin).

Video on YouTube

To get this working with Blue3Wide:
1. Apply patch (duh) and compile MediaPortal

2. Copy the attached PNG listscroller_bg.png to the "Media" folder of Blue3Wide

3. Copy this code into the screens where you want the listscroller to show up. In these examples I have used mymusicsongs.xml and myvideo.xml. To see the control in use, you must use the "Shares" view of MyMusic or just MyVideos and select "List":

	<control>
        <description>listscroller bg</description>
        <type>image</type>
        <id>1</id>
        <posX>486</posX>
        <posY>620</posY>
        <width>69</width>
        <height>104</height>
        <texture>listscroller_bg.png</texture>
        <visible>facadeview.list+string.contains(#scrolling.up,yes)|string.contains(#scrolling.down,yes)</visible>
        <animation effect="slide" start="0,50" end="0,0" delay="250" time="150" reversible="true">visiblechange</animation>
    </control>
    
    <control>
        <type>label</type>
        <id>1</id>
        <posX>482</posX>
        <posY>620</posY>
        <width>78</width>
        <height>90</height>
        <label>#selecteditem.firstchar</label>
        <textcolor>6aa5cb</textcolor>
        <visible>facadeview.list+string.contains(#scrolling.up,yes)|string.contains(#scrolling.down,yes)</visible>
        <animation effect="slide" start="0,50" end="0,0" delay="250" time="150" reversible="true">visiblechange</animation>
        <align>center</align>
        <valign>middle</valign>
    </control>

Hope you all support my idea

 

[Inker]

I think you forgot to attach the patch

 

[pilehave]

I think you forgot to attach the patch

Haha...you're absolutely right :sorry:

Fixed in first post.

 

[DieBagger]

Could you please also attach the fully skin files? Makes it much easier to test than having to search for the controls in the xml files...

Awesome patch btw

 

[pilehave]

Could you please also attach the fully skin files? Makes it much easier to test than having to search for the controls in the xml files...

Awesome patch btw

I have attached files for music and videos for testing purposes

 

[disaster123]

AW: GUIListControl, popup first letter when scrolling

+1 vote for this patch

 

[catavolt]

AW: GUIListControl, popup first letter when scrolling

Nice gimmick, we should integrate it after 1.2.0 Alpha

 

[jameson_uk]

Not at a machine where I can look at this but a couple of questions.

1. Is this configurable so could be turned off if wanted?
2. What happens if view is not sorted alphabetically?
3. What happens if view is not textual (eg a list of years in music)
4. Does this check for the control being null (ie. not implemented in skin?)

Thanks

 

[DieBagger]

Tested it and it's working very well but jameson_uk had a few pretty good points:

Not at a machine where I can look at this but a couple of questions.

1. Is this configurable so could be turned off if wanted?
2. What happens if view is not sorted alphabetically?
3. What happens if view is not textual (eg a list of years in music)
4. Does this check for the control being null (ie. not implemented in skin?)

Thanks

1) Depends on the skin, in the current patch there is no config option (do we need one?).
2) It becomes pretty useless (@pilehave: can you disable it when sorted other then alphabetically?)
3) It will display the first character.
4) the patch doesn't introduce a new control, just skin properties (#scrolling.up, #scrolling.down, #selecteditem.firstchar)

This is an addition of a really neat feature that xmbc already has, so we should definately include this for 1.2 beta...

 

[pilehave]

Tested it and it's working very well but jameson_uk had a few pretty good points:

1) Depends on the skin, in the current patch there is no config option (do we need one?).
2) It becomes pretty useless (@pilehave: can you disable it when sorted other then alphabetically?)
3) It will display the first character.
4) the patch doesn't introduce a new control, just skin properties (#scrolling.up, #scrolling.down, #selecteditem.firstchar)

This is an addition of a really neat feature that xmbc already has, so we should definately include this for 1.2 beta...

As far as I can tell it should be possible to expose the sortmethod in this function from GUIListControl.cs:

    public void Sort(IComparer<GUIListItem> comparer)
    {
      try
      {
        _listItems.Sort(comparer);
      }
      catch (Exception) { }
      _refresh = true;
    }

There you can disable the scrollcode if it is not sorted by name (which is alphabetically). But I'm not sure of how it would be correct to do it