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

Reply via email to