#2057 Hairpin hits barline at line break

[Anonymous]

Originally created by: *anonymous

Originally created by: PhilEHol...@googlemail.com
Originally owned by: k-ohara5...@oco.net

%% Hairpin at a line break touches the barline, whereas for a normal bar
%% (i.e. not at a line break) it does not.

\version "2.15.18"

music = \relative c' {
  c1\p\< | c\f\> | \break
  c1\p\< | c\f\> | c1\p\< | \break
  c1\f
}

\score {
  <<
    \new StaffGroup <<
      \new Staff {
        \music
      }
      \new Staff {
        \music
      }
    >>
  >>
}

[Google Importer]

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

Pushed to staging as [r5543eb5b0378bc726684a686b799e4ff4dc5f4cc].

Status: Fixed

[Google Importer]

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

Rectification - the SHA1 ID is [rf417a6007e951fda4f1fb5fd3da0ecd1d25cec2f]

Cheers,
MS

[Google Importer]

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

Verified that this has been committed to the repo.

Status: Verified

[Google Importer]

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

I've put this back to Fixed. Phil Holmes pointed out that this one is a bug fix and so should not simply be marked Verified if it is in the codebase.

Status: Fixed

[Google Importer]

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

I've put this back to Started.  The code base at the moment treats hairpins ending at the end of the line the same as those that continue, so all hairpins get the smaller padding at the end of the line, and none at all if there is no span bar.

Mike and I agreed to integrate his code into the old system.  Then ending hairpins get the simple 'bound-padding while those continuing across the line break get 'broken-bound-padding, which by default uses Mike's new code, and people can override either if needed. After-patch output attached.
<http://codereview.appspot.com/5479071/>

Labels: -Patch-push Patch-new
Owner: k-ohara5...@oco.net
Status: Started

[Google Importer]

Originally posted by: gra...@percival-music.ca

continued in issue 2130.  sorry.

Don't add any patch tags to this issue.

Labels: -Patch-new
Mergedinto: 2130
Status: Duplicate

[Google Importer]

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

Verified with T2130 using 2.15.24

Mergedinto:
Status: Verified