[Bug 17334] (setTargetValueAtTime): AudioParam.setTargetValueAtTime is misleading and underdefined

https://www.w3.org/Bugs/Public/show_bug.cgi?id=17334

Olivier Thereaux <olivier.thereaux@bbc.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |ASSIGNED
         AssignedTo|dave.null@w3.org            |crogers@google.com

--- Comment #7 from Olivier Thereaux <olivier.thereaux@bbc.co.uk> 2012-08-23 10:01:37 UTC ---
We had a lengthy conversation about this during the group's teleconference on
22nd August 2012:
http://www.w3.org/2012/08/22-audio-minutes#item03

There was agreement that the name was imperfect, but there was a compromise to
be reached between using a nice, short method name and one that describes the
behaviour of the method well. We discussed whether to remove the "at time",
whether to remove "set", and so on.  

In the end, we reached the following proposed resolution:

* Change the setTargetValueAtTime method to setTargetAtTime
* The rest of the methods are a decent enough compromise, will be kept as is

Related changes were also proposed, which would make the description and
behaviour of AudioParam automation easier to understand and use:

* Use startTime / endTime parameter names for AudioParam automation methods
https://www.w3.org/Bugs/Public/show_bug.cgi?id=18661

* Setting audioparam value while there is an automation curve will cancel that
automation curve and set value immediately
https://www.w3.org/Bugs/Public/show_bug.cgi?id=18662

* Need a method to get a readonly reading of the combined value when using
AudioParam automation curve
https://www.w3.org/Bugs/Public/show_bug.cgi?id=18663

-- 
Configure bugmail: https://www.w3.org/Bugs/Public/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Received on Thursday, 23 August 2012 10:01:51 UTC