-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/midi map #145
Merged
Merged
Feature/midi map #145
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
de4e341
add parameter waiting for midi mapping entry
x37v d463d95
add waitingForMidiMapping to instance and parameter control
x37v e60acf6
add boolean for isMidiMapped
x37v 5036a30
add actions for midi mapping and fix messages update
x37v 7c74c25
plumb in midi/last updates via OSC
x37v c6a8e2c
setup midi mapping in param tab
x37v 077f395
lint fix for param
x37v bd4ea9b
add action to clear param midi mapping
x37v b0c22df
add indication for midi mapping and action to clear it
x37v 9544125
clear waiting before sending midi map
x37v 11bf7ff
fixed Wat type-o
x37v 0819e79
setInstanceWaitingForMidiMappingOnRemote
x37v b769686
better error message for report (midi map)
x37v 860f58c
parsed meta method
x37v 864c221
consolidate param iteration and use getParsedMetaObject
x37v 58ba515
only send meta when we actually clear midi
x37v f9289a3
improved MIDI mapping UI and colorscheme to resemble pieces of Max be…
fde31 59b3a85
instance display: moved tab panels up in component tree to ensure the…
fde31 8fc12d0
allow disabling MIDI mapping mode by hitting Escape key
fde31 77c74f1
disable MIDI mapping when navigating away from params tab or changing…
fde31 58e89d7
fixed parameter overflow layout
fde31 64ee5f9
fixed unused imports
fde31 da9bb46
fixed TS error as typeof [] === "object" = true
fde31 0e8daf9
fix lint
x37v 086863a
parameter focus doesn't need sending params
x37v File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixes the TS build error as
typeof []
is also an array. I guess the issue here is thatAnyJSON
means it could be any kind of value. I wonder if really what we want to ensure is that meta is aRecord<string, AnyJSON>
throughout and if not fall back to{}
. Maybe even the Meta Editor should not allow any non JsonMap input as currently one could also provide[]
which is valid JSONThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have meta that we support but a user might have other ideas for it.. if they use midi mapping, osc mapping, they need to use objects but if they're simply adding meta that they use in their own way, i figure we shouldn't limit them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, sure but the
[param]
object doesn't support it either - in fact doing anything than aRecord<string, AnyJson>
throws. Additionally setting it to eg[]
means that functionality like MIDI mapping won't work so I personally am in favor of enforcing that it needs to be a JSON map at the top level.