Re: Enable manual-specific styling of documentation; issue 3714 (issue 36480048)

[Carl Peterson]

On Thu, Dec 12, 2013 at 7:49 AM, Carl Peterson <address@hidden> wrote:

On Thu, Dec 12, 2013 at 2:21 AM, <address@hidden> 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