Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(440)

Issue 38420044: Update documentation for MIDI articulations.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by Devon Schudy
Modified:
10 years, 4 months ago
Reviewers:
Trevor Daniels
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Update documentation for MIDI articulations. Also clarify what articulate.ly supports. Change the snippet for redefining a shorthand to use an ArticulationEvent instead of an articulation name, so users who follow its example will have their articulations work in MIDI.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Move @ref to end of line; mention glissandi. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -12 lines) Patch
M Documentation/changes.tely View 1 chunk +10 lines, -0 lines 0 comments Download
M Documentation/notation/input.itely View 1 2 chunks +11 lines, -8 lines 0 comments Download
M Documentation/snippets/modifying-default-values-for-articulation-shorthand-notation.ly View 2 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 2
Trevor Daniels
LGTM, apart from a slight rewording. https://codereview.appspot.com/38420044/diff/1/Documentation/notation/input.itely File Documentation/notation/input.itely (right): https://codereview.appspot.com/38420044/diff/1/Documentation/notation/input.itely#newcode2891 Documentation/notation/input.itely:2891: @ref{The Articulate script} ...
10 years, 4 months ago (2013-12-16 11:57:33 UTC) #1
Devon Schudy
10 years, 4 months ago (2013-12-17 15:34:27 UTC) #2
Move @ref to end of line; mention glissandi.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b