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

Issue 4641074: Doc: CG remove ugly bars at end of lines in doc (Closed)

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

Description

Doc: CG remove ugly bars at end of lines in doc Caused by 'too long' lines in tely/texi files. Using a variety of methods to break lines that are not normally broken use @smallexample or ' \ ' in the case of long CLI syntax Tracker issue 1691 Some minor text edits to also accommodate the lines lengths. No new information added, some URLs that I couldn't shrink down enough have been edited so that the information is still correct but there are less 'arguments' in the URL itself (i.e. for filtering on google.code.

Patch Set 1 #

Total comments: 11

Patch Set 2 : Second Draft #

Total comments: 6

Patch Set 3 : Third Draft #

Unified diffs Side-by-side diffs Delta from patch set Stats (+87 lines, -75 lines) Patch
M Documentation/contributor/administration.itexi View 3 chunks +5 lines, -3 lines 0 comments Download
M Documentation/contributor/build-notes.itexi View 1 2 15 chunks +55 lines, -50 lines 0 comments Download
M Documentation/contributor/doc-work.itexi View 1 chunk +2 lines, -0 lines 0 comments Download
M Documentation/contributor/introduction.itexi View 2 chunks +3 lines, -3 lines 0 comments Download
M Documentation/contributor/issues.itexi View 2 chunks +8 lines, -8 lines 0 comments Download
M Documentation/contributor/lsr-work.itexi View 3 chunks +4 lines, -4 lines 0 comments Download
M Documentation/contributor/programming-work.itexi View 1 2 1 chunk +2 lines, -3 lines 0 comments Download
M Documentation/contributor/regressions.itexi View 1 chunk +2 lines, -0 lines 0 comments Download
M Documentation/contributor/release-work.itexi View 1 chunk +2 lines, -1 line 0 comments Download
M Documentation/contributor/source-code.itexi View 1 chunk +2 lines, -2 lines 0 comments Download
M Documentation/contributor/website-work.itexi View 2 chunks +2 lines, -1 line 0 comments Download

Messages

Total messages: 6
Graham Percival (old account)
mostly fine, but a few problems that definitely shouldn't be pushed. http://codereview.appspot.com/4641074/diff/1/Documentation/contributor/build-notes.itexi File Documentation/contributor/build-notes.itexi (right): ...
12 years, 10 months ago (2011-06-25 16:12:02 UTC) #1
pkx166h
2nd Draft. Thanks for the comments Graham. http://codereview.appspot.com/4641074/diff/1/Documentation/contributor/build-notes.itexi File Documentation/contributor/build-notes.itexi (right): http://codereview.appspot.com/4641074/diff/1/Documentation/contributor/build-notes.itexi#newcode445 Documentation/contributor/build-notes.itexi:445: LOOP=+$(foreach i, ...
12 years, 10 months ago (2011-06-26 00:02:00 UTC) #2
Graham Percival (old account)
some more little problems. http://codereview.appspot.com/4641074/diff/5001/Documentation/contributor/build-notes.itexi File Documentation/contributor/build-notes.itexi (right): http://codereview.appspot.com/4641074/diff/5001/Documentation/contributor/build-notes.itexi#newcode679 Documentation/contributor/build-notes.itexi:679: $(LYS_TO_TELY) --name=$(outdir)/collated-files.tely \ oops, I ...
12 years, 10 months ago (2011-06-26 11:48:45 UTC) #3
pkx166h
Third Draft - Thanks again Graham. James http://codereview.appspot.com/4641074/diff/5001/Documentation/contributor/build-notes.itexi File Documentation/contributor/build-notes.itexi (right): http://codereview.appspot.com/4641074/diff/5001/Documentation/contributor/build-notes.itexi#newcode679 Documentation/contributor/build-notes.itexi:679: $(LYS_TO_TELY) --name=$(outdir)/collated-files.tely ...
12 years, 10 months ago (2011-06-26 14:05:12 UTC) #4
Graham Percival (old account)
ok, push it.
12 years, 10 months ago (2011-06-26 14:12:47 UTC) #5
pkx166h
12 years, 10 months ago (2011-06-26 14:35:53 UTC) #6

          
Sign in to reply to this message.

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