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

Issue 13300048: Docs: new defaults for rehearsal mark alignment (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 7 months ago by Keith
Modified:
10 years, 5 months ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Docs: new defaults for rehearsal mark alignment Documentation/changes.tely | 30 +++++++++ ...aligning-marks-with-various-notation-objects.ly | 65 ++++++++++++++++++++ 2 files changed, 95 insertions(+), 0 deletions(-) makelsr ...aligning-marks-with-various-notation-objects.ly | 44 ++++++++----------- 1 files changed, 19 insertions(+), 25 deletions(-)

Patch Set 1 #

Total comments: 2

Patch Set 2 : relative 3c''{} #

Patch Set 3 : fix error in NR 5.5.1 #

Total comments: 4

Patch Set 4 : whitespace #

Unified diffs Side-by-side diffs Delta from patch set Stats (+34 lines, -2 lines) Patch
M Documentation/changes.tely View 1 2 3 1 chunk +32 lines, -0 lines 0 comments Download
M Documentation/notation/changing-defaults.itely View 1 2 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
pkx166h
fails make doc https://codereview.appspot.com/13300048/diff/1/Documentation/changes.tely File Documentation/changes.tely (right): https://codereview.appspot.com/13300048/diff/1/Documentation/changes.tely#newcode92 Documentation/changes.tely:92: @end lilypond This @lilypond breaks make ...
10 years, 7 months ago (2013-09-10 21:51:21 UTC) #1
thomasmorley651
typo LGTM https://codereview.appspot.com/13300048/diff/1/Documentation/changes.tely File Documentation/changes.tely (right): https://codereview.appspot.com/13300048/diff/1/Documentation/changes.tely#newcode66 Documentation/changes.tely:66: the allowance of horizontal sapce for tempo ...
10 years, 7 months ago (2013-09-10 21:58:46 UTC) #2
Mark Polesky
https://codereview.appspot.com/13300048/diff/9001/Documentation/changes.tely File Documentation/changes.tely (right): https://codereview.appspot.com/13300048/diff/9001/Documentation/changes.tely#newcode86 Documentation/changes.tely:86: of the clef and key signature by default. As ...
10 years, 7 months ago (2013-09-18 23:08:32 UTC) #3
Keith
10 years, 7 months ago (2013-09-19 05:25:18 UTC) #4
https://codereview.appspot.com/13300048/diff/9001/Documentation/changes.tely
File Documentation/changes.tely (right):

https://codereview.appspot.com/13300048/diff/9001/Documentation/changes.tely#...
Documentation/changes.tely:86: of the clef and key signature by default. As in
previous versions, the
On 2013/09/18 23:08:33, Mark Polesky wrote:
> 2 spaces after sentence in texinfo.

Done.

https://codereview.appspot.com/13300048/diff/9001/Documentation/changes.tely#...
Documentation/changes.tely:88: @lilypond[quote,relative=2]
On 2013/09/18 23:08:33, Mark Polesky wrote:
> I think we usually put an empty line before @lilypond, though I don't see it
> mentioned in the CG.

Done.
Sign in to reply to this message.

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