search

jurialmunkey / skin.arctic.horizon

A simple minimal skin for Kodi
Other
174 stars 49 forks source link

Revert Watched Indicators Colors to Default #428

Open ntilikp opened 3 years ago

ntilikp commented 3 years ago

Hi, silly question but can't seem to revert the watched indicators back to their default color. I found the color code in /colors/defaults.xml but manually entering the value doesn't change the color, and none of the options in the pallet match the color code found in defaults. Is there a way to do this that I've just missed?

drinfernoo commented 3 years ago

Ah, I went to try and figure this out for you, but found that the box to type in a color manually is also broken now 😫 It seems you can type stuff in there but it never actually changes the setting 🤔

I'm testing in Matrix... but if it ever helps, the default color is here: https://github.com/jurialmunkey/skin.arctic.horizon/blob/120c514f0e8c18769f3bd3bb604bf66585bd00e1/colors/defaults.xml#L51

Since it doesn't seem like we can set it manually back to that (currently), you can do it by shutting down Kodi, and editing the settings file at special://userdata/addon_data/skin.arctic.horizon/settings.xml. Somewhere in that file, you'll find the line:

    <setting id="watchedprogresscolor.name" type="string">ffabcdef</setting>

With whichever color you had set. You can either change the color code to the default, or omit it entirely, like so:

    <setting id="watchedprogresscolor.name" type="string"></setting>
ntilikp commented 3 years ago

Thanks for looking, I found the color there as well but encountered the same problem you did trying to manually enter the color code! Just a very minor annoyance, there's definitely other colors close enough that suffice.

drinfernoo commented 3 years ago

I went ahead and edited my previous comment with how you can manually reset it... but you could also simply delete that file (setting all skin settings back to default), or run the setup wizard again from Settings -> Skin Settings -> Lists and other -> Run skin setup wizard.

ntilikp commented 3 years ago

Great! Thanks for the additional info

drinfernoo commented 3 years ago

I'm also noticing that manually typing the color does work for the other color pickers (highlight, gradient, and overlay), but not for indicators.

While we're at it, it would be pretty cool if the chosen color could also be used for the rest of the indicators as well, like these: image and even better if it also worked for corner indicators, like these: image or these: image

drinfernoo commented 3 years ago

I made a PR (#429) to fix the entry box, but that doesn't include the coloring of other indicators, as I requested above 😉