Here is the nearest I can get to Kieren’s suggestion 
(https://codereview.appspot.com/327620043#msg21). This is far from optimal: 
Currently, it works only easy with grob properties of MetronomeMark (see line 
54ff.), not with context and event properties. But it looks as if it works 
(line 75ff.).

Disclaimer: I don’t exactly know what I’m doing here. F. e. I have no idea why 
the props argument knows about the grob in line 8. I copied that from Harm’s 
proposal at https://sourceforge.net/p/testlilyissues/issues/5215/#e472.


Attachments:

- 
[fancyMetronome.ly](https://sourceforge.net/p/testlilyissues/issues/_discuss/thread/a8839387/505b/attachment/fancyMetronome.ly)
 (2.6 kB; text/x-lilypond)


---

** [issues:#5215] make metronomeMarkFormatter more flexible**

**Status:** Started
**Created:** Tue Oct 10, 2017 12:46 PM UTC by Malte Meyn
**Last Updated:** Mon Oct 23, 2017 06:38 AM UTC
**Owner:** Malte Meyn
**Attachments:**

- 
[Image1.png](https://sourceforge.net/p/testlilyissues/issues/5215/attachment/Image1.png)
 (26.5 kB; image/png)
- 
[Image2.png](https://sourceforge.net/p/testlilyissues/issues/5215/attachment/Image2.png)
 (35.1 kB; image/png)


make metronomeMarkFormatter more flexible

This adds the context properties tempoEquationText, tempoBetweenText and
tempoShowParentheses as shown in http://lsr.di.unimi.it/LSR/Item?id=1008

It also allows to scale the size of the notes in a metronome mark
independently from or rather relatively to the text and numbers.
I added this possibility because http://lsr.di.unimi.it/LSR/Item?id=1008
suggests smaller note sizes; so there seems to be a need for that.

The default values are chosen so that the whole thing is backwards
compatible; to achieve this, tempoShowParentheses accepts not only
boolean values but also the symbol 'if-text.

I chose the name tempoShowParentheses instead of tempoHideParentheses
because this property also allows parenthesizing text-less
MetronomeMarks.

Contains regtests.

http://codereview.appspot.com/327620043


---

Sent from sourceforge.net because [email protected] is 
subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/testlilyissues/admin/issues/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
  • [Lilypond-... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto

Reply via email to