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

Issue 5489118: Doc: NR improve 'visibility' of 'q' operation (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 4 months ago by pkx166h
Modified:
12 years, 3 months ago
Reviewers:
Visibility:
Public.

Description

Doc: NR improve 'visibility' of 'q' operation Improve visibility of q (repeated chord) operation within NR a few users have noted that this has been hard to find. Added some @cindex and an additional paragraph to repeats.itely. Edited/added some @seealso and @cindex to frettd-strings.itely and simultaneous.itely. Minor blank line space deletions.

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix Typo - thanks Graham #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -24 lines) Patch
M Documentation/notation/fretted-strings.itely View 5 chunks +6 lines, -8 lines 0 comments Download
M Documentation/notation/repeats.itely View 1 5 chunks +16 lines, -16 lines 0 comments Download
M Documentation/notation/simultaneous.itely View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 3
Graham Percival
typo http://codereview.appspot.com/5489118/diff/1/Documentation/notation/repeats.itely File Documentation/notation/repeats.itely (right): http://codereview.appspot.com/5489118/diff/1/Documentation/notation/repeats.itely#newcode548 Documentation/notation/repeats.itely:548: @seealso there shouldn't be two @seealso here.
12 years, 4 months ago (2011-12-25 16:45:44 UTC) #1
Graham Percival
LGTM
12 years, 4 months ago (2011-12-30 07:37:32 UTC) #2
pkx166h
12 years, 3 months ago (2012-01-04 07:55:07 UTC) #3
Wed, 4 Jan 2012 07:52:39 +0000 (07:52 +0000)
commit	4269f54b663421ca2c5c2c57baa5d9c49709d6d2

James
Sign in to reply to this message.

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