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

On Wed, 23 Jul 2014 05:46:10 +0200, Tab Atkins Jr. <jackalmage@gmail.com>  
wrote:

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

Sounds OK.

-- 
Simon Pieters
Opera Software

Received on Saturday, 26 July 2014 06:44:10 UTC