[pcextreme]

Hi! Thank You for this great screensaver. I have been using it for a long time.

Recently I started using the skin Mimic-LR ( https://forum.kodi.tv/forumdisplay.php?fid=218 ), and on that skin my layout looks like this :



On any other skin with the same screensaver settings it looks like this :



I have noticed, that in the folder "addons\screensaver.digitalclock\resources\skins\default\720p\" there is only "skin.mimic.xml" ... but the skin is called "mimic-lr"
also all the changes I make in the screensaver settings page are saved in "userdata\addon_data\skin.mimic.lr\settings.xml" file ... not in "userdata\addon_data\screensaver.digitalclock\settings.xml"

I have contacted the maintaner of the skin and this is the reply I have got :

These are questions that probably should be asked of the screensaver plugin maintainer.

I've installed that screen saver and taken a look.  It appears to keep most of its settings under addon_data/screensaver.digitalclock/settings.xml, (which is proper), but appears to be putting color selections in the skin settings.  Generally, a plugin putting settings anywhere other than it's own plugins settings is frowned upon.  I'm not sure why it's putting the colors in the skin settings.  Perhaps there's an expectation that skinners will want to take over those properties and set the settings for the colors as part of the skin settings (example, logic in the settings which sets a "theme" that sets those colors as well as the normal skin ones)

From the look of the resources/defaults diriectory in the addons, I notice 2 things:
1.  It appears to be written with 720p displays in mind, while most skins are now designed for 1080p displays, so I'm not sure if those scale properly or not.
2.  The vast majority of those skin defaults are for old skins (pre Matrix, and pre Leia) but the code falls back to skin.default.xml as the display if it cannot find one matching the skin.  Presumably, if you just copy the skin.mimic.xml to skin.mimic.lr.xml, it will then pick that up.  (a diff between the mimic one and default.xml only shows the mimic as having a couple font size differences though)

Perhaps there was an expectation that (similar to how a skin can "support" other plugins) that a skin could provide it's own xml for the display, but I do not see anything in the python code that would allow that. it appears to hard-code the plugin path for the xml file.

Could You please take a look at this problem? Thanx!