Re: Various chain-assoc-get -> #:properties (issue 323940043 by d...@gnu.org)

2017-06-17 Thread dak
https://codereview.appspot.com/323940043/diff/20001/Documentation/snippets/new/three-sided-box.ly File Documentation/snippets/new/three-sided-box.ly (right): https://codereview.appspot.com/323940043/diff/20001/Documentation/snippets/new/three-sided-box.ly#newcode36

Re: Various chain-assoc-get -> #:properties (issue 323940043 by d...@gnu.org)

2017-06-16 Thread Carl . D . Sorensen
LGTM. One small possible change. https://codereview.appspot.com/323940043/diff/20001/Documentation/snippets/new/three-sided-box.ly File Documentation/snippets/new/three-sided-box.ly (right):

Various chain-assoc-get -> #:properties (issue 323940043 by d...@gnu.org)

2017-06-16 Thread thomasmorley65
LGTM one nit: https://codereview.appspot.com/323940043/diff/1/Documentation/snippets/three-sided-box.ly File Documentation/snippets/three-sided-box.ly (right): https://codereview.appspot.com/323940043/diff/1/Documentation/snippets/three-sided-box.ly#newcode45