rednir / OsuSkinMixer

Create a skin mix in a couple clicks.
MIT License
124 stars 3 forks source link

Skin.ini not read if capital S. New, incomplete skin.ini is created. #117

Closed tarorororororo closed 11 months ago

tarorororororo commented 11 months ago

When creating or modifying a skin, if the base skin's file is Skin.ini with a capital S, all commands in there will be ignored and a new file, skin.ini with lowercase s, will be created, giving a messed up skin.

I am using arch linux. This issue may be specific to the linux release, I have not tried on Windows.

tarorororororo commented 11 months ago

image

tarorororororo commented 11 months ago

This subsequently doesn't add files that may be included in the skin. For example, if my number assets are in a folder named "num", and the skin.ini's line under Fonts is default: num/default*, it is not parsed, so the newly created skin by the application will not have this num folder.

rednir commented 11 months ago

I dont really have a easy way of testing this right now, but this should be fixed in the latest update. Let me know if otherwise