Beta Amber for Leia with Skinshortcuts
(2019-06-16, 18:30)bsoriano Wrote:
(2019-06-16, 15:35)edjalmo Wrote:
(2019-06-15, 18:30)bsoriano Wrote: @edjalmo , both things done in Git (text ratings moved to the end of the list, ratings bar starts where plot text starts).  Please test and let me know.  Thanks.

Regards,

Bart

Thank you very much! You're great!
I like the tall list and new position/align of ratings/certificates and media flags and option to enabler thumbnail if user like.
(maybe this is too much from my part, but I think the top and bottom of Brazil certificate is not the same (I can't notice this on PC, but in my TV, that's have a more large screen, I can))

The only new thing I haves to suggest in video info is show movie tagline too in plot when it is clicked, and make the info menu (director, writers...) have a infinite scrolling (maybe an option to enable/disaber too) (look how you in and out this section, how you navegate in it and I think you will agree with me).

The "bugs" I found:
  • In some Tv show like Black Mirror, which are anthologies, and there's no character fixed, this space be empty. Suggestion, put banner;
Image
  • When I change the view type of a season or episodes they don't change for the others Tv shows (just change a few). This also happend in add-ons, and I think in all view type;
  • Sometimes this text with next episode show, look
Image
I don't know but this don't show all the time and not with all TV show. What I suggest is show next episode text (if have) after having passed 3 seconds in a Tv show selected (or maybe, again, an option to enabler/disabler this).

Thanks!   
@edjalmo , thanks for continuing to test and use Amber! I think it´s a good idea when there is no cast to show the banner for TV shows, I will implement that shortly.  As for the "infinite scrolling" )a wraplist in skinning), I tried it, but did not like how it looked.  I will try again and see if I can improve how it looks.  If not, it will remain as is.

I don't know what is happening with views not remaining the same for all of your tv shows, and I don't know how I could manage that from the skin side.

In regards to the "Next Episode" text, it shows up if any of your tv shows has a next episode (this is provided by tv show next aired addon).  I don't plan to make this optional.  If the information is wrong for any of your tv shows, please check the settings of the tv show next aired addon (it's a program addon), and make sure you are using the kodi repo version of the addon, not the one from marcelveldt's beta repo.  Thanks.

Regards,

Bart 

@edjalmo , please test the latest from Git.  I centered the certification flag as best as I could, the tv show banner is now displayed (if present) if there is no cast, and the list on the right now has "infinite scrolling" as you call it.  Please let me know if this is what you were looking for.  Thanks.

Regards,

Bart


Messages In This Thread
RE: Amber for Leia with Skinshortcuts - by bsoriano - 2019-06-16, 21:44
Logout Mark Read Team Forum Stats Members Help
Amber for Leia with Skinshortcuts1