Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test all input types with scrollTo and onscroll #20172

Open
wants to merge 2 commits into
base: master
from

Conversation

@zcorpan
Copy link
Member

zcorpan commented Nov 8, 2019

This test does not follow any spec yet.
Instead it expects all input types to be scrollable
and fire a scroll event in response to scrollTo(),
to more clearly demonstrate which elements do this in
browsers currently.

Variants tested:

  • overflow: visible / hidden / scroll
  • appearance: auto / none

See whatwg/html#4840

This test does not follow any spec yet.
Instead it expects all `input` types to be scrollable
and fire a `scroll` event in response to `scrollTo()`,
to more clearly demonstrate which elements do this in
browsers currently.

Variants tested:

* overflow: visible / hidden / scroll
* appearance: auto / none

See whatwg/html#4840
@zcorpan
Copy link
Member Author

zcorpan commented Nov 8, 2019

I think actually the Firefox result on wpt.fyi matches what whatwg/html#4840 currently requires (if we take the "overflow doesn't apply" + no statement to allow or require a scroll container to mean that scrolling shouldn't be supported).

However, on macOS I get different results in Firefox Nightly: month, week, and datetime-local are also PASS (get a scroll event).

@zcorpan
Copy link
Member Author

zcorpan commented Nov 18, 2019

I've updated the test based on my previous comment. Firefox passes all subtests.

@gsnedders gsnedders closed this Jan 24, 2020
@gsnedders gsnedders deleted the zcorpan/input-element-scrollable branch Jan 24, 2020
@gsnedders gsnedders restored the zcorpan/input-element-scrollable branch Jan 24, 2020
@Hexcles Hexcles reopened this Jan 24, 2020
@zcorpan
Copy link
Member Author

zcorpan commented Jan 27, 2020

@MatsPalmgren can you review this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.