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

Release Reduce interactively the number of events in Midi CC lanes v1.0.1 #1281

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rgruet
Copy link

@rgruet rgruet commented Nov 5, 2023

Doc changes

//GetAppVersion(#reaper_version);
//strnicmp(#reaper_version, "6.24", 4) >=0 && APIExists("JS_Window_Find") ? (
(APIExists("JS_Window_Find") && APIExists("JS_Window_AttachTopmostPin")) ? (
JS_Window_AttachTopmostPin(JS_Window_Find(SCRIPT_NAME, true));
Copy link
Member

Choose a reason for hiding this comment

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

Sorry for the long delay. This fails to parse if js_ReaScript API is not installed. This should use the extension_api("JS_Window_...", ...) syntax instead.

If the name of the script is just "Trim CCs", then the filename should be "rig_Trim CCs.eel" and @description should be "Trim CCs" to match.

The documentation mention adding the script to the MIDI Editor section, but the package is configured to add the script to the Main section only. Consider registering the script to the MIDI Editor section or both sections.

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