- From: Niklas Hagelroth <niklas.hagelroth@ikivo.com>
- Date: Fri, 19 Sep 2008 10:58:13 +0200
- To: <public-svg-wg@w3.org>
- Message-ID: <234EB4699C751A4A95DF4FD8D041BBFDF8D95E@SESTHSRV10.zoomon.local>
Hello SVG-WG, We have found some invalid tests in the latest test suite. This is our comments. animate-elem-86-t.svg According to the SMIL Spec (The normative part at the end of section 3.3.5 The animation effect function F(t,u)) the freeze time will be effectively the same as the AD (active duration). Since the end attribute is 2 the AD will also be 2. The discrete animation will change its value exactly at 2s which will put the rectangle in its left position (see, 3.5.2 Specifying the simple animation function f(t)). udom-svg-205-t.svg Line 51 should read NAV_PREV. interact-focus-201-t.svg The two focusable <rect> elements are before the text element pointed out by the svg elements nav-next attribute. Thus, the text element will receive focus instead of the rectangles when focus moves "next". 13.13.1 Navigation behavior. media-anim-201-t.svg The test assumes that the focusable red circle should have focus from the start and never loose it, this is not the behavior specified in the spec, see section 13.13.1, it states that the SVG User Agent should initially receive focus and when going from a focused object in the SVG tree focus should be offered to the SVG User Agent. struct-class-201-t.svg On line 71 in the test file there is an attempt to use "nodeValue", this is not a part of the uDOM, it is from the full spec. media-audio-212-t.svg Problems with audio and display=none and visibility=hidden. The spec says that visibility and display is not applicable to the audio element, hence it should not be affected. The test assumes that it should be muted when display=none or visibility=hidden. media-audio-213-t.svg Problems with audio and display=none and visibility=hidden. The spec says that visibility and display is not applicable to the audio element, hence it should not be affected. The test assumes that it should be muted when display=none or visibility=hidden. struct-frag-02-t.svg The viewbox="100 100 100 100" and preserveAspectRatio="none" has been removed from the SVG element, this makes the test invalid. struct-frag-03-t.svg The viewbox="100 100 100 100" and preserveAspectRatio="xMidYMid" has been removed from the SVG element, this makes the test invalid. interact-event-204-t.svg SVG as child element defined, otherwise OK. media-alevel-201-t.svg According to 13.12.1 only renderable elements are required to accept focus. The test expects that audio elements can be focusable. media-alevel-202-t.svg According to 13.12.1 only renderable elements are required to accept focus. The test expects that audio elements can be focusable. interact-focus-208-t.svg The description does not match the expected behavior. Expected behavior: Text should turn green in the following order when focusing "next": "Old", "pond", ("frogs" and "in"), "jumped", "sound", "of", "water". When "jumped" gets focus the test states: > "jumped" must turn green when given focus and in addition "frogs" and > "in" must turn green at the same time due to focus bubbling. This is not true due to the order of focusin & focusout animations. Since both will be activated at the same time it is the latter animation that will have precedence. So "frogs" and "in" will be fill="inherit" and not fill="green". udom-svg-228-t.svg All of the navigation attributes should have a "url()" around the "#t1" etc references. struct-use-207-t.svg There is a small error in the used file "svgREF13.svg", line 19, the color "ghostwhite" is not defined in tiny. udom-svg-220-t.svg Error in script, line 84, 88 and 89 uses a "set" incorrect; changing it to use a list as on lines 78-80 makes it correct. coords-constr-202-t.svg Animation element is not child of svg root when trying to animate viewBox. It should be moved to be a direct child of the svg root element. udom-svgcolor-201-t.svg Section A8.5 in the specification is unclear when discussing the components of the SVGRGBColor object. The test assumes that the components can be set, our interpretation is that they can _not_ be set using the syntax as in the test file, ie color.red = 128. interact-event-201-t.svg Minor error in the test; the rect has a nav-index attribute which is not used in Tiny. Regards, Niklas
Received on Friday, 19 September 2008 08:59:04 UTC