jeroenpardon / skin.grid

Grid, a UI for Kodi. By using this code you agree with the license terms as included.
http://www.gridskin.net
Other
39 stars 25 forks source link

Vertical widget focus 2.3.1 #166

Closed snoopyjoe closed 6 years ago

snoopyjoe commented 6 years ago

The screen height on the vertical widget screen is not registering as 1080. This was causing the widget focus to be off by one line. So when scrolling down the highlighted widget was not the visable widget on the screen. The $PARAM[Height] at line 281 in Includes_Home.xml is not getting the height param for some reason. I used the screen_Height variable, which is 1080, instead and that resolved the issue. I'm running LE9 alpha with Kodi 18. Also as seen in the last picture, when a video ask to resume from a last resume time, the dialog focus is off screen.

I'm not good with XML and skins so creating a fork for one line of code wasn't necessary. Python is my expertise.

img_20180306_021037 img_20180306_021047 img_20180206_004920

Checklist:

jeroenpardon commented 6 years ago

I have a fix lined up for this, bear with me

jeroenpardon commented 6 years ago

fix in 3cabb45cd9ba42b9c7d47c8984cdef191f07a741