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

Issue 5783073: NR: General tidy up - @seealso @knownissue spacing (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by pkx166h
Modified:
12 years, 1 month ago
Reviewers:
Graham Percival
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

NR: General tidy up - @seealso @knownissue spacing No new information added. Trying to make @seealso sections follow CG policy including line breaks for consistent visual layout in *.itely files. Notation Reference only: Made sure all existing @seealso entries were i. In the correct order as per the CG ii. Each @seealso was labelled correctly (i.e. 'Extending LilyPond' instead of just 'Extending' - as per CG). iii. Correct punctuation marks at the end of each line entry iv. Made line breaks between each section consistent (one line space after main sections). Kept one line space between @seealso and @knownissues. Made sure two line breaks before next @node. Any @cindex have line space either side of entry. Removed line space between @seealso and @knownissues and their respective text entries. v. Removed @c sections that contained empty @seealso entries that were just 'placeholders' for the 'future' - as this isn't standard practice.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+117 lines, -359 lines) Patch
M Documentation/notation/ancient.itely View 19 chunks +6 lines, -20 lines 0 comments Download
M Documentation/notation/changing-defaults.itely View 16 chunks +16 lines, -29 lines 0 comments Download
M Documentation/notation/chords.itely View 19 chunks +2 lines, -60 lines 0 comments Download
M Documentation/notation/editorial.itely View 13 chunks +0 lines, -15 lines 0 comments Download
M Documentation/notation/expressive.itely View 13 chunks +3 lines, -18 lines 0 comments Download
M Documentation/notation/fretted-strings.itely View 11 chunks +6 lines, -11 lines 0 comments Download
M Documentation/notation/input.itely View 8 chunks +8 lines, -8 lines 0 comments Download
M Documentation/notation/keyboards.itely View 8 chunks +3 lines, -5 lines 0 comments Download
M Documentation/notation/notation-appendices.itely View 3 chunks +2 lines, -1 line 0 comments Download
M Documentation/notation/percussion.itely View 6 chunks +2 lines, -9 lines 0 comments Download
M Documentation/notation/pitches.itely View 20 chunks +1 line, -24 lines 0 comments Download
M Documentation/notation/repeats.itely View 3 chunks +1 line, -3 lines 0 comments Download
M Documentation/notation/rhythms.itely View 40 chunks +11 lines, -40 lines 0 comments Download
M Documentation/notation/simultaneous.itely View 11 chunks +6 lines, -10 lines 0 comments Download
M Documentation/notation/spacing.itely View 27 chunks +8 lines, -38 lines 0 comments Download
M Documentation/notation/staff.itely View 13 chunks +5 lines, -15 lines 0 comments Download
M Documentation/notation/text.itely View 16 chunks +22 lines, -38 lines 0 comments Download
M Documentation/notation/unfretted-strings.itely View 3 chunks +0 lines, -3 lines 0 comments Download
M Documentation/notation/vocal.itely View 13 chunks +10 lines, -6 lines 0 comments Download
M Documentation/notation/wind.itely View 5 chunks +5 lines, -6 lines 0 comments Download

Messages

Total messages: 2
Graham Percival
LGTM, please push directly to staging.
12 years, 1 month ago (2012-03-11 19:20:53 UTC) #1
pkx166h
12 years, 1 month ago (2012-03-11 20:20:20 UTC) #2
On 2012/03/11 19:20:53, Graham Percival wrote:
> LGTM, please push directly to staging.

committer	James Lowe <pkx166h@gmail.com>	
Sun, 11 Mar 2012 20:16:43 +0000 (20:16 +0000)
commit	80340bdefc128536c79733d8c6f3467d91b01f72

Thanks,
Sign in to reply to this message.

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