Skin Editing Tool Alpha 0.2 - NOT SAFE TO USE YET (16 Viewers)

[sloty32]

Hi,

just put my xml trough your batch, and now its not working.

i compared those two files attached, and the only difference are those changes trough the batchfile (ID instead of id and so on)
the original works, the new one not. mp says invalid or missing mykino.xml...

and the error log says:

2009-01-08 23:56:00.578125 [ERROR][MPMain]: No translation found for id 178912
2009-01-08 23:56:01.343750 [ERROR][MPMain]: OnMessage exceptionystem.NullReferenceException: Der Objektverweis wurde nicht auf eine Objektinstanz festgelegt.
bei myKino.myKinnPageLoad()
bei MediaPortal.GUI.Library.GUIWindow.OnMessage(GUIMessage message)


anyone?

edit: an the status line at the bottom of xmlmind says invalid too if i load the new xml. any hints why?

 

[Alltaken]

Hi Sloty,

Thanks for pointing that out. I am curious, if you change the ID tag back to id, if the skin loads fine? (just a find and replace on the <ID> and </ID> tags would do it.

Also XMLmind is saying the file is invalid for a few different reasons, but this doesn't mean its invalid in media portals eyes:

• Document Order
  • Animation should be at the end of the controls (Media portal doesn't impose this, but I do for editing purposes, as it is best to have all animations grouped in one location to avoid duplication (as i have seen in a number of skin files)).
  • <import> tags are expected at the top of the document, directly under the <Controls> title. similarly this is in order to group them for consistency.
• My definitions file being wrong
  • "delay" is currently not supported on the version on the net (so ignore this, as it will be fixed shortly)
  • Anywhere that you have a delay option with a value of 0, maybe consider removing it (or is the default not 0?).

Let me know if you have any luck loading your skin after changing the ID tag.
I have updated the batch script on the assumption that id must be case sensitive.

Cheers

 

[p13man]

Just to show how much I don't know.......

Alltaken:

Animations can exist in a <control> tag, and will animate the button, or group of buttons the animation belongs to. I have run into a few files which include animations as part of the <controlS> tag. Are these legitimate? how do they function differently to animations inside a control?

Apols for such an indirect response but the closest I could get to answering this follows:

All the GUI controls are put between the <controls> and </controls> tag and each one is contained within a <control> tag. Some controls (e.g. group) may allow you to nest subcontrols within them so you may have <control> tags within a parent <control> tag

...but no mention of any tag - other than <control> - being permitted inside the <controls> tag.

Alltaken:

Does anyone know what the minus character "-" does in elements that normally contain a number. I.e. <spinWidth>-</spinWidth>

I've seen it used in several different elements in place of integers, strings and boolean values but it isn't considered valid by XXE : "element contains invalid data: "-" is not one of the allowed values [cvc-enumeration-valid] [cvc-type.3.1.3].

My best guess is that it's used as a shorthand method of clearly identifying an element that is deliberately -as opposed to accidentally - empty.

Finally:
Alltaken:

Does the order in which an <include> tag appears effect anything?

"XML Inclusion is a recursive process, so each xi:include element in an included document is processed too. And of course to keep it safe, circular inclusions are detected and treated as fatal errors. What is a fatal error? XInclude defines two types of errors, which may occur during inclusion process—"fatal error," which as you can suppose refers to the presence of factors that prevent normal processing from continuing; and "resource error," which refers to a failure of an attempt to fetch a resource. Fatal errors are deadly, meaning that processing must be stopped, while resource errors are subject to fallback processing.

"Now how is the xi:include element is processed? First of all, it depends on the parse attribute value. When parse="xml" (which is a default, by the way), the referenced document is fetched, is parsed as XML, and replaces the xi:include element in the source infoset along with its descendants. As you can suppose, an attempt to include non well-formed XML document results in fatal error as well-formedness is true Holy Grail of XML."

....Full article here

I'm not sure if any of this is any use to you but trying to find some relevant info was a useful exercise for me, at least. I'm approaching the point where I know enough about xml to be slightly dangerous!

 

[explodedk]

alltaken it works a bit the same, but with a timer saying when to change from one picture to another. sametime it can randomize the images in a certain folder.

 

[Alltaken]

It has been updated to include multiimage explodedK, let me know if i have missed anything with it.

p13man, cheers for that.

... but it isn't considered valid by XXE...

I control the validity. which is where the real bottle-neck will be (its a moving target)
It will all need to change over time, so i guess i need to get bug reports in, in order to fix things.

Sloty, id in lower-case should be valid now. and should be default.

Cheers.

 

[sloty32]

Hi,

tested a bit with upper and lower case tags. as it seems defaultControl is also giving a non valid inMP.
in streamedMP there are shadow effect on the titles. the color is given with the textColor tag. this is also not working.

maybe they are all casesensitive, or at least some. better to stay all lowercase? makes it maybe easier with existing skin files.

or iam the only one were the uper case versions not work? ondown/onleft and so on, only working if not onLeft/onRight...

testing a bit more. deleted delay, seems doing no harm.

BUT putting the imports at the begining, breaks one of them. I need to put<import>common.facade.video.xml</import> at the end, because ther will be no icons to view. strange.

 

[Alltaken]

Ah, bugger.

It seems they are all incredibly case sensitive, so i'll need to go through and change all the cases to the correct ones.

Oh well.... Back to it. (I will also allow imports to be at any location in the document)

 

[Alltaken]

These tools are more likely to break a skin than help edit one until I get to the bottom of this case sensitivity, and work out what is the correct case for each tag.

Thanks for giving them a go so far. I'll try to get a useful update done as quick as possible, but it may take some time. I'll try to make some bigger upgrades at the same time as getting the cases right, so it should be a slightly different beast next time around.

Cheers.

 

[explodedk]

sounds nice. keep it up

 

[p13man]

Quick question. It takes an age to load a file - 30 secs to 1 minute - is that because it has to get the CSS files etc from your server or might I have got something wrong?