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

Issue 8622047: Doc: Replace transposition example (3159) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years ago by Trevor Daniels
Modified:
11 years ago
Reviewers:
Graham Percival, Jean-Charles, t.daniels
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Doc: Replace transposition example (3159) New example courtesy of Keith O'Hara

Patch Set 1 #

Patch Set 2 : Fix old chunk mismatch #

Total comments: 2

Patch Set 3 : Respond to Graham's comment #

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

Messages

Total messages: 8
Graham Percival
error: old chunk mismatch. Could you re-upload?
11 years ago (2013-04-21 08:36:25 UTC) #1
Trevor Daniels
Fix old chunk mismatch
11 years ago (2013-04-21 08:49:13 UTC) #2
t.daniels_treda.co.uk
graham@percival-music.ca > error: old chunk mismatch. Could you re-upload? Done; what causes this? Did I ...
11 years ago (2013-04-21 08:52:43 UTC) #3
Jean-Charles
LGTM
11 years ago (2013-04-21 09:09:33 UTC) #4
Graham Percival
https://codereview.appspot.com/8622047/diff/8002/Documentation/notation/pitches.itely File Documentation/notation/pitches.itely (right): https://codereview.appspot.com/8622047/diff/8002/Documentation/notation/pitches.itely#newcode1499 Documentation/notation/pitches.itely:1499: \flute Since the \flute really belongs to the line ...
11 years ago (2013-04-21 09:57:13 UTC) #5
Trevor Daniels
Thanks Graham. https://codereview.appspot.com/8622047/diff/8002/Documentation/notation/pitches.itely File Documentation/notation/pitches.itely (right): https://codereview.appspot.com/8622047/diff/8002/Documentation/notation/pitches.itely#newcode1499 Documentation/notation/pitches.itely:1499: \flute On 2013/04/21 09:57:13, Graham Percival wrote: ...
11 years ago (2013-04-21 12:24:03 UTC) #6
Trevor Daniels
Respond to Graham's comment
11 years ago (2013-04-21 12:25:05 UTC) #7
Trevor Daniels
11 years ago (2013-04-24 07:43:48 UTC) #8
Pushed to staging as
1b718439afd5e4022392183831797532ab4d1f1f
Closing ...
Sign in to reply to this message.

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