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

Issue 4172047: Add dots to tocItemMarkup (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 2 months ago by Graham Percival (old account)
Modified:
13 years, 2 months ago
Reviewers:
Bertrand Bordage, Patrick McCarty, carl.d.sorensen
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Add dots to tocItemMarkup * New markup command fill-with-dots * Add it to tocItemMarkup

Patch Set 1 #

Total comments: 2

Patch Set 2 : new version. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+45 lines, -3 lines) Patch
M ly/toc-init.ly View 1 1 chunk +1 line, -3 lines 0 comments Download
M scm/define-markup-commands.scm View 1 1 chunk +44 lines, -0 lines 0 comments Download

Messages

Total messages: 6
Graham Percival (old account)
LGTM.
13 years, 2 months ago (2011-02-15 00:53:38 UTC) #1
Carl
I like the idea. I think to become part of the distribution it needs to ...
13 years, 2 months ago (2011-02-15 00:56:02 UTC) #2
Patrick McCarty
http://codereview.appspot.com/4172047/diff/1/scm/define-markup-commands.scm File scm/define-markup-commands.scm (right): http://codereview.appspot.com/4172047/diff/1/scm/define-markup-commands.scm#newcode1033 scm/define-markup-commands.scm:1033: #:with-dimensions (cons 0 middle-width) '(0 . 0) #:postscript (string-append ...
13 years, 2 months ago (2011-02-15 02:37:52 UTC) #3
Bertrand Bordage
Update on http://code.google.com/p/lilypond/issues/detail?id=1517
13 years, 2 months ago (2011-02-15 13:17:01 UTC) #4
Graham Percival (old account)
On 2011/02/15 13:17:01, Bertrand Bordage wrote: > Update on http://code.google.com/p/lilypond/issues/detail?id=1517 I've added this to codereview. ...
13 years, 2 months ago (2011-02-15 17:50:53 UTC) #5
Bertrand Bordage
13 years, 2 months ago (2011-02-15 19:14:18 UTC) #6
Yes, sorry...
I made a new codereview issue : http://codereview.appspot.com/4182056/


...(currently reading the Contributor's Guide again)...
Sign in to reply to this message.

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