Kodi Community Forum

Full Version: Wackiness with updated addon settings dialog
You're currently viewing a stripped down version of our content. View the full version with proper formatting.
Just installed the latest nightly and made the changes to the addon settings dialog as described in the "Changes to the skinning engine..." thread. https://forum.kodi.tv/showthread.php?tid...pid2603091

Noticed a few strange things that weren't present when using the previous control IDs. I have a scrollbar on the grouplist that is now id 5. It doesn't seem to honor the showonepage=false setting though, and is always displayed. And it doesn't seem to be measuring the correct number of items until it is focused. I tried changing the scrollbar id, but it still had the same issues.

The other thing is that I used to display the addon thumbnail in the settings dialog using the following code:
Code:
$INFO[Window.Property(Addon.ID),special://home/addons/,/icon.png]

This no longer works though, the image isn't displayed. Is this property still available in the addon settings dialog?

Here's my DialogAddonSettings.xml: https://github.com/braz96/skin.omni/comm...e566b0761c

Thanks for any suggestions!
Thanks for your report. Please see PR12314 for a fix.
(2017-06-17, 14:39)Montellese Wrote: [ -> ]Thanks for your report. Please see PR12314 for a fix.
Thanks Montellese, will test when there's a Windows build available!
Watch the test-builds folder for "addon_settings_fallout_3" in about 2 hours
Will the scrollbar issue described above be fixed in the new build? I just realized that the addon settings dialog isn't the only one affected. It seems like scrollbars are not working properly for any grouplist that reuses the same ID, including the one in SettingsCategory.xml. Here's a screenshot from Estuary that illustrates the issue. The list only has a few items, but the scrollbar always shows and can be focused.

I can report on trac or a new thread if this is a bigger issue.

Image
I only fixed tha Addon.ID issue because my knowledge of the GUI library is not good enough to fix the scrollbar issue.
(2017-06-17, 18:49)Montellese Wrote: [ -> ]I only fixed tha Addon.ID issue because my knowledge of the GUI library is not good enough to fix the scrollbar issue.
No problem, bug report submitted on Trac. https://trac.kodi.tv/ticket/17504
Confirm this issue and another strange issue with Skin Settings, size of scrollbar does not match with size of list when scrollbar is not focused :

Before focus, not correct :

Image


When focus, ok :

Image
(2017-06-17, 17:21)Martijn Wrote: [ -> ]Watch the test-builds folder for "addon_settings_fallout_3" in about 2 hours
I couldn't find this test build, but I can confirm that the Addon.ID property is now available in the addon settings dialog when using the latest nightly.
I have confirmed that this isn't the cause of the grouplist problems; it is PR 12213 instead.

https://github.com/xbmc/xbmc/pull/12213#...-312435368
Confirmed fixed with this commit -

https://github.com/xbmc/xbmc/pull/12424