|
From: | Auto mailings of changes to Lily Issues |
Subject: | [Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4723 Autogenerate documentation for all outside-staff-priority values in NR |
Date: | Wed, 25 May 2016 15:24:07 +0000 |
Diff:
--- old +++ new @@ -1,7 +1,4 @@ -In reference to: - -https://sourceforge.net/p/testlilyissues/issues/4719/ - +In reference to issue [#4719] From Carl Sorensen:
[issues:#4723] Autogenerate documentation for all outside-staff-priority values in NR
Status: Accepted
Created: Wed Jan 06, 2016 04:17 PM UTC by pkx166h
Last Updated: Wed Jan 06, 2016 04:17 PM UTC
Owner: nobody
In reference to issue [#4719]
From Carl Sorensen:
regarding:
https://codereview.appspot.com/280580043
After much tutoring from Graham, I believe this is the wrong thing to do,
for two reasons.
1) An exhaustive table that is manually, not automatically, generated is a
maintainability nightmare.
2) If an exhaustive table exists, it belongs in the NR, not the LM.
I believe the proper way to respond to the issue is the following:
Definitely right now:
A) In the LM, clarify the sentence on looking up OttavaBracket as follows:
"All we need to do is look up the outside-staff-priority of a TextSpanner
in the IR, and then set the outside-staff-priority of the OttavaBracket to
a slightly smaller value."
Optionally, based on some advanced texinfo magic:
B) Create an appendix in the NR that automatically finds all default
values of outside-staff-priority and creates a table.
C) Refer to this automatically-created table in the LM.
In my opinion, absent the automatically-created table, the right thing is
to strengthen the ability of the user to find the outside-staff-priority
of the relevant objects, not to create a potentially wrong table.
Thanks,
Carl
Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------ Mobile security can be enabling, not merely restricting. Employees who bring their own devices (BYOD) to work are irked by the imposition of MDM restrictions. Mobile Device Manager Plus allows you to control only the apps on BYO-devices by containerizing them, leaving personal data untouched! https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________ Testlilyissues-auto mailing list address@hidden https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
[Prev in Thread] | Current Thread | [Next in Thread] |