RE: Scroll bar issue in long page of ref case

> From: Gérard Talbot [mailto:css21testsuite@gtalbot.org]
> Sent: Thursday, June 25, 2015 12:12 AM
> To: Xu, JianfengX
> Cc: Public CSS Test suite mailing list
> Subject: Re: Scroll bar issue in long page of ref case
> 
> 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-val
> > ues-b.xht<https://github.com/w3c/csswg-test/blob/master/css-color-3/t4

> > 24-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

Thanks for listing the tests here, as you know in another thread that we are blocked to access them.

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

Indeed, I don't like this either.

> The problem is the length of the test page. The problem is not the ability of
> an UA to generate a vertical scrollbar.

That's the point.
 
> 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.

Thanks for these info. I believe Jianfeng is able to move this forward.

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

Good catch.

Thanks,
Zhiqiang
 
> 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 Thursday, 25 June 2015 02:01:44 UTC