Kodi Community Forum
Release Skin XML Converter + Formatter - Printable Version

Kodi Community Forum (https://forum.kodi.tv)
+-- Forum: Development (https://forum.kodi.tv/forumdisplay.php?fid=32)
+--- Forum: Skinning (https://forum.kodi.tv/forumdisplay.php?fid=12)
+---- Forum: Skin helper addons (https://forum.kodi.tv/forumdisplay.php?fid=300)
---- Thread: Release Skin XML Converter + Formatter (/showthread.php?tid=132720)

Pages: 1 2 3 4 5 6 7 8 9 10 11 12 13


Skin XML Converter + Formatter - phil65 - 2012-05-30

[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)

Image

here´s the download

https://github.com/phil65/xmlconvertgui/tree/master/PublicRelease

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


RE: [RELEASE] Skin XML Converter - phil65 - 2012-06-04

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.


RE: [RELEASE] Skin XML Converter - Jezz_X - 2012-06-04

(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


RE: [RELEASE] Skin XML Converter - Tobby - 2012-06-04

(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 Smile
Haha Big Grin



RE: [RELEASE] Skin XML Converter - Jezz_X - 2012-06-04

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>



RE: [RELEASE] Skin XML Converter - Jezz_X - 2012-06-04

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/blob/master/addon.xml

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



RE: [RELEASE] Skin XML Converter - Hitcher - 2012-06-04

(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/blob/master/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.


RE: [RELEASE] Skin XML Converter - phil65 - 2012-06-04

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.


RE: [RELEASE] Skin XML Converter - phil65 - 2012-06-04

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


RE: [RELEASE] Skin XML Converter - Hitcher - 2012-06-04

(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.


RE: [RELEASE] Skin XML Converter - phil65 - 2012-06-04

(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.


RE: [RELEASE] Skin XML Converter - Jezz_X - 2012-06-05

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


RE: [RELEASE] Skin XML Converter - Martijn - 2012-06-05

(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


RE: [RELEASE] Skin XML Converter - phil65 - 2012-06-05

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


RE: [RELEASE] Skin XML Converter - Jezz_X - 2012-06-05

(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


This forum uses Lukasz Tkacz MyBB addons.