Reviewers: Trevor Daniels, thomasmorley65,
http://codereview.appspot.com/6851102/diff/3001/Documentation/learning/tweaks.itely File Documentation/learning/tweaks.itely (right): http://codereview.appspot.com/6851102/diff/3001/Documentation/learning/tweaks.itely#newcode1499 Documentation/learning/tweaks.itely:1499: \omit Staff.Clef On 2012/11/26 08:34:37, Trevor Daniels wrote:
That's really confusing. If a context is specified in an \override in the same location it generates no warning or error but becomes ineffective. Unless this can be changed this needs much more explanation.
The underlying rationale was that music containing overrides is _filtered_ when used in context modifications, and only those overrides applying to a context of the proper type are actually being applied. I think the idea was to be able to cram \time, \clef and other stuff into one \global context modification and use that everywhere, sorting itself out properly. I agree that the effect that this has here is not really harmonizing well with \override (which does not allow a context spec inside of context mods, and will not silently ignore Bottom overrides in non-Bottom contexts), so it might make sense to just drop this (not-really-yet-reached) design goal and the associated "picky" context modifications which are not really documented on the user or even the concept level. Description: Issue 2916: Document \hide and \omit Also fixes a few other inaccuracies. Please review this at http://codereview.appspot.com/6851102/ Affected files: M Documentation/learning/tweaks.itely _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
