Menu

#997 RehearsalMarks are not kept inside the page

Accepted
nobody
None
Defect
2015-09-27
2010-01-25
Anonymous
No

Originally created by: *anonymous

Originally created by: v.villenave

The following example is taken from the page-label.ly regtest.
Though the regtest doesn't show the bug, when compiled separately the mark goes out of the page.

:::TeX
\version "2.13.7"
%% This obviously doesn't do anything.
\layout {
  \context {
    \Score
    \override NonMusicalPaperColumn #'keep-inside-line = ##t
  }
}

%BEGIN REGTEST

#(set-default-paper-size "a6")

{
  c'2 c' \mark \markup { A (page 3) }
  \break
  c' c'
}

How can this be fixed, and how can we make sure the regtest shows the bug?

Related

Issues: #637

Discussion

  • Google Importer

    Google Importer - 2010-01-25

    Originally posted by: v.villenave

    Looks like a known limitation to me. But IIRC keep-inside-line used to work for these
    things, has the syntax changed or something?

    BTW, here's the PNG.

     
  • Google Importer

    Google Importer - 2010-01-25

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

    "Owner" now means "the person who will be fixing the issue".  I don't think that
    either of you are offering to do the programming.

    New issues should have no owner, unless it's totally obvious who would be working on
    it (such as Joe for spacing stuff).

    Labels: -regtest
    Owner: ---
    Cc: -v.villenave

     
  • Google Importer

    Google Importer - 2010-01-25

    Originally posted by: n.putt...@gmail.com

    I don't think this is a regression; I'm pretty sure 'keep-inside-line is documented
    to work only with markup and lyrics.

     
  • Google Importer

    Google Importer - 2010-01-25

    Originally posted by: n.putt...@gmail.com

    Ah, I've just read James's original report.

    Since 2.13.7 (following improvements to the code for default margins), the left and
    right margins in the regtest are smaller (they're automatically scaled in terms of
    the default paper size), which means the centre of the rehearsal mark is closer to
    the edge of the page.

     
  • Google Importer

    Google Importer - 2010-01-25

    Originally posted by: n.putt...@gmail.com

    Heheh, you got me there. :)

    Though as Nicolas pointed out, it's not very useful.

    Looking at the code briefly, it seems to be deliberately restricted to
    NonMusicalPaperColumn grobs in the middle of a line.

     
  • Google Importer

    Google Importer - 2010-06-27

    Originally posted by: n.putt...@gmail.com

    (No comment was entered for this change.)

    Labels: -Regression

     
  • Google Importer

    Google Importer - 2011-02-21

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

    Issue 1512 has been merged into this issue.

     

    Related

    Issues: #1512

  • Simon Albrecht

    Simon Albrecht - 2015-09-27
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -2,26 +2,29 @@
    
     *Originally created by:* [v.villenave](http://code.google.com/u/v.villenave/)
    
    -% The following example is taken from the page-label.ly regtest.
    +The following example is taken from the page-label.ly regtest.
    +Though the regtest doesn't show the bug, when compiled separately the mark goes out of the page.
    
    -% Though the regtest doesn't show the bug, when compiled separately
    -% the mark goes out of the page.
    -
    +~~~~
    +:::TeX
     \version "2.13.7"
     %% This obviously doesn't do anything.
    -\layout \{ \context \{ \Score
    -    \override NonMusicalPaperColumn \#'keep-inside-line = \#\#t
    -  \} \}
    +\layout {
    +  \context {
    +    \Score
    +    \override NonMusicalPaperColumn #'keep-inside-line = ##t
    +  }
    +}
    
     %BEGIN REGTEST
    
    -\#\(set-default-paper-size "a6"\)
    +#(set-default-paper-size "a6")
    
    -\{
    -  c'2 c' \mark \markup \{ A \(page 3\) \}
    -  \break
    -  c' c'
    -\}
    +{
    +  c'2 c' \mark \markup { A (page 3) }
    +  \break
    +  c' c'
    +}
    +~~~~
    
    -%How can this be fixed, and how can we make sure
    -%the regtest shows the bug?
    +How can this be fixed, and how can we make sure the regtest shows the bug?
    
    • Needs: -->
    • Patch: -->
     
  • Simon Albrecht

    Simon Albrecht - 2015-09-27
     
MongoDB Logo MongoDB