Android Movie quality info inside player
#16
(2016-01-31, 22:25)mjbxx Wrote: Can you provide a screenshot of the OSD showing the Info button?

Sure, with info turned off

http://1drv.ms/20ACjN9

and turned on

http://1drv.ms/20ACoQM
Reply
#17
Looks great. One last request on this. Is it possible to center each row above the video control row? Example below.

Requested:
Image

Existing:
Image
Reply
#18
(2016-01-31, 23:44)mjbxx Wrote: Looks great. One last request on this. Is it possible to center each row above the video control row? Example below.

Requested:
Image

Existing:
Image

I think it would look better too.
Reply
#19
(2016-02-01, 06:26)Iaai Wrote: I think it would look better too.

Personally I would prefer them to be right aligned for consistency as the other text is left aligned. The containers are right aligned but I think they may have additional padding which is putting them out of line with each other. I'll see if I can do anything with it.
Reply
#20
(2016-02-01, 10:47)joethepartylion Wrote: Personally I would prefer them to be right aligned for consistency as the other text is left aligned. The containers are right aligned but I think they may have additional padding which is putting them out of line with each other. I'll see if I can do anything with it.
I have figured out how to center align the two rows above the video control row and it looks awesome. Hoping you will add these two lines of code changes to enable this center alignment of rows for the Amber video OSD.

VideoFullScreen.xml - <control type="grouplist" id="1"> - Image
Link → VideoFullScreen.xml

VideoOSD.xml - <control type="grouplist" id="100"> - Image
Link → VideoOSD.xml

Image

Image
Reply
#21
So guys I've been thinking, what if it looked something like this:

Image

I think it would look much cleaner, what do you think?
Reply
#22
(2016-02-14, 09:56)Iaai Wrote: So guys I've been thinking, what if it looked something like this:

Image

I think it would look much cleaner, what do you think?

I agree, if possible, do a pull request here:

https://github.com/jester-xbmc/skin.amber

Thanks
Reply
#23
(2016-02-18, 00:36)Jester Wrote:
(2016-02-14, 09:56)Iaai Wrote: So guys I've been thinking, what if it looked something like this:

Image

I think it would look much cleaner, what do you think?

I agree, if possible, do a pull request here:

https://github.com/jester-xbmc/skin.amber

Thanks

Whilst it does look better in that situation there are a couple of issues I can see with that layout.

1. The title line needs to be the longest text field yet the player controls are much longer than those shown on this screen in PVR as there are also channel up and down, EPG and Guide buttons.
2. In movie, PVR and TV show there is additional info shown above and below the title, e.g next show in PVR, director and year for movies etc.

Happy to help (once I've moved house this weekend) if you can explain what things you would like to see, problems you have.
Reply
#24
(2016-02-18, 00:36)Jester Wrote: I agree, if possible, do a pull request here:

https://github.com/jester-xbmc/skin.amber

Thanks

A Pull request for this has been added to the jester-xbmc/skin.amber in github.
Reply
#25
(2016-02-18, 00:57)joethepartylion Wrote: Whilst it does look better in that situation there are a couple of issues I can see with that layout.

1. The title line needs to be the longest text field yet the player controls are much longer than those shown on this screen in PVR as there are also channel up and down, EPG and Guide buttons.
2. In movie, PVR and TV show there is additional info shown above and below the title, e.g next show in PVR, director and year for movies etc.

Happy to help (once I've moved house this weekend) if you can explain what things you would like to see, problems you have.
Then a good compromise would be to leave it the way it is now and and go with the center alignment proposals outlined in Post# 20. In all honesty, it will look great either way you decide to proceed.
Reply

Logout Mark Read Team Forum Stats Members Help
Movie quality info inside player0