Menu

#427 ugly collision slur and phrasing slur

Verified
nobody
Ugly
2012-07-08
2007-08-27
Anonymous
No

Originally created by: *anonymous

Originally created by: gpermus@gmail.com

%{
Slur starts before the key signature, but phrasing slur and tie start after
the key signature.  This results in an ugly collision between slur and
phrasing slur.
%}

\version "2.11.30"

\relative c'' {
  \key c \minor
  f e\( d( c~
  \break
  c d) e\) f
}

Discussion

  • Google Importer

    Google Importer - 2007-08-27

    Originally posted by: gpermus@gmail.com

    Sorry, forgot the png.

     
  • Google Importer

    Google Importer - 2008-05-18

    Originally posted by: v.villenave

    Comment from Andrew:

    http://lists.gnu.org/archive/html/bug-lilypond/2008-05/msg00097.html

    how much work would it be to change ordinary slurs to begin after the key signature?
    I believe that this would solve Issue 379 and Issue 427

    I tried reading the slur code myself but I was totally lost.
    I'm just looking for an estimate of the difficulty/likelihood of a fix for starters.

     
  • Google Importer

    Google Importer - 2008-05-19

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

    that surprises me. I can't remember that there was special handling for this. Can you
    reproduce this with 2 identical situations where only the phrasingslur vs slur is
    different?

     
  • Google Importer

    Google Importer - 2008-05-23

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

    % Difference between slurs and phrasing slurs across line breaks:

    \version "2.11.44"

    \paper {
      ragged-right = ##t
      indent = 0
    }

    \relative c' {
      \key b \major
      b4( cis dis8 e fis4 \break
      b,4^\markup "...slur" dis8 cis8 b2) \break
      b4\( cis dis8 e fis4 \break
      b,4^\markup "...phrasing slur" dis8 cis8 b2\)
    }

     
  • Google Importer

    Google Importer - 2008-06-27

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

    The difference almost certainly lies in slur-engraver.cc and
    phrasing-slur-engraver.cc, since they have identical "include" sections. I'm guessing
    that the culprit is the stop_translation_timestep() method, since the Slur_engraver
    version has a bunch of add_extra_encompass commands that are not found in the
    Phrasing_slur_engraver.

    There are a few other differences that appear unintentional, like more robust error
    checking in the Slur version, and no acknowledge_tuplet_number in Phrasing_slur_engraver.

    Here's a better example of the difference

    \version "2.11.49"
    \paper { ragged-right=##t }
    {
      \key e \major
      b'1(\( \break
      e')\)
    }

     
  • Google Importer

    Google Importer - 2008-10-20

    Originally posted by: csnyd...@mvpsoft.com

    I'll chip in $50 USD to see these two bugs fixed.

     
  • Google Importer

    Google Importer - 2008-10-27

    Originally posted by: v.villenave

    See also the comment from Mark at Issue 379 #c4.

     
  • Google Importer

    Google Importer - 2009-08-13

    Originally posted by: pnorcks@gmail.com

    (No comment was entered for this change.)

    Status: Started

     
  • Google Importer

    Google Importer - 2009-12-19

    Originally posted by: percival.music.ca@gmail.com

    (No comment was entered for this change.)

    Owner: pnorcks

     
  • Google Importer

    Google Importer - 2011-02-22

    Originally posted by: pnorcks@gmail.com

    I'm no longer working on this issue.  Also, see Issue 379.

    Here's the patch, in case someone wants to have a look:

    http://codereview.appspot.com/109071

    Labels: -Engraving-nitpick
    Owner: ---
    Status: Accepted

     
  • Google Importer

    Google Importer - 2011-02-22

    Originally posted by: percival.music.ca@gmail.com

    (No comment was entered for this change.)

    Labels: Patch-abandoned

     
  • Google Importer

    Google Importer - 2011-08-20

    Originally posted by: percival.music.ca@gmail.com

    (No comment was entered for this change.)

    Labels: -type-Collision Type-Ugly

     
  • Google Importer

    Google Importer - 2011-08-25

    Originally posted by: kie...@alumni.rice.edu

    + C$50 to fix cross-break tie/slur/phrasingslur problems

     
  • Google Importer

    Google Importer - 2012-06-20

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

    commit [r16a328617e867892e8d5608f95bd6127196e03a2]

    Labels: -Patch-abandoned fixed_2_15_41
    Status: Fixed

     
  • Google Importer

    Google Importer - 2012-07-06

    Originally posted by: ma...@gregoriana.sk

    (No comment was entered for this change.)

    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.