Just for the record, I experimented with setting top-system-spacing to 0 in the 
"one-line-auto-height-breaking.cc" file like so:

`book_->paper_->set_variable (ly_symbol2scm ("top-system-spacing"), 
scm_from_int (0));`

But this had no effect as far as I could tell.  (Perhaps top-system-spacing is 
given a value later and any changes made here are overwritten?)

To illustrate what happens with top-system-spacing, attached are a couple of 
images with all margins set to 1.  One shows leaving top-system-spacing 
unchanged (which pushes the music down so it falls off the bottom of the page) 
and one shows it set to 0 (so that the music is centered vertically between the 
top and bottom margins).

I assume that for many use cases, top-system-spacing will need to be set to 0, 
so I included a note about that in the documentation.


Attachments:

- 
[margins-1-top-system-spacing-unset.png](https://sourceforge.net/p/testlilyissues/issues/_discuss/thread/5a24f851/904b/attachment/margins-1-top-system-spacing-unset.png)
 (5.7 kB; image/png)
- 
[margins-1-top-system-spacing-zero.png](https://sourceforge.net/p/testlilyissues/issues/_discuss/thread/5a24f851/904b/attachment/margins-1-top-system-spacing-zero.png)
 (5.9 kB; image/png)


---

** [issues:#4743] Add ly:one-line-auto-height-breaking**

**Status:** Started
**Created:** Thu Jan 14, 2016 05:21 AM UTC by Paul Morris
**Last Updated:** Thu Jan 14, 2016 03:54 PM UTC
**Owner:** Paul Morris


Add ly:one-line-auto-height-breaking

Includes 4 commits:

Changes entry for ly:one-line-auto-height-breaking

Doc: NR 4.3.2 add ly:one-line-auto-height-breaking
  Includes a few related edits to NR 4.3.2
  mainly for ly:one-line-breaking

Add regtest for ly:one-line-auto-height-breaking

Add ly:one-line-auto-height-breaking

http://codereview.appspot.com/284240043


---

Sent from sourceforge.net because [email protected] is 
subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/testlilyissues/admin/issues/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto

Reply via email to