Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(691)

Issue 9747044: Issue 3379: move all tablature related settings to TabStaff (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 11 months ago by dak
Modified:
10 years, 9 months ago
Reviewers:
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Issue 3379: move all tablature related settings to TabStaff Previous to this patch, settings relevant to tablature have been distributed among TabVoice and TabVoice, presumably modeled along the corresponding distribution of engravers. As one consequence, the command \tabFullNotation contained overrides pertaining to both TabStaff and TabVoice level, leading to inconsistent behavior when employed in one but not all TabVoice contexts of a TabStaff, or when used as \new TabStaff \with { \tabFullNotation } Moving all overrides and property settings to TabStaff level should lead to more consistent results.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+80 lines, -79 lines) Patch
M ly/engraver-init.ly View 3 chunks +45 lines, -44 lines 0 comments Download
M ly/performer-init.ly View 2 chunks +1 line, -1 line 0 comments Download
M ly/property-init.ly View 1 chunk +34 lines, -34 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b