- From: Gérard Talbot <css21testsuite@gtalbot.org>
- Date: Wed, 24 Jun 2015 12:12:21 -0400
- To: "Xu, JianfengX" <jianfengx.xu@intel.com>
- Cc: Public CSS Test suite mailing list <public-css-testsuite@w3.org>
Le 2015-06-24 05:16, Xu, JianfengX a écrit :
> Hi
> When I use automated ref test tool to run case
> @https:/github.com/w3c/csswg-test/blob/master/css-color-3/t424-hsl-values-b.xht<https://github.com/w3c/csswg-test/blob/master/css-color-3/t424-hsl-values-b.xht>
> and related ref case,
[src]
http://test.csswg.org/source/css-color-3/t424-hsl-values-b.xht
[src]
http://test.csswg.org/source/css-color-3/t424-hsl-values-b-ref.html
[nightly-unstable]
http://test.csswg.org/suites/css-color-3_dev/nightly-unstable/html4/t424-hsl-values-b.htm
[nightly-unstable]
http://test.csswg.org/suites/css-color-3_dev/nightly-unstable/html4/reference/t424-hsl-values-b-ref.htm
> I meet two issues.
>
> 1) The page is too long to be displayed on the screen, so scroll
> bar show on the page.
>
> On PC, the scroll bar always show.
>
> On android mobile, the scroll bar gradually disappear. But if you
> slide the page, the scroll bar show again.
> When I use tool to get screenshot on android mobile,
> sometimes the screenshot has scroll bar, and sometimes not. This
> different lead to test fail.
> Due to browser which I use is based on webkit, I add a CSS style to
> the hide the scroll bar, then get pass result. The CSS style as below:
> ::-webkit-scrollbar {
> width: 0px;
> height: 0px;
> }
We do not recommend this.
The problem is the length of the test page. The problem is not the
ability of an UA to generate a vertical scrollbar.
We also always want to avoid vendor-prefix in all tests... but that's
another issue here.
>
> 2) Because this page is too long and show scroll bar, which need
> user interaction to finish the testing. It is difficult to get
> screenshot which cover the entire page.
>
> If a test needs user interaction, it cannot be automated by ref test.
> So, shall we rewrite (or split) the test/ref files?
Yes. Split the test and split the ref files.
When we use the Test harness, the iframe containing the test will use
about 500px in height by the width of the window viewport. Anything
taller than 500px will make the user using the Test harness scroll up
and down... which is something we want to avoid.
"
Short
Tests should be as short as possible. For reftests in particular
scrollbars at 800×600px window size must be avoided unless scrolling
behaviour is specifically being tested. For all tests extraneous
elements on the page should be avoided so it is clear what is part of
the test (for a typical testharness test, the only content on the page
will be rendered by the harness itself).
"
http://testthewebforward.org/docs/test-format-guidelines.html#short
--------
JianfengX,
<table border="border">
is not valid HTML5. Besides, it contradicts
line 6: table { border-spacing: 0; padding: 0; border: none; }
You can safely remove the 15 occurences of attribute specification
border="border" in that test.
Gérard
--
Test Format Guidelines
http://testthewebforward.org/docs/test-format-guidelines.html
Test Style Guidelines
http://testthewebforward.org/docs/test-style-guidelines.html
Test Templates
http://testthewebforward.org/docs/test-templates.html
CSS Naming Guidelines
http://testthewebforward.org/docs/css-naming.html
Test Review Checklist
http://testthewebforward.org/docs/review-checklist.html
CSS Metadata
http://testthewebforward.org/docs/css-metadata.html
Received on Wednesday, 24 June 2015 16:12:53 UTC