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

Issue 240810043: Some updates to the ancient music section (Closed)

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

Description

Spelling correction. Remove the description of non-ancient rests since it is no longer exhaustive and duplicates other parts of the NR. Grammar. Remove out-commented example. Edit ligature example to get rid of ledger lines, which are not appropriate with mensural music.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -34 lines) Patch
M Documentation/notation/ancient.itely View 8 chunks +17 lines, -34 lines 0 comments Download

Messages

Total messages: 3
PhilEHolmes
Please review.
8 years, 12 months ago (2015-05-19 10:17:00 UTC) #1
mail_philholmes.net
----- Original Message ----- From: ""Jürgen Reuter"" <reuter_j@web.de> To: ""Jürgen Reuter"" <reuter_j@web.de>; <PhilEHolmes@googlemail.com>; <tdanielsmusic@googlemail.com>; <lilypond-devel@gnu.org>; ...
8 years, 12 months ago (2015-05-19 13:01:41 UTC) #2
Trevor Daniels
8 years, 12 months ago (2015-05-19 20:17:10 UTC) #3
LGTM

Trevor
Sign in to reply to this message.

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