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

Issue 349080043: Issue 5218: document footnote-separator-markup

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 months, 2 weeks ago by Malte Meyn
Modified:
4 months, 1 week ago
Reviewers:
thomasmorley651
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Issue 5218: document footnote-separator-markup

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -1 line) Patch
M Documentation/notation/input.itely View 1 chunk +4 lines, -1 line 0 comments Download
M Documentation/notation/spacing.itely View 1 chunk +6 lines, -0 lines 2 comments Download

Messages

Total messages: 1
thomasmorley651
4 months, 1 week ago (2019-01-05 11:05:31 UTC) #1
Some nits:

https://codereview.appspot.com/349080043/diff/1/Documentation/notation/spacin...
File Documentation/notation/spacing.itely (right):

https://codereview.appspot.com/349080043/diff/1/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1082: bottom of the page.  Default: a
centered horizontal line.
I'd go for:
Default: a centered horizontal line, defined in @file{ly/paper-defaults-init.ly}

https://codereview.appspot.com/349080043/diff/1/Documentation/notation/spacin...
Documentation/notation/spacing.itely:1088: @file{ly/titling-init.ly}.
And add here:
@file{ly/paper-defaults-init.ly}
Sign in to reply to this message.

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