1290152 - Add "nearest" support for scrollIntoView on elements

Status: RESOLVED DUPLICATE of bug 1389274

(Core :: DOM: Core & HTML, defect, P3)

Description

[Boris Zbarsky [:bzbarsky]]

https://drafts.csswg.org/cssom-view/#dom-element-scrollintoview has this IDL:

  void scrollIntoView();
  void scrollIntoView((boolean or object) arg
);

and converts the object to a ScrollIntoViewOptions dictionary.  This allows more control over scrolling behavior.

Kip, you did similar stuff for scrolling on Window, iirc.  Are you interested in taking this?

Comment 1

[Neil Deakin]

We already support the object form, but it looks like the spec linked above has some additional options we don't support:

enum ScrollLogicalPosition { "start", "center", "end", "nearest" };

 - we support only start and end

dictionary ScrollIntoViewOptions : ScrollOptions {
  ScrollLogicalPosition block = "center";
  ScrollLogicalPosition inline = "center";
};

 - we support only block

Comment 2

[Boris Zbarsky [:bzbarsky]]

> We already support the object form

Oh, so we do.  I managed to look at the .idl instead of the .webidl.

Anyway, "nearest" is what would be nice here.

Comment 3

[:kip (Kearwood Gilbert)]

(In reply to Boris Zbarsky [:bz] from comment #0)
> https://drafts.csswg.org/cssom-view/#dom-element-scrollintoview has this IDL:
> 
>   void scrollIntoView();
>   void scrollIntoView((boolean or object) arg
> );
> 
> and converts the object to a ScrollIntoViewOptions dictionary.  This allows
> more control over scrolling behavior.
> 
> Kip, you did similar stuff for scrolling on Window, iirc.  Are you
> interested in taking this?

I don't have many cycles outside of the WebVR work currently, but could certainly give support to anyone that is taking this one.

Comment 4

[Hsin-Yi Tsai (she/her) [:hsinyi]]

I am inferring this is a nice but not urgent complement so set P3 for now. Please feel free to modify it if you have a different opinion.

Comment 5

[Eric Shepherd [:sheppy]]

Having "center" available would be nice; to be able to get the relevant content front and center easily would be helpful in several cases.

Comment 6

[Bart Nagel]

To summarize, there are a few things missing here:

- block: 'nearest'
- block: 'center'
- inline option with all of its possible values

Should these be separate bugs, or does this bug cover them all?

Chrome now supports all of these fully, as far as I can tell, as of Chrome 61.

Comment 7

[Thomas Wisniewski]

scrollIntoView support now matches the draft spec, now that bug 1389274 landed. Marking this as a duplicate.