Re: [tablatures] tie bug in 2.13.10?

[ Thread Index | Date Index | More lilynet.net/tablatures Archives ]




On 1/13/10 1:05 AM, "Marc Hohl" <marc@xxxxxxxxxx> wrote:

> Carl Sorensen schrieb:
>> 
>>  
>> 
>> This seems to me to call for a new variable
>> 
>> hideTabNoteHead = {
>>   \once \override TabNoteHead #'transparent = ##t
>>   \once \override TabNoteHead #'whiteout = ##f
>> }
>> 
>> and to replace
>> 
>> \once \override TabNoteHead #'transparent = ##t
>> 
>> with
>> 
>> \hideTabNoteHead
>> 
>> 
>>  
> This should go to ly/property-init then, right?
> 
> Should I create a patch? Has this to be documented somewhere
> in the docs?

Do you intend for this to be used by users?  If so, it would go in
ly/property-init.ly and should be documented.  A single paragraph and a
simple snippet in the tablature section would suffice.  It would be great if
you could create a patch.

>>> and every
>>> 
>>> (ly:grob-set-property! right-tab-note-head 'transparent #t)
>>> 
>>> should be followed by
>>> 
>>> (ly:grob-set-property! right-tab-note-head 'whiteout #f)
>>> 
>>> in bend.ly.
>>>    
>> 
>> Similarly, we should have
>> 
>> (define (hide-tab-note-head note-head)
>>   (ly:grob-set-property! note-head 'whiteout #f)
>>   (ly:grob-set-property! note-head 'transparent #t))
>> 
>> and replace
>> 
>> (ly:grob-set-property! right-tab-note-head 'transparent #t)
>> 
>> with
>> 
>> (hide-tab-note-head right-tab-note-head)
>> 
>> 
>> On a related note, Marc, why do you want to use ly:grob-set-property!
>> instead of using an override function?
>>  
> This is used in bend.ly, which has alpha status, and since we
> want to do bends by a new engraver, I don't think it makes
> sense to enhance this file.

Sounds great to me.

Thanks,

Carl




Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/