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

Issue 46280043: Doc: NR Tidy up of A4 - fretboard diagrams (Closed)

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

Description

Doc: NR Tidy up of A4 - fretboard diagrams Removed bar numbers - inappropriate for this section Added an @appendixsubsec for each of the three pre-defined fretboard instruments (Guitar, Ukulele and Mandolin) so as to make it easier to find in the index but also to break up the apparent 'wall' of fretboard diagrams when viewing the Appendix.

Patch Set 1 #

Patch Set 2 : Removed music Staff to just leave the fret boards as this looks clearer. Changed from using @append… #

Patch Set 3 : typo - grr. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+58 lines, -37 lines) Patch
M Documentation/included/display-predefined-fretboards.ly View 1 3 chunks +10 lines, -5 lines 0 comments Download
M Documentation/included/display-predefined-mandolin-fretboards.ly View 1 3 chunks +18 lines, -14 lines 0 comments Download
M Documentation/included/display-predefined-ukulele-fretboards.ly View 1 3 chunks +18 lines, -15 lines 0 comments Download
M Documentation/notation/notation-appendices.itely View 1 2 1 chunk +12 lines, -3 lines 0 comments Download

Messages

Total messages: 4
pkx166h
Removed music Staff to just leave the fret boards as this looks clearer. Changed from ...
10 years, 3 months ago (2013-12-29 20:24:10 UTC) #1
pkx166h
typo - grr.
10 years, 3 months ago (2013-12-29 20:26:33 UTC) #2
Trevor Daniels
LGTM Trevor
10 years, 3 months ago (2013-12-29 21:58:18 UTC) #3
pkx166h
10 years, 3 months ago (2014-01-04 12:31:15 UTC) #4
author	James Lowe <pkx166h@gmail.com>	
	Sun, 29 Dec 2013 05:46:45 +0000 (05:46 +0000)
committer	James Lowe <pkx166h@gmail.com>	
	Sat, 4 Jan 2014 12:22:59 +0000 (12:22 +0000)
commit	442a4b8c3b4d4890292556703aa125d8de92b2d6
Sign in to reply to this message.

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