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

Issue 259710043: Improve wording for an NR paragraph

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

Description

Improve wording for an NR paragraph The paragraph from "Avoid (or reduce)" in spacing.itely used 'for example' twice in a row. The description of the 'volta bracket' issue was unclear in terminology and phrasing.

Patch Set 1 #

Total comments: 6

Patch Set 2 : React to comments by Dan and James. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -5 lines) Patch
M Documentation/notation/spacing.itely View 1 1 chunk +5 lines, -5 lines 0 comments Download

Messages

Total messages: 8
simon.albrecht
I came across this rather mishapen wording in NR 4.6.2 – so I changed it.
8 years, 7 months ago (2015-09-02 19:26:52 UTC) #1
Trevor Daniels
LGTM Trevor
8 years, 7 months ago (2015-09-02 20:11:54 UTC) #2
Dan Eble
https://codereview.appspot.com/259710043/diff/1/Documentation/notation/spacing.itely File Documentation/notation/spacing.itely (right): https://codereview.appspot.com/259710043/diff/1/Documentation/notation/spacing.itely#newcode3489 Documentation/notation/spacing.itely:3489: will take up more space than if they are ...
8 years, 7 months ago (2015-09-03 01:28:06 UTC) #3
marc
LGTM, Marc Am 02.09.2015 um 21:26 schrieb simon.albrecht@mail.de: > Reviewers: , > > Message: > ...
8 years, 7 months ago (2015-09-03 04:47:48 UTC) #4
pkx
https://codereview.appspot.com/259710043/diff/1/Documentation/notation/spacing.itely File Documentation/notation/spacing.itely (right): https://codereview.appspot.com/259710043/diff/1/Documentation/notation/spacing.itely#newcode3490 Documentation/notation/spacing.itely:3490: Or dynamics that @q{stick out} of a system can ...
8 years, 7 months ago (2015-09-03 09:27:52 UTC) #5
simon.albrecht
React to comments by Dan and James.
8 years, 7 months ago (2015-09-03 10:18:54 UTC) #6
simon.albrecht
Thanks for the input. https://codereview.appspot.com/259710043/diff/1/Documentation/notation/spacing.itely File Documentation/notation/spacing.itely (right): https://codereview.appspot.com/259710043/diff/1/Documentation/notation/spacing.itely#newcode3489 Documentation/notation/spacing.itely:3489: will take up more space ...
8 years, 7 months ago (2015-09-03 10:20:06 UTC) #7
simon.albrecht
8 years, 7 months ago (2015-09-03 12:30:49 UTC) #8
> It’s plain to me that in this is only one item in the list, with two examples.
I
> think they’re both valuable.

‘that this is only one item in the list’
Sign in to reply to this message.

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