Re: [cssom-view] Specifying scroll options for scrollIntoView() without the "top" argument

On Tue, Jul 22, 2014 at 6:59 PM, Boris Zbarsky <bzbarsky@mit.edu> wrote:
> On 7/22/14, 9:05 PM, Tab Atkins Jr. wrote:
>> I propose we add a third overload that takes just a ScrollOptions
>> dict
>
> You mean just change the first overload to take an optional (has to be; it's
> a dictionary) ScrollOptions dict, right?

Yeah.

>> If no one's implemented the current ScrollOptions overload yet
>
> Gecko has it implemented, but it hasn't shipped yet.  We can probably change
> it if the changes happen soonish.
>
>> That way you either
>> call the function with the legacy boolean arg, or with the modern dict
>> arg, not both.
>
> This generally makes sense to me.

Cool.  zcorpan, what do you think?  If you're cool with it, we can
pull a CfC on the list and hopefully get it approved and edited within
a week.

~TJ

Received on Wednesday, 23 July 2014 03:46:57 UTC