On Thu, Dec 12, 2013 at 7:49 AM, Carl Peterson <carlopeter...@gmail.com>wrote:
> On Thu, Dec 12, 2013 at 2:21 AM, <lemzw...@googlemail.com> wrote: > >> LGTM, with one small nit. >> >> >> https://codereview.appspot.com/36480048/diff/1/ >> Documentation/css/lilypond-manuals.css >> File Documentation/css/lilypond-manuals.css (right): >> >> https://codereview.appspot.com/36480048/diff/1/ >> Documentation/css/lilypond-manuals.css#newcode456 >> Documentation/css/lilypond-manuals.css:456: body.learning { >> background-color: #ccffcc; } >> This and the next lines are a duplication from the top of the patch. >> Why? Looks like a mistake... >> >> https://codereview.appspot.com/36480048/ >> > > It could be. It may need to be deleted, or there may have been something > else I was intending to style. I'll take a look a bit later today. > I just remembered. It does need to be deleted. I had pulled down the earlier chunk of code to get the hex color values and forgot to take it back out. I'll update the patch later, when I have access to my dev machine. Thanks for catching. Carl
_______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel