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

Issue 344070043: Doc: 2.4.2, fix harmonic-mixed example

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 3 weeks ago by lilypond-pkx
Modified:
2 months, 3 weeks ago
Reviewers:
thomasmorley651
Visibility:
Public.

Description

Doc: 2.4.2, fix harmonic-mixed example Issue 5404 Remove 'Staff', simplify braces and change note value to give a clearer example.

Patch Set 1 #

Total comments: 1

Patch Set 2 : change d to d' #

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

Messages

Total messages: 2
thomasmorley651
One nit, otherwise LGTM https://codereview.appspot.com/344070043/diff/1/Documentation/notation/fretted-strings.itely File Documentation/notation/fretted-strings.itely (right): https://codereview.appspot.com/344070043/diff/1/Documentation/notation/fretted-strings.itely#newcode1802 Documentation/notation/fretted-strings.itely:1802: d8^\markup { \italic \fontsize #-2 ...
2 months, 3 weeks ago (2018-08-26 13:22:21 UTC) #1
lilypond-pkx
2 months, 3 weeks ago (2018-08-26 14:09:45 UTC) #2
change d to d'
Sign in to reply to this message.

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