Menu

#2276 multiple stanzas chord symbols collide with staff

Verified
nobody
Critical
2012-02-11
2012-02-03
Anonymous
No

Originally created by: *anonymous

Originally created by: colingh...@gmail.com

Reported by Patrick here:

http://article.gmane.org/gmane.comp.gnu.lilypond.bugs/32983

Hey all,

this is a weird one! Chord symbols collide with the staff when the score contains more than one
stanza/verse. For some reason the effect only shows up after the first line break. It doesn't matter
whether it's a manual or automatic break. Even stranger: there is no collision after the second \break. In
some scores the chord symbols only collide when there are three or more stanzas/verses.

HTH
patrick

Related

Issues: #3160

Discussion

  • Google Importer

    Google Importer - 2012-02-03

    Originally posted by: colingh...@gmail.com

    Screenshot for bug under Windows 7 SP1

     
  • Google Importer

    Google Importer - 2012-02-03

    Originally posted by: PhilEHol...@googlemail.com

    It's a regression that happened between 2.13.39 and 2.13.40.  The CG says "Type-Critical: normally a regression against a previous stable version" and strictly, this is a regression against _a_ previous stable version, not _the_ previous stable version.  Not marked as such currently, since I believe we're working to the _the_ wording.

    Labels: Regression
    Status: Accepted

     
  • Google Importer

    Google Importer - 2012-02-03

    Originally posted by: Carl.D.S...@gmail.com

    Critical, since it's regression against last 2 stable releases.

    Labels: -Type-Ugly Type-Critical

     
  • Google Importer

    Google Importer - 2012-02-03

    Originally posted by: Carl.D.S...@gmail.com

    Probably related to 2217 -- both show up only on second line.

     
  • Google Importer

    Google Importer - 2012-02-03

    Originally posted by: dak@gnu.org

    Late 2.13 seems to be good for a lot of regressions.  If we had had 2.16 out earlier, all these would no longer count as critical.

    Hopefully we'll fare better with late 2.15.

     
  • Google Importer

    Google Importer - 2012-02-03

    Originally posted by: k-ohara5...@oco.net

    This, and issue 2217, seem to be due to the fact that
    1) the required spacing between systems is determined by looking at their skylines, which allows for interleaving between protruding graphical objects
    but
    2) the baseline of a non-staff line (Lyrics, ChordNames, Dynamics, etc.) is currently not allowed to be higher than that of a previous line, even if they could interleave.

    \paper { #(set-paper-size "a6") indent = 0 }
      <<
        \new Dynamics \repeat unfold 6 {s1\f s }
        \new Dynamics \repeat unfold 6 {s\f s }
        \new Staff \repeat unfold 6 {g' g'}
        \new Dynamics \repeat unfold 6 {s s\f }
        \new Dynamics \repeat unfold 6 {s s\f }
        \new Dynamics \repeat unfold 6 {s s\f }  >>

     
  • Google Importer

    Google Importer - 2012-02-03

    Originally posted by: k-ohara5...@oco.net

    (No comment was entered for this change.)

     
  • Google Importer

    Google Importer - 2012-02-04

    Originally posted by: k-ohara5...@oco.net

    Probably we should increase the space between systems, so that the loose lines (ChordNames and Lyrics) do not interleave, even if gaps would allow them to interleave.

    http://codereview.appspot.com/5624055/

    It would very be good if someone can adopt this, because I will be overseas for the rest of February and not in position to commit changes.

    Labels: Patch-review
    Status: Started

     
  • Google Importer

    Google Importer - 2012-02-05

    Originally posted by: k-ohara5...@oco.net

    Patch at
    http://codereview.appspot.com/5624055/
    changes no regression test output, but modifies one regression test to catch this issue.

     
  • Google Importer

    Google Importer - 2012-02-05

    Originally posted by: ColinPKC...@gmail.com

    @Keith: Do your comments on Rietveld mean you still want to work on this, and so it should be set to "needs work"?  If you would like it added to today's count down list, would you let me know and I'll update accordingly.

     
  • Google Importer

    Google Importer - 2012-02-05

    Originally posted by: k-ohara5...@oco.net

    My comments were in case somebody else adopts the patch.  It is good to go so far as I can see.

    Labels: -Patch-review Patch-push

     
  • Google Importer

    Google Importer - 2012-02-05

    Originally posted by: k-ohara5...@oco.net

    (No comment was entered for this change.)

    Labels: -Patch-push Patch-countdown

     
  • Google Importer

    Google Importer - 2012-02-07

    Originally posted by: ColinPKC...@gmail.com

    Counted down to 20120207, please push.

    Labels: -Patch-countdown Patch-push

     
  • Google Importer

    Google Importer - 2012-02-08

    Originally posted by: k-ohara5...@oco.net

    Patch committed as 7a60a342

    Labels: -Patch-push Fixed_2_15_29
    Status: Fixed

     
  • Google Importer

    Google Importer - 2012-02-11

    Originally posted by: ColinPKC...@gmail.com

    Verified with the code from Patrick Schmidt's report

    Status: Verified

     
Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.