On 2019/10/19 06:08:51, lemzwerg wrote:
Very nice, thanks!

Thanks for all your reviewing time lately.

Wouldn't it be better/more convenient/more compact to write the CSS
stuff into
the HTML file, too?

The style is also used for the "details" files, e.g.


http://faithful.be/tmp/test-results/input/regression/out-test/rest-dot-position.details.html

So I think it makes more sense to leave it external.

Otherwise I can imagine to provide `style.css` in the
source tree, to be not generated at all.

We could probably say the same for the JavaScript.  Viewing things as
they are, that's not a bad idea; however, there was a point during this
task where I almost introduced differences in the CSS based on the
--no-compare-images option.  I decided I had more useful tasks to get on
with, but I don't want to move the CSS to its own file and learn later
that it wasn't well considered.

https://codereview.appspot.com/566920043/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to