Menu

#5489 improve prall glyphs

Fixed
Enhancement
2019-03-09
2019-03-03
Malte Meyn
No

Issue 5489: improve prall glyphs

• rename trilelement → trillelement
• fix erroneous shape of pralldown/prallup
• enlarge arc of downprall/pralldown
• move and shorten line of lineprall

http://codereview.appspot.com/341560043

Discussion

  • Anonymous

    Anonymous - 2019-03-03
    • Description has changed:

    Diff:

    
    
    • Needs: -->
    • Patch: new --> review
    • Type: --> Enhancement
     
    • David Kastrup

      David Kastrup - 2019-03-03

      "pkx166h" lilypond-pkx@users.sourceforge.net writes:

      • Description has changed:

      Diff:

      ~~~~

      ~~~~

      • Needs: -->
      • Patch: new --> review
      • Type: --> Enhancement
      • Comment:

      Passes make, make check and a full make doc.

      (although I don't know why I am not seeing any extra diffs - do we not
      reg test these glyphs anywhere?)

      Our regtests only trigger for new metrics, not new visuals. So you only
      get to see new visuals in comparisons if the metrics changed as well.

      --
      David Kastrup

       
  • Anonymous

    Anonymous - 2019-03-03

    Passes make, make check and a full make doc.

    (although I don't know why I am not seeing any extra diffs - do we not reg test these glyphs anywhere?)

     
    • Malte Meyn

      Malte Meyn - 2019-03-03

      (although I don't know why I am not seeing any extra diffs - do we not reg test these glyphs anywhere?)

      No, we don’t, only \prall is used in some regtests. Would the normal regtests show differences? I thought they only show differences in positioning, not in appearance of grobs/glyphs.

       
      • David Kastrup

        David Kastrup - 2019-03-03

        "Malte Meyn" maltem@users.sourceforge.net writes:

        (although I don't know why I am not seeing any extra diffs - do we
        not reg test these glyphs anywhere?)

        No, we don’t, only \prall is used in some regtests. Would the normal
        regtests show differences? I thought they only show differences in
        positioning, not in appearance of grobs/glyphs.

        They show the difference in appearance (they are graphical comparisons
        after all). But only when triggered by a difference in
        positioning/metrics.

        --
        David Kastrup

         
  • Anonymous

    Anonymous - 2019-03-04
    • Needs: -->
    • Patch: new --> review
    • Type: --> Enhancement
     
  • Anonymous

    Anonymous - 2019-03-04

    Passes make, make check and a full make doc.

     
  • Anonymous

    Anonymous - 2019-03-06
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2019-03-06

    Patch on countdown for March 9th

     
  • Anonymous

    Anonymous - 2019-03-09
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2019-03-09

    Patch counted down - please push to staging.

     
  • Malte Meyn

    Malte Meyn - 2019-03-09
    • labels: --> Fixed_2_21_0
    • summary: Issue 5489: improve prall glyphs --> improve prall glyphs
    • status: Started --> Fixed
    • Patch: push -->
     
  • Malte Meyn

    Malte Meyn - 2019-03-09

    commit 56942ccc5f5a7ac0088b16e3f7d28ff6bffc530d
    Author: Malte Meyn lilypond@maltemeyn.de
    Date: Sun Mar 3 09:57:10 2019 +0100

    Issue 5489/2: rename trilelement → trillelement
    
    including convert-ly rule
    

    commit 2afb45bd85d85ddd06855aa3eea730d72581444c
    Author: Malte Meyn lilypond@maltemeyn.de
    Date: Sun Mar 3 09:47:56 2019 +0100

    Issue 5489/1: improve prall glyphs
    
    • fix erroneous shape of pralldown/prallup
    • enlarge arc of downprall/pralldown
    • move and shorten line of lineprall