[Crssi]

@im85288 and @jurialmunkey (i am sure both of you are following this thread ), and others too

I didnt like header in furniture, because the title was generic and had no valuable info, at least for me.
If its OK with you two, I would like to ease some improvements I see, for you two, spare you some time and try to write the code.
So I inserted a bit of code into Includes_Furniture.xml file, to show me the parent name (for whathever it is, the source, smartplaylist or folder name)

Code:

        <value condition="Window.IsVisible(MyVideoNav.xml) + !IsEmpty(Container.FolderName)">$INFO[Container.FolderName]</value> <!-- crssi -->
        <value condition="Window.IsVisible(MyVideoNav.xml)  + SubString(Container.FolderPath,special://)">$INFO[Container.FolderName]</value> <!-- crssi -->
<!-- Insert upper 2 lines BEFORE this line - crssi -->    
        <value condition="Window.IsVisible(MyVideoNav.xml) + substring(Container.FolderPath,videodb://movies) | substring(Container.FolderPath,videodb://recentlyaddedmovies)">$LOCALIZE[20342]</value>



        <value condition="Window.IsVisible(MyMusicNav.xml) + !IsEmpty(Container.FolderName)">$INFO[Container.FolderName]</value> <!-- crssi -->
        <value condition="Window.IsVisible(MyMusicNav.xml)  + SubString(Container.FolderPath,special://)">$INFO[Container.FolderName]</value> <!-- crssi -->
<!-- Insert upper 2 lines BEFORE this line - crssi -->    
        <value condition="Window.IsVisible(MyMusicNav.xml) | Window.IsVisible(MyMusicSongs.xml)">$LOCALIZE[2]</value>

Bear in mind that I mostly use video sections, here is also for the music section, but can be used for any section.
Comments with crssi signiture is there just to indentify newly inserted code for removal if needed.

The next thing I will take a look into (also easy one, but anyway) is that I dont like home header, except when playing media or paused media in background, when I would like to see playing now/paused media on the top right corner (in that case header I dont mind).
I am sure you know what I mean, if not, I will clarify myself after.

With best regards and thank you all, jurialmunkey for this best ever skin, im85288 for extending it further and anyone else who contributes constructivelly

p.s.: @jurialmunkey and @im85288
please, use this lines, alter them if needed, to improve the skin

UPDATE: NOTE:I have changed line order the first part of the code... please see screenshoots, try the code, then I will explain if needed (two lines after can be safelly removed, since racing condition would never reach them in this case, but if they stay there, "nobody will be harmed")

See description in the screenshoots:

watch gallery

[Crssi]

OK... about the other thing, that I dont like home header, except when playing media or paused media in background, when I would like to see playing now/paused media on the top right corner (in that case header I dont mind, but only now playing part).
I have altered two lines in Includes_Home.xml

Line 6:

Code:

<visible>Skin.HasSetting(home.showheader)</visible>

replaced with:

Code:

<visible>Skin.HasSetting(home.showheader) | Player.HasMedia</visible>

Line 31:

Code:

<include>Furniture_Header</include>

replaced with:

Code:

<include condition="Skin.HasSetting(home.showheader) | !Player.HasMedia">Furniture_Header</include>

Additional note for excplanation:
Altering in line 6 is telling the skin that home header should be shown when media is playing or paused even if it is disabled in settings.
Altering in line 31 is telling the header that, if it is disabled in settings, that should show only "now playing" segment of home header.

See screenshoots and read description on the picture

watch gallery

[im85288]

For anyone who wants to use the netfix style next up notification that I previously had only in the Emby addon it is now available for all: http://forum.kodi.tv/showthread.php?tid=228461

It's already skinned for this addon with icons and the plot hidden (if spoilers are hidden in settings)

@Crssi - I'm happy to include your changes if they suit the skin, can you submit pull requests and maybe post screenshots of what effect the changes have? Thanks

[Crssi]

OK... will try to make screenshots tonight with explanations, since its just a few lines of code and I am not familliar with git... at least not yet. ... but have no need to create some big stuff, since this skin is really a gem almost too perfect, lol
There are no design changes, just informational, but will be much understandable with screenshoots.
Guys, I love you for your work and sharing it for us... before I was on Aeon Nox, but this skin is clearly superior, at least from my point of view... I had a little bit of fight at home when changed to this skin, just because my family was used to previous skin... but after an hour or two of usage, they dont want to hear about changing it back. lol

[Crssi]

@im85288: i have updated my previous two suggestion posts with screenshoots and update.

[Crssi]

I have found a new bug. If you go into for example detailed info screen of the movie (button press "I" on the selected movie).
Then you select last button (to download art, poster, fanart,etc.), select any of the choices (poster, fanart,diskart, etc.).
The dialog box for download is not shown.
The dialog box is there but I believe that it is zordered behind everything, because if you press back, you are back... if you know how many times you have to press down to select one to download and pressing enter, it will actually download that item.

I can take a look into code, to spare you some time if you wish?

[arithine]

I have found a new bug. If you go into for example detailed info screen of the movie (button press "I" on the selected movie).
Then you select last button (to download art, poster, fanart,etc.), select any of the choices (poster, fanart,diskart, etc.).
The dialog box for download is not shown.
The dialog box is there but I believe that it is zordered behind everything, because if you press back, you are back... if you know how many times you have to press down to select one to download and pressing enter, it will actually download that item.

I can take a look into code, to spare you some time if you wish?

I've found the same thing.

[im85288]

I have found a new bug. If you go into for example detailed info screen of the movie (button press "I" on the selected movie).
Then you select last button (to download art, poster, fanart,etc.), select any of the choices (poster, fanart,diskart, etc.).
The dialog box for download is not shown.
The dialog box is there but I believe that it is zordered behind everything, because if you press back, you are back... if you know how many times you have to press down to select one to download and pressing enter, it will actually download that item.

I can take a look into code, to spare you some time if you wish?

Thanks, I have seen the updated screenshots and understand things much better now. As for the artwork Downloader bug..yes please do have a look..it's a feature you never need to use if you use Emby as your server..so I have to admit to never using it. I also don't really think it's a good place for media management to be presented (after all it's the movie info people want to see not select a new poster/Fanart) so if it was up to me I would remove the function completely..but I'll leave it

[Crssi]

Cool... can you just help me at the start and tell me which XML file shows this screen:



Thanx

[im85288]

Yes sure, that is DialogVideoInfo.xml

[Crssi]

How obvious... been lookin into this file couple of times and didnt find what I was looking for, was sure that some other file is involved... guess its time to do it again.

[Crssi]

Update: Felling stupid now, the identifier I was searching for was in inclusion... lol

OK... open DialogVideoInfo.xml
and AFTER line 16: <onunload>ClearProperty(Similar,home)</onunload>
insert a new line: <zorder>0</zorder>



p.s. Dont know if this is right way to do it hygienically, but it works.

[Crssi]

What do you think of this?

I know, i know, this is a design change, but just as a suggestion... dont beat me, please.

Anyway, its just a reuse of the code from pause screen, which is already the part of design.

watch gallery

[badaas]

Hmm I like that (I miss having a clock on all screens as I watch in early morning, nice to know the time before the missus starts moanin' hehe), maybe put it with the media flags.or aove the poster.. hmm or below poster so it ties in with the progression bar if film is resumable
I know atm it ties in with the logo if disc art is enabled..

[Crssi]

Dont get me wrong here. I am just thinking out "loud". This was not a wish or request.
The authors of the skin and mod are proven to be more than capable of designing and implementing.