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