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

Issue 324420043: NR: Update Clef styles Appendix

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 2 days ago by pkx166h
Modified:
1 week ago
Reviewers:
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

NR: Update Clef styles Appendix Added the full range of Ancient music clefs. Removed the incomplete table from the Ancient music section and gave an @ref to the appendix. Added @cindex entries.

Patch Set 1 #

Patch Set 2 : Fix some TexInfo syntax problems #

Patch Set 3 : Added some @lilypond examples in to ancient.itely #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+576 lines, -113 lines) Patch
M Documentation/notation/ancient.itely View 1 2 2 chunks +49 lines, -109 lines 0 comments Download
M Documentation/notation/notation-appendices.itely View 1 6 chunks +527 lines, -4 lines 2 comments Download

Messages

Total messages: 3
pkx166h
Fix some TexInfo syntax problems
1 week, 2 days ago (2017-09-10 15:35:48 UTC) #1
pkx166h
Added some @lilypond examples in to ancient.itely
1 week, 2 days ago (2017-09-10 16:12:41 UTC) #2
pkx166h
1 week ago (2017-09-12 14:47:05 UTC) #3
Note to self.

https://codereview.appspot.com/324420043/diff/40001/Documentation/notation/no...
File Documentation/notation/notation-appendices.itely (left):

https://codereview.appspot.com/324420043/diff/40001/Documentation/notation/no...
Documentation/notation/notation-appendices.itely:1484:
@lilypond[line-width=3\cm,notime,ragged-right,relative=1]
Note to self - this @item needs to be an @tab

https://codereview.appspot.com/324420043/diff/40001/Documentation/notation/no...
File Documentation/notation/notation-appendices.itely (right):

https://codereview.appspot.com/324420043/diff/40001/Documentation/notation/no...
Documentation/notation/notation-appendices.itely:1542: @code{\clef
varpercussion}
Note to self - this @item needs to be an @tab
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 80a51fa-tainted