SkinPreLd 1. Fixed a missing brace in the code. 2. Now reports the number of items loaded from skin.dat. SkinSwitch 1. Now has direct call to restart.kmm instead of code in SkinSwitch. 2. Changed wording on menu to reflect current mission. 3. Changed wording in macroinfo statements to reflect change 2. 4. Changed saved skin layout filename by adding "-skin" to the skin name. This was done to allow beter filtering in when the user wishes to select a skin layout file. Previous filter allowed all .ini files where the change will show only *-skin.ini file. Files for other purposes which have the extention ini will not display for selection. If you have current < skin name >.ini files, you can rename them in the form < skin name >-skin.ini. Example below: Klassic.ini becomes Klassic-skin.ini
Quote
Myself
Unless someone finds a problem, I hope these are finished now.
Quote
JamesD
Well, I found a problem. I was reading the wrong file for the default prefs. The file root\defaults\prefile.js is not the one read by KM.
Quote
deadlock
Interesting:
prefskin.js didn't exist in KM 0.7.
prefskin.js was seen in official KM version in 2008.
prefskin.js isn't used by KM 1.6 CVS sources.
prefskin.js isn't referenced in official KM 1.6 package.
...
My best guess is, that it was used as an example
for skin.js creation and should be deleted.
Quote
deadlock
Quote
JamesD
Well, I found a problem. I was reading the wrong file for the default prefs. The file root\defaults\prefile.js is not the one read by KM.
prefile.js is a typo, the used string was:
SkinSwitch_Root_Path + '\\defaults\\prefskin.js' ;
Quote
I had used prefskin.js in the defaults folder for SkinSwitch, but was that right? Just to check I put extra prefs in each file. I put pref("kmeleon.toolband.ExtraPrefSkin.visibility", false); into the prefskin.js file and I put pref("kmeleon.toolband.ExtraSkin.visibility", false); into the skin.js file. I then ran my KM and did an edit on the configuration with a filter of Extra. I only saw kmeleon.toolband.ExtraSkin.visibility. I used the wrong file in SwitchSkin. It was a very simple fix to have SwitchSkin.js read the correct file.