lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Issue 2916: Document \hide and \omit (issue 6851102)


From: dak
Subject: Re: Issue 2916: Document \hide and \omit (issue 6851102)
Date: Mon, 26 Nov 2012 10:03:28 +0000

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





reply via email to

[Prev in Thread] Current Thread [Next in Thread]