jeffreycarlson / vast

VAST Tools
3 stars 3 forks source link

Minor correction in VAST_v4.0.xsd #2

Open Alexuy opened 7 years ago

Alexuy commented 7 years ago

From documentation:

  1. Section 3.15.3: HTMLResource element: it hasn't any attributes, but in your xsd schema HTMLResource can have boolean attribute 'xmlEncoded'. May be simplified to simple String.

  2. Section 3.11.1: Icon element: why attributes 'program', 'width', 'height', 'xPosition', 'yPosition' are required?

jeffreycarlson commented 6 years ago

@Alexuy Sorry I missed this issue for some reason.

Regarding 1, great catch. You are correct here.

Regarding 2, those attributes were listed as "required" in the original IAB VAST 3.0 XSD. You are correct that the 4.0 documentation actually lists these as optional.

I'll work on making some updates to these.

jeffreycarlson commented 6 years ago

@Alexuy First issue fixed, I'll bring the second issue up with the VAST working group.