Kodi Community Forum
Release Amber for Leia/Matrix/Nexus - Printable Version

+- Kodi Community Forum (https://forum.kodi.tv)
+-- Forum: Support (https://forum.kodi.tv/forumdisplay.php?fid=33)
+--- Forum: Skins Support (https://forum.kodi.tv/forumdisplay.php?fid=67)
+---- Forum: Amber (https://forum.kodi.tv/forumdisplay.php?fid=203)
+---- Thread: Release Amber for Leia/Matrix/Nexus (/showthread.php?tid=342130)



RE: Amber for Leia - bsoriano - 2019-03-31

(2019-03-31, 09:46)DarrenHill Wrote:
(2019-03-31, 01:56)bsoriano Wrote: @DarrenHill , I honestly had not tested with those addons.  I think I fixed the issue, and I think I finally got rid of the weird jump.  Please download latest from GitHub, test, and let me know if you find any issues.  Thanks!

Regards,

Bart

@bsoriano - almost perfect Smile The list is where it should be now. And no need to apologise. No expectation for testing with every addon and scenario - that's our job as users/testers Wink

Only one very minor thing - for larger show titles that spread across two lines, the top is getting slightly cut off and I think the right hand side may be as well (I'm guessing below it should say - 3 episodes available, based on my previous screenshot):

Image

It needs repositioning or the element resizing I think, then all will be exactly perfect. A little nitpick I know, but I guess we may as well get it as good as possible.

Thanks again for all the excellent work.  

@DarrenHill, thanks for the testing and feedback! I did see that issue, I will get to it today. What repo did you get the iplayer addon from? I would like to test using that one, since I see it provides info other video add-ons do not.

Regards,

Bart

EDIT: Never mind the question, I just saw the addon is in the Kodi repo.  I will test with it today.

EDIT 2: Fixed the issue of the title size, already on GitHub.


RE: Amber for Leia - bsoriano - 2019-03-31

(2019-03-31, 02:37)shedrock Wrote:
(2019-03-31, 02:07)bsoriano Wrote:
(2019-03-30, 02:24)shedrock Wrote: Sorry I didn't explain myself correctly. Exactly how it is in that image I attached. That's the banner view I use, and you can see it at the top. I think it's available in other views as well on other skins, but I was just showing you on the view I use.  
@shedrock , while I am not skilled enough to create an entirely new view, what I did is add the next episode info for tv shows in all of Amber's views.  Please download the latest from GitHub and test this, and let me know if you find any issues.  Thanks.

Regards,

Bart 

Hi Bart,

That's exactly what I meant. Thank you so much for that. You just made my wife so happy. Big Grin

Shedrock

All right! Thanks @shedrock ! Big Grin


RE: Amber for Leia - shedrock - 2019-03-31

(2019-03-31, 14:41)bsoriano Wrote:
(2019-03-31, 02:37)shedrock Wrote:
(2019-03-31, 02:07)bsoriano Wrote: @shedrock , while I am not skilled enough to create an entirely new view, what I did is add the next episode info for tv shows in all of Amber's views.  Please download the latest from GitHub and test this, and let me know if you find any issues.  Thanks.

Regards,

Bart 

Hi Bart,

That's exactly what I meant. Thank you so much for that. You just made my wife so happy. Big Grin

Shedrock 

All right! Thanks @shedrock ! Big Grin 
I was just thinking that maybe at some point you can add Plot & Cast information when info is pulled up while the video is playing. I know right now it just displays the plot, but no pressure. I am sure you have more important things to do with the skin. Big Grin

Thanks,

Shedrock


RE: Amber for Leia - bsoriano - 2019-03-31

(2019-03-31, 14:48)shedrock Wrote:
(2019-03-31, 14:41)bsoriano Wrote:
(2019-03-31, 02:37)shedrock Wrote: Hi Bart,

That's exactly what I meant. Thank you so much for that. You just made my wife so happy. Big Grin

Shedrock 

All right! Thanks @shedrock ! Big Grin 
I was just thinking that maybe at some point you can add Plot & Cast information when info is pulled up while the video is playing. I know right now it just displays the plot, but no pressure. I am sure you have more important things to do with the skin. Big Grin

Thanks,

Shedrock

@shedrock, I will keep it in mind! Is there a skin you like that has that from which you can send me a screenshot so I can get an idea of what you are looking for? Thanks.

Regards,

Bart


RE: Amber for Leia - DarrenHill - 2019-03-31

One other possible small bug (which I think has been in there for a while, in the original version too).

I can't seem to change the background image on the TV PVR screen. I can do so on all the others (including the radio PVR one) but not that specific one. I had this problem in the past with older versions, so it's not something you've introduced. But if you can have a quick look there too it would be nice, as it's the only screen on my box with a non-customised background.


RE: Amber for Leia - shedrock - 2019-03-31

(2019-03-31, 14:51)bsoriano Wrote:
(2019-03-31, 14:48)shedrock Wrote:
(2019-03-31, 14:41)bsoriano Wrote: All right! Thanks @shedrock ! Big Grin 
I was just thinking that maybe at some point you can add Plot & Cast information when info is pulled up while the video is playing. I know right now it just displays the plot, but no pressure. I am sure you have more important things to do with the skin. Big Grin

Thanks,

Shedrock       

@shedrock, I will keep it in mind! Is there a skin you like that has that from which you can send me a screenshot so I can get an idea of what you are looking for? Thanks.

Regards,

Bart       

Here is what I am referring to. I think it would be a great addition.

This is on Aeon MQ8.

https://i.postimg.cc/50K4VfcK/screenshot000.png

Note: I also noticed that if you have concert views as a widget, then the square window looks odd. It should be Poster used for this area.

Image

Below is another issue I ran into. Maybe reduce the font size for this area to avoid scrolling. As you can see, the switches are on top of the text.

https://i.postimg.cc/kgWh3wrk/screenshot003.png

Shedrock


RE: Amber for Leia - bsoriano - 2019-03-31

(2019-03-31, 15:48)shedrock Wrote:
(2019-03-31, 14:51)bsoriano Wrote:
(2019-03-31, 14:48)shedrock Wrote: I was just thinking that maybe at some point you can add Plot & Cast information when info is pulled up while the video is playing. I know right now it just displays the plot, but no pressure. I am sure you have more important things to do with the skin. Big Grin

Thanks,

Shedrock       

@shedrock, I will keep it in mind! Is there a skin you like that has that from which you can send me a screenshot so I can get an idea of what you are looking for? Thanks.

Regards,

Bart         

Here is what I am referring to. I think it would be a great addition.

This is on Aeon MQ8.

https://i.postimg.cc/50K4VfcK/screenshot000.png

Note: I also noticed that if you have concert views as a widget, then the square window looks odd. It should be Poster used for this area.

Image

Below is another issue I ran into. Maybe reduce the font size for this area to avoid scrolling. As you can see, the switches are on top of the text.

https://i.postimg.cc/kgWh3wrk/screenshot003.png

Shedrock  
@shedrock , thanks for the feedback.  The font size issue for Addon Settings I just fixed, I will upload to GitHub in a little bit.
EDIT: Already uploaded to Git.


Thanks for the screenshot for the Cast on Info.  I will keep it in mind for a future inclusion, along with an option to disable it for people who don't want that info.

In terms of the music videos, I can understand your concern, since your music videos are concerts that would look better with the poster in the shelf.  What I have found is that most music videos do not have a poster, just a thumb (and maybe fanart), so having square instead of poster is what makes the most sense.  I will not be changing this until I can tackle rewriting the menu with script.skinshortcuts.  When I do that, I will provide art options for all the widgets.  I apologize if I cannot get to this request at this time.

Regards,

Bart


RE: Amber for Leia - bsoriano - 2019-03-31

(2019-03-31, 15:25)DarrenHill Wrote: One other possible small bug (which I think has been in there for a while, in the original version too).

I can't seem to change the background image on the TV PVR screen. I can do so on all the others (including the radio PVR one) but not that specific one. I had this problem in the past with older versions, so it's not something you've introduced. But if you can have a quick look there too it would be nice, as it's the only screen on my box with a non-customised background.
@DarrenHill , thanks for reporting.  I just saw this issue and will look into it today.

Regards,

Bart


RE: Amber for Leia - shedrock - 2019-03-31

(2019-03-31, 17:21)bsoriano Wrote:
(2019-03-31, 15:48)shedrock Wrote:
(2019-03-31, 14:51)bsoriano Wrote: @shedrock, I will keep it in mind! Is there a skin you like that has that from which you can send me a screenshot so I can get an idea of what you are looking for? Thanks.

Regards,

Bart        

Here is what I am referring to. I think it would be a great addition.

This is on Aeon MQ8.

https://i.postimg.cc/50K4VfcK/screenshot000.png

Note: I also noticed that if you have concert views as a widget, then the square window looks odd. It should be Poster used for this area.

Image

Below is another issue I ran into. Maybe reduce the font size for this area to avoid scrolling. As you can see, the switches are on top of the text.

https://i.postimg.cc/kgWh3wrk/screenshot003.png

Shedrock  
@shedrock , thanks for the feedback.  The font size issue for Addon Settings I just fixed, I will upload to GitHub in a little bit.

Thanks for the screenshot for the Cast on Info.  I will keep it in mind for a future inclusion, along with an option to disable it for people who don't want that info.

In terms of the music videos, I can understand your concern, since your music videos are concerts that would look better with the poster in the shelf.  What I have found is that most music videos do not have a poster, just a thumb (and maybe fanart), so having square instead of poster is what makes the most sense.  I will not be changing this until I can tackle rewriting the menu with script.skinshortcuts.  When I do that, I will provide art options for all the widgets.  I apologize if I cannot get to this request at this time.

Regards,

Bart 
No problem at all. I appreciate the work you've done so far, and I look forward to the changes in the future. Big Grin

Shedrock


RE: Amber for Leia - bsoriano - 2019-03-31

(2019-03-31, 17:22)bsoriano Wrote:
(2019-03-31, 15:25)DarrenHill Wrote: One other possible small bug (which I think has been in there for a while, in the original version too).

I can't seem to change the background image on the TV PVR screen. I can do so on all the others (including the radio PVR one) but not that specific one. I had this problem in the past with older versions, so it's not something you've introduced. But if you can have a quick look there too it would be nice, as it's the only screen on my box with a non-customised background.
@DarrenHill , thanks for reporting.  I just saw this issue and will look into it today.

Regards,

Bart 
@DarrenHill , fixed on Git.  Please download, test, and let me know if there are any additional issues.  Thanks.

Regards,

Bart


RE: Amber for Leia - DarrenHill - 2019-03-31

@bsoriano - looks good to me, both issues seem to be fixed perfectly.

No more issues from my side to report Big Grin


RE: Amber for Leia - shedrock - 2019-03-31

@bsoriano - I just realized that the Plot & Cast is in fact already in the skin. The problem is that I am running LibreELEC and there is a problem with it, which is being addressed by the LibreELEC team. It works perfectly with Windows.

Shedrock


RE: Amber for Leia - bsoriano - 2019-04-02

(2019-03-31, 22:09)shedrock Wrote: @bsoriano - I just realized that the Plot & Cast is in fact already in the skin. The problem is that I am running LibreELEC and there is a problem with it, which is being addressed by the LibreELEC team. It works perfectly with Windows.

Shedrock

Thanks for confirming @shedrock !

Regards,

Bart


RE: Amber for Leia - bsoriano - 2019-04-02

Hello all,

Updated version on GitHub.  Only change is that I skinned the Global Search addon.

Some notes:

1. There is only one view available, Icons.  I am not planning on adding any other views to this (design is not my strong suite).
2. Please do not enable "Search Live TV" in the addon settings.  This, in my testing under Windows, makes the addon unstable, in particular if you have a large number of channels.  This option is turned off by default, and, in my opinion, is not generally needed, as PVR in Kodi has a very strong search functionality without needing addons.
3. Remember to turn on "Enable Global Search" in "Settings", "Amber Settings", "Advanced" if you want to use the Global Search addon.

As always, please test and let me know if you find any issues.  Thanks.

Regards,

Bart


RE: Amber for Leia - spudstuff - 2019-04-02

(2019-04-02, 03:06)bsoriano Wrote: 1. There is only one view available, Icons.  I am not planning on adding any other views to this (design is not my strong suite).

Global search looks great @bsoriano !

I thought just a slight transparency would be nice though, like a lot of the other Amber views, like this:

Image

Versus current:

Image

Just adding a colordiffuse to the background images:

xml:

<control type="image" description="Left Menu background">
    <colordiffuse>dfcebe9e</colordiffuse>
    <left>60</left>
    <top>60</top>
    <width>412</width>
    <height>980</height>
    <texture border="45">dialogs/TinyDialog.png</texture>
</control>
<control type="image" description="2/2 Right Menu">
    <colordiffuse>dfcebe9e</colordiffuse>
    <left>462</left>
    <top>60</top>
    <width>1459</width>
    <height>980</height>
    <texture border="45">dialogs/TinyDialog.png</texture>
</control>

What do you think?