Login at Kodi Home

phil65 · (This post was last modified: 2015-05-09, 00:08 by ronie.)

[RELEASE] Skin XML Converter + Formatter 1.0

Thaught it would be best to make a new thread for this...
At the moment the tool can:

- convert between 1080 and 720 resolutions
- convert encoding
- change indenting and format the code
- remove trailing whitespaces
- change EOL
- add headers
- scale animation times/delays (useful to get rid of effectslowdown values <> 1 in addons.xml)
- check for unused and undefined includes
- start a script involving texturepacker to build the skin (thx to JezzX / Hitcher for the batch script)

here´s the download

https://github.com/phil65/xmlconvertgui/...licRelease

and here the according github repo

https://github.com/phil65/xmlconvertgui

texture and font checking not workin properly atm

if you want to use the script easiest way is to clone the repo.
cheers

phil65 · (This post was last modified: 2012-06-04, 02:17 by phil65.)

just as a heads-up for all skinners here.
just checked some skins with the tool, and almost all of them (except confluence) have a fair amount of either unused or undefined includes hidden in the code.

Jezz_X · 2012-06-04, 05:24

(2012-06-04, 02:14)phil65 Wrote: just as a heads-up for all skinners here.
just checked some skins with the tool, and almost all of them (except confluence) have a fair amount of either unused or undefined includes hidden in the code.

hehe I rule Smile

Tobby · 2012-06-04, 06:13

(2012-06-04, 05:24)Jezz_X Wrote:
(2012-06-04, 02:14)phil65 Wrote: just as a heads-up for all skinners here.
just checked some skins with the tool, and almost all of them (except confluence) have a fair amount of either unused or undefined includes hidden in the code.

hehe I rule

Haha

Jezz_X · 2012-06-04, 07:07

SO I gave it a shot and its not real accurate on the checking texutres bit I did a test on jx720 and while the fonts and the includes were right the missing textures where way off it gave me this list

Quote:Building Texture List
Scanning XMLs. This may take a while...
Please check the textures of the upcoming list for usage.
Unused Textures:
blades-size1-left-NF.png
blades-size1-right-NF.png
blades-size2-left-NF.png
blades-size2-right-NF.png
blades-size3-left-NF.png
blades-size3-right-NF.png
check-box.png
check-boxNF.png
dialog-button-focus.png
dialog-button-nofocus.png
Dialog.png
Dialog2.png
FullscreenNowPlayingBack.png
gamecard_home-silver-focus.png
homebutton-0-shadow.png
homebutton-0.png
homebutton-1-shadow.png
homebutton-1.png
homebutton-2-shadow.png
homebutton-2.png
homebutton-3-shadow.png
homebutton-3.png
homebutton-4-shadow.png
homebutton-4.png
homebutton-5-shadow.png
homebutton-5.png
Icon-SystemInfo.png
mediacontrol-pause-focus.png
mediacontrol-pause.png
OSDAudioFO.png
OSDAudioNF.png
OSDBookmarksFO.png
OSDBookmarksNF.png
OSDDvdFO.png
OSDDvdNF.png
OSDForwardFO.png
OSDForwardNF.png
OSDLyricsFO.png
OSDLyricsNF.png
OSDNextTrackFO.png
OSDNextTrackNF.png
OSDPauseFO.png
OSDPauseNF.png
OSDPlayFO.png
OSDPlaylistFO.png
OSDPlaylistNF.png
OSDPlayNF.png
OSDPreFO.png
OSDPreNF.png
OSDPresetSettingsFO.png
OSDPresetSettingsNF.png
OSDPrevTrackFO.png
OSDPrevTrackNF.png
OSDRandomOffFO.png
OSDRandomOffNF.png
OSDRandomOnFO.png
OSDRandomOnNF.png
OSDRecord2.png
OSDRecordFO.png
OSDRecordNF.png
OSDRecordOff.png
OSDRepeatAllFO.png
OSDRepeatAllNF.png
OSDRepeatFO.png
OSDRepeatNF.png
OSDRepeatOneFO.png
OSDRepeatOneNF.png
OSDRewindFO.png
OSDRewindNF.png
OSDStopFO.png
OSDStopNF.png
OSDSubtitlesFO.png
OSDSubtitlesNF.png
OSDVideoFO.png
OSDVideoNF.png
OSDVizFO.png
OSDVizNF.png
OverlayLocked.png
OverlayRAR.png
OverlayUnwatched.png
OverlayZIP.png
slider_nibFO.png
slider_nibNF.png
systemhomebutton-4-shadow.png
systemhomebutton-4.png
systemhomebutton-5-shadow.png
systemhomebutton-5.png
systemhomebutton-6-shadow.png
systemhomebutton-6.png
systemhomebutton-7-shadow.png
systemhomebutton-7.png
systemhomebutton-8-shadow.png
systemhomebutton-8.png
systemhomebutton-9-shadow.png
systemhomebutton-9.png
flagging/aspectratio/1.33.png
flagging/aspectratio/1.66.png
flagging/aspectratio/1.78.png
flagging/aspectratio/1.85.png
flagging/aspectratio/2.20.png
flagging/aspectratio/2.35.png
flagging/audio/0.png
flagging/audio/1.png
flagging/audio/10.png
flagging/audio/2.png
flagging/audio/3.png
flagging/audio/4.png
flagging/audio/5.png
flagging/audio/6.png
flagging/audio/7.png
flagging/audio/8.png
flagging/audio/ac3.png
flagging/audio/aif.png
flagging/audio/aifc.png
flagging/audio/aiff.png
flagging/audio/alac.png
flagging/audio/cdda.png
flagging/audio/dca.png
flagging/audio/dts.png
flagging/audio/dtshd_hra.png
flagging/audio/dtshd_ma.png
flagging/audio/eac3.png
flagging/audio/mp1.png
flagging/audio/mp2.png
flagging/audio/pcm_bluray.png
flagging/audio/truehd.png
flagging/audio/vorbis.png
flagging/audio/wavpack.png
flagging/audio/wmapro.png
flagging/audio/wmav2.png
flagging/video/1080.png
flagging/video/480.png
flagging/video/540.png
flagging/video/576.png
flagging/video/720.png
flagging/video/avc1.png
flagging/video/flv.png
flagging/video/h264.png
flagging/video/mpeg1video.png
flagging/video/mpeg2video.png
flagging/video/vc-1.png
flagging/video/vp8.png
flagging/video/wvc1.png
flagging/video/xvid.png

And Just by doing a search in Notepad++ on a couple of the top ones (in bold above) I new I had I got this

Quote:Search "check-box.png" (1 hits in 1 files)
D:\Jezz_Skin_Git\skin.jx720\720p\defaults.xml (1 hits)
Line 119: <textureradiofocus>check-box.png</textureradiofocus>
Search "blades-size1-left-NF.png" (3 hits in 3 files)
D:\Jezz_Skin_Git\skin.jx720\720p\Common_WindowBuilding.xml (1 hits)
Line 829: <texture>blades-size1-left-NF.png</texture>
D:\Jezz_Skin_Git\skin.jx720\720p\Home.xml (1 hits)
Line 544: <texture>blades-size1-left-NF.png</texture>
D:\Jezz_Skin_Git\skin.jx720\720p\Settings.xml (1 hits)
Line 368: <texture>blades-size1-left-NF.png</texture>

Jezz_X · 2012-06-04, 07:23

It also fails on the ipad touched skin because it doesn't have the folders you assume it does 720p/1080p
Touched uses the new way of having a skin folders which lets you use any folder name you want in this case the folder is named 4x3Hirez https://github.com/xbmc/skin.touched
You can easily do the proper check in your app by using the lines in the addon.xml that define it and gives you all the dimensions to use with you calculations https://github.com/xbmc/skin.touched/blo.../addon.xml

PHP Code:
<res width="1280" height="960" aspect="4:3" default="true" folder="4x3Hirez" /> 

**Hitcher** · 2012-06-04, 11:10

(2012-06-04, 07:23)Jezz_X Wrote: It also fails on the ipad touched skin because it doesn't have the folders you assume it does 720p/1080p
Touched uses the new way of having a skin folders which lets you use any folder name you want in this case the folder is named 4x3Hirez https://github.com/xbmc/skin.touched
You can easily do the proper check in your app by using the lines in the addon.xml that define it and gives you all the dimensions to use with you calculations https://github.com/xbmc/skin.touched/blo.../addon.xml

PHP Code:
<res width="1280" height="960" aspect="4:3" default="true" folder="4x3Hirez" />

+1 for this as I'm not using 720p/1080i folders anymore either.

phil65 · (This post was last modified: 2012-06-04, 12:23 by phil65.)

Texture checking is broken (and probably will never be 100% because of infolabels inside texture tag)
include check works good though.
will adjust the tool to read the folder name from addon.xml.

phil65 · 2012-06-04, 12:21

changed it to use the folder name from addon.xml.

**Hitcher** · (This post was last modified: 2012-06-04, 13:03 by Hitcher.)

(2012-06-04, 12:21)phil65 Wrote: changed it to use the folder name from addon.xml.

Nice one phil, thanks.

EDIT: There seems to be a problem converting from 1080 to 720.

1920 becomes 1267 instead of 1280
1080 becomes 713 instead of 720

EDIT2: That was quick, thanks.

phil65 · 2012-06-04, 13:01

(2012-06-04, 12:47)Hitcher Wrote:
(2012-06-04, 12:21)phil65 Wrote: changed it to use the folder name from addon.xml.

Nice one phil, thanks.

EDIT: There seems to be a problem converting from 1080 to 720.

1920 becomes 1267 instead of 1280
1080 becomes 713 instead of 720

fixed now, thx.

Jezz_X · 2012-06-05, 11:14

So I have to ask when you say fixed do you mean its just the code or you updated the original file without a version bump ? because it still links to the same one

**Martijn** · 2012-06-05, 11:24

(2012-06-05, 11:14)Jezz_X Wrote: So I have to ask when you say fixed do you mean its just the code or you updated the original file without a version bump ? because it still links to the same one

Judging from the github commit date it should be the fix one without a version bump

phil65 · 2012-06-05, 14:12

yep, i´m always pushing the code change + the compiled exe to github so people do not have to compile themselves.

Jezz_X · 2012-06-05, 15:28

(2012-06-05, 14:12)phil65 Wrote: yep, i´m always pushing the code change + the compiled exe to github so people do not have to compile themselves.

btw you do know github has a binary downloads section do you don't have to commit it to your git repo (but I guess putting it in the repo is easier)
https://github.com/phil65/xmlconvertgui/downloads