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

Examples could be modernized #173

Open
marcoscaceres opened this issue Dec 5, 2016 · 5 comments
Open

Examples could be modernized #173

marcoscaceres opened this issue Dec 5, 2016 · 5 comments
Labels
class: editorial https://www.w3.org/policies/process/#correction-classes Needs Discussion The issue needs more discussion before it can be fixed.
Milestone
CR

Comments

@marcoscaceres
Copy link
Contributor

The examples do odd things in strange ways... like use .forEach to map to elements, etc.

They could be much improved.
@cwilso cwilso self-assigned this Jan 5, 2017
@cwilso cwilso added this to the V1 milestone Jan 5, 2017
@toyoshim
Copy link
Contributor

keep this open for update in the future, and may work together with #174

@mjwilson-google
Copy link
Contributor

See also #230

@mjwilson-google mjwilson-google added the class: editorial https://www.w3.org/policies/process/#correction-classes label Sep 13, 2023
@mjwilson-google mjwilson-google added the Needs Discussion The issue needs more discussion before it can be fixed. label Sep 30, 2023
@mjwilson-google
Copy link
Contributor

The examples are now in the explainer. Let's use this issue to track updating them.

@mjwilson-google mjwilson-google modified the milestones: CR, future-work Oct 3, 2023
@mjwilson-google
Copy link
Contributor

There are still .forEach instances in the main body of the spec, so I will move this back to the CR milestone.

@mjwilson-google mjwilson-google modified the milestones: future-work, CR Oct 2, 2024
@mjwilson-google
Copy link
Contributor

Specifically, in sections:
https://webaudio.github.io/web-midi-api/#MIDIInputMap
https://webaudio.github.io/web-midi-api/#MIDIOutputMap

And in the explainer here:
https://github.com/WebAudio/web-midi-api/blob/gh-pages/explainer.md#handling-midi-input

I'm not sure what other style issues we need to resolve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
class: editorial https://www.w3.org/policies/process/#correction-classes Needs Discussion The issue needs more discussion before it can be fixed.
Projects
None yet
Milestone
CR
Development

No branches or pull requests
4 participants
@cwilso @toyoshim @marcoscaceres @mjwilson-google