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

Document pianoroll #784

Merged
merged 5 commits into from Nov 9, 2023
Merged

Conversation

ilesinge
Copy link
Contributor

@ilesinge ilesinge commented Nov 5, 2023

Add pianoroll, color and label function documentation for inline usage (reference tab or autocomplete)

Object key parameters were documented as classic parameters, for ease of formatting

* @param {integer} cycles defaults to 4 - number of cycles to be displayed at the same time
* @param {number} playhead 0 to 1, defaults to 0.5 - location of the active notes on the time axis
* @param {integer} vertical 0 (default) or 1 - displays the roll vertically
* @param {boolean} labels false (default) or true - displays labels on individual notes (see the label function)
Copy link
Collaborator

Choose a reason for hiding this comment

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

all booleans can either be 0 | 1 or false | true

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I guess we can document them all as boolean : 1/0 ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

looks good

@ilesinge
Copy link
Contributor Author

ilesinge commented Nov 5, 2023

@jarmitage would you like to review?

@felixroos felixroos merged commit 9dbd073 into tidalcycles:main Nov 9, 2023
1 check passed
@ilesinge ilesinge deleted the document_pianoroll branch November 17, 2023 14:23
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.

None yet

2 participants