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.

Sign up for GitHub

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

Jump to bottom

Number of output channels for OscillatorNode not specified #2309

Closed
rtoy opened this issue Mar 19, 2021 · 1 comment
Closed

Number of output channels for OscillatorNode not specified #2309

rtoy opened this issue Mar 19, 2021 · 1 comment
Assignees
@rtoy
Projects
V1

Comments

@rtoy
Copy link
Member

rtoy commented Mar 19, 2021

Describe the issue

This came up on the WebAudio slack channel. The spec doesn't say that the OscillatorNode has a mono output, AFACT. In contrast, the ConstantSourceNode explicitly says the output is mono. We should add a sentence to the OscillatorNode to say the output is always mono, like for the ConstantSourceNode.

This is a minor editorial update; all browsers I know of produce a mono output.
@rtoy rtoy self-assigned this Mar 19, 2021
@rtoy
Copy link
Member Author

rtoy commented Mar 25, 2021

Teleconf: Yes, we should add a sentence for that.

@rtoy rtoy added this to Untriaged in V1 via automation Mar 25, 2021
@rtoy rtoy moved this from Untriaged to Ready for Editing in V1 Mar 25, 2021
rtoy pushed a commit to rtoy/web-audio-api that referenced this issue Mar 25, 2021
Fix WebAudio#2309: OscillatorNode output is mono
75c2bd4 
Add a sentence (like we have for `ConstantSourceNode`) that says the
`OscillatorNode` has a single output that is mono.

Not exactly sure where the best place is to put this sentence, though.
V1 automation moved this from Ready for Editing to Done Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rtoy rtoy

Labels
None yet
Projects
No open projects
V1
  
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rtoy