Re: Prevent granados.ly image rotation in web.pdf (issue 317560043 by truer...@gmail.com)

2017-04-29 Thread dak
LGTM Now the following does not concern this issue directly but I see two more occurences in input/regression/landscape.ly and input/regression/page-links.ly . I feel ambiguous about either here. I could imagine leaving landscape.ly in "true" landscape while merely using landscape paper size in

Re: Prevent granados.ly image rotation in web.pdf (issue 317560043 by truer...@gmail.com)

2017-04-29 Thread trueroad
Thank you for your opinion. I've removed the comment. https://codereview.appspot.com/317560043/diff/20001/Documentation/ly-examples/granados.ly File Documentation/ly-examples/granados.ly (right): https://codereview.appspot.com/317560043/diff/20001/Documentation/ly-examples/granados.ly#newcode12

Re: Prevent granados.ly image rotation in web.pdf (issue 317560043 by truer...@gmail.com)

2017-04-29 Thread dak
https://codereview.appspot.com/317560043/diff/20001/Documentation/ly-examples/granados.ly File Documentation/ly-examples/granados.ly (right): https://codereview.appspot.com/317560043/diff/20001/Documentation/ly-examples/granados.ly#newcode12 Documentation/ly-examples/granados.ly:12: % If we use

Re: Prevent granados.ly image rotation in web.pdf (issue 317560043 by truer...@gmail.com)

2017-04-29 Thread trueroad
Thank you for your reviewing. Here's new commit message. Issue 5128: Prevent granados.ly image rotation in web.pdf granados.ly used 'landscape symbol for page settings. As a result, granados.pdf which was generated from granados.ly contained `/Rotate 90` that is not supported by

Prevent granados.ly image rotation in web.pdf (issue 317560043 by truer...@gmail.com)

2017-04-29 Thread dak
https://codereview.appspot.com/317560043/diff/1/Documentation/ly-examples/granados.ly File Documentation/ly-examples/granados.ly (right): https://codereview.appspot.com/317560043/diff/1/Documentation/ly-examples/granados.ly#newcode16 Documentation/ly-examples/granados.ly:16: % as landscape lette