Closed bernd-wechner closed 3 years ago
@bernd-wechner , thanks for bringing this to my attention. There is indeed an issue with some of the power menu options in Amber for Matrix, including Quit.
I have already uploaded the fix to GitHub. If after downloading the latest from the master branch you are still experiencing the issue, unfortunately, you will have to delete the Amber-related skinshortcuts settings. Go to userdata\addon_data\script.skinshortcuts and delete anything that has amber in the file name.
That was fast, and must have been easy, I'll check the commit out to learn something about it too.
Wow, that was a trivial fix. Out of interest, what am I to make of the fork tree on this skin. I ask because maintenance of this sort of stuff gets confusing over time and yours is a fork of @jester-xbmc jester-xbmc's repo which is a fork of pecinko's repo abd I'm wondering which one trickles into the formal Kodi skins repo. pecinko seems to have fallen inactive 2 years ago, jester too, and you have a a lot of peer forks. But I'm reading yours as the Matrix fork and as active. If so, over time, I might like to add some features to Amber and I'm big on offering PR's up the food chain so to speak to the point where it enters the productions stream (Kodi's skin repo) and isn't just a pet project among a gazillion of them that dies in time due to divergence from the live branch and neglect.
@bernd-wechner , I am the official maintainer for Amber. At some point soon I will be updating the Kodi repo with the latest versions for Leia and Matrix. I welcome any PRs you might want to submit. After all, that's what Kodi is about, community :)
Great. I think it might be an idea some time to clean up the repo fork tree then for clarity. What some folk do is just start a whole new repo with anew name. I often see 2 or ng used or a new relase tag.
Something kike skin.amber2 or skin.amber.ng or skin.amber.matrix, just some thoughts. I can even remain skin.amber but if created fresh not as a form, and with a README stating this is the currently maintained repo that pushes to Kodi skin repos, then the fork tree would look cleaner,
My main interest in tweaking Amber is rearranging the Fullscreen playback a bit so it renders better on our TV ;-). I imagine it would be activated with a setting for people who need larger fonts on Fullscreen (are running it on a TV jukebox and can't read the writing from standard spots in the room ;-). But some layout tweaks to suit our Jukebox use case I have in mind too.
This I suspect has been a bug for a long time. I thought it a Kodi bug but it's never been so much of a problem to bother reporting as my HTPC runs 24/7 anyhow and Kodi runs on it.
But upgraded to Matrix recently and ended up with the default skin for a while. Was pleased that Quit worked. Then installed Amber fro Matrix and suddenly it doesn't work again. It locks up Kodi, to be sure, but does not quit. It just locks it up and I need to kill it forcefully (kill -9). I attach a debbug log taken as follows:
kodi (power-quit - amber skin).log
And system details are: