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.

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

update docs for Panner3d #217

Merged
merged 2 commits into from
Aug 29, 2017
Merged

update docs for Panner3d #217

merged 2 commits into from
Aug 29, 2017

Conversation

therewasaguy
Copy link
Member

  • fix duplicate method yuidoc definitions
  • update p5.Panner3D description

@jvntf this fixes an issue when running grunt yui to generate the docs in p5.js repo

Warning: for Spatializer.positionZ() defined in lib/addons/p5.sound.js:8442, additional overloads should have no description ("<p>Getter and setter methods for orient coordinates</p>\n" !== "") Use --force to continue.

...but I'm not sure of the best way to get sub-properties of the p5.Panner3d.panner (distanceModel, panningModel) to display properly in our reference pages.

@therewasaguy therewasaguy requested a review from jvntf August 21, 2017 00:25
src/panner3d.js Outdated
* Web Audio Spatial Panner Node</a>
*
* Properties include
* - panningModel: "equal power" or "HRTF"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is ok, maybe we should just add links to the Web Audio docs for these properties? distanceModel and panningModel

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great idea, just added links

@therewasaguy therewasaguy merged commit da1d778 into processing:master Aug 29, 2017
@therewasaguy therewasaguy deleted the docs branch August 29, 2017 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants