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

Issue 90270044: Doc: Clarify grace note synchronisation workaround

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

Description

Doc: Clarify grace note synchronisation workaround As pointed out in the thread http://lists.gnu.org/archive/html/lilypond-user/2014-04/msg00540.html it is necessary to use \grace for the old grace note sync issue. This commit makes that explicit in the known issues section in the NR

Patch Set 1 #

Total comments: 2

Patch Set 2 : Texinfo syntax fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M Documentation/notation/rhythms.itely View 1 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 4
uliska
9 years, 11 months ago (2014-04-22 18:56:32 UTC) #1
Trevor Daniels
A comment on Texinfo syntax; otherwise LGTM. Trevor https://codereview.appspot.com/90270044/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): https://codereview.appspot.com/90270044/diff/1/Documentation/notation/rhythms.itely#newcode3514 Documentation/notation/rhythms.itely:3514: Please ...
9 years, 11 months ago (2014-04-22 19:44:52 UTC) #2
uliska
Texinfo syntax fix
9 years, 11 months ago (2014-04-22 19:47:10 UTC) #3
uliska
9 years, 11 months ago (2014-04-22 19:48:13 UTC) #4
https://codereview.appspot.com/90270044/diff/1/Documentation/notation/rhythms...
File Documentation/notation/rhythms.itely (right):

https://codereview.appspot.com/90270044/diff/1/Documentation/notation/rhythms...
Documentation/notation/rhythms.itely:3514: Please make sure that you use the
@cmd{\grace} command for the
On 2014/04/22 19:44:52, Trevor Daniels wrote:
> The convention in the docs is to use @code
> for LilyPond commands.

Done.
Sign in to reply to this message.

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