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

2013-12-14 Thread Carl Peterson
On Sat, Dec 14, 2013 at 1:50 AM, James pkx1...@gmail.com wrote: On 14/12/13 05:57, Carl Peterson wrote: I have updated the patch in Rietvald But not the tracker. So the patch will not get tested. Please remember to either use git-cl or change the tracker to patch-new whenever you make a

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

2013-12-14 Thread David Kastrup
Carl Peterson carlopeter...@gmail.com writes: On Sat, Dec 14, 2013 at 1:50 AM, James pkx1...@gmail.com wrote: On 14/12/13 05:57, Carl Peterson wrote: I have updated the patch in Rietvald But not the tracker. So the patch will not get tested. Please remember to either use git-cl or change

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

2013-12-14 Thread Phil Holmes
- Original Message - From: David Kastrup d...@gnu.org To: Carl Peterson carlopeter...@gmail.com Cc: James pkx1...@gmail.com; Lilypond Dev lilypond-devel@gnu.org; Phil Holmes m...@philholmes.net Sent: Saturday, December 14, 2013 2:27 PM Subject: Re: Enable manual-specific styling

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

2013-12-13 Thread Phil Holmes
- Original Message - From: Carl Peterson To: Carl Peterson Cc: Lilypond Dev Sent: Thursday, December 12, 2013 10:58 PM Subject: Re: Enable manual-specific styling of documentation; issue 3714(issue 36480048) On Thu, Dec 12, 2013 at 1:06 AM, carlopeter...@gmail.com wrote

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

2013-12-13 Thread James
manual-specific styling of documentation; issue 3714(issue 36480048) On Thu, Dec 12, 2013 at 1:06 AM, carlopeter...@gmail.com mailto:carlopeter...@gmail.com wrote: Message: Patch for initial solution to issue 3714, regarding color-coding of manuals

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

2013-12-13 Thread Carl Peterson
On Fri, Dec 13, 2013 at 8:31 AM, James pkx1...@gmail.com wrote: On 13/12/13 13:04, Phil Holmes wrote: Thanks for what you're doing, but please don't put a lot of images on the Google Issue tracker. For bizarre reasons only known to themselves, the storage available for attachments is _very_

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

2013-12-13 Thread David Kastrup
Carl Peterson carlopeter...@gmail.com writes: On Fri, Dec 13, 2013 at 8:31 AM, James pkx1...@gmail.com wrote: On 13/12/13 13:04, Phil Holmes wrote: Thanks for what you're doing, but please don't put a lot of images on the Google Issue tracker. For bizarre reasons only known to themselves,

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

2013-12-13 Thread Phil Holmes
- Original Message - From: Carl Peterson carlopeter...@gmail.com To: James pkx1...@gmail.com Cc: Lilypond Dev lilypond-devel@gnu.org Sent: Friday, December 13, 2013 2:01 PM Subject: Re: Enable manual-specific styling of documentation;issue 3714(issue 36480048) On Fri, Dec 13, 2013

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

2013-12-13 Thread Carl Peterson
On Thu, Dec 12, 2013 at 7:36 PM, carl.d.soren...@gmail.com wrote: I think that the index sidebar colors are too dark. They dominate the page, in my opinon. In the current design, the sidebar color and the highlight box fill color are the same. Why not keep it that way? Attaching another

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

2013-12-13 Thread Carl Sorensen
On 12/13/13 8:02 AM, Carl Peterson carlopeter...@gmail.com wrote: On Thu, Dec 12, 2013 at 7:36 PM, carl.d.soren...@gmail.com wrote: I think that the index sidebar colors are too dark. They dominate the page, in my opinon. In the current design, the sidebar color and the highlight box fill

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

2013-12-13 Thread Carl Sorensen
On 12/13/13 6:04 AM, Phil Holmes m...@philholmes.net wrote: Thanks for what you're doing, but please don't put a lot of images on the Google Issue tracker. For bizarre reasons only known to themselves, the storage available for attachments is _very_ limited. By accident you've just used about

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

2013-12-13 Thread Phil Holmes
- Original Message - From: Carl Sorensen c_soren...@byu.edu To: Phil Holmes m...@philholmes.net; Carl Peterson carlopeter...@gmail.com Cc: Lilypond Dev lilypond-devel@gnu.org Sent: Friday, December 13, 2013 3:31 PM Subject: Re: Enable manual-specific styling of documentation; issue

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

2013-12-13 Thread Graham Percival
On Fri, Dec 13, 2013 at 10:02:23AM -0500, Carl Peterson wrote: On Thu, Dec 12, 2013 at 7:36 PM, carl.d.soren...@gmail.com wrote: I think that the index sidebar colors are too dark. They dominate the page, in my opinon. In the current design, the sidebar color and the highlight box fill

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

2013-12-13 Thread Carl Peterson
On Fri, Dec 13, 2013 at 10:41 PM, Graham Percival gra...@percival-music.ca wrote: On Fri, Dec 13, 2013 at 10:02:23AM -0500, Carl Peterson wrote: On Thu, Dec 12, 2013 at 7:36 PM, carl.d.soren...@gmail.com wrote: I think that the index sidebar colors are too dark. They dominate the page, in

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

2013-12-13 Thread James
On 14/12/13 05:57, Carl Peterson wrote: On Fri, Dec 13, 2013 at 10:41 PM, Graham Percival gra...@percival-music.ca wrote: On Fri, Dec 13, 2013 at 10:02:23AM -0500, Carl Peterson wrote: On Thu, Dec 12, 2013 at 7:36 PM, carl.d.soren...@gmail.com wrote: I think that the index sidebar colors are

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

2013-12-12 Thread tdanielsmusic
LGTM, although it is difficult to visualise the effect of the various colours without seeing them. https://codereview.appspot.com/36480048/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

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

2013-12-12 Thread Carl Peterson
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):

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

2013-12-12 Thread Carl Peterson
On Thu, Dec 12, 2013 at 7:49 AM, Carl Peterson carlopeter...@gmail.comwrote: 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

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

2013-12-12 Thread Carl Peterson
On Thu, Dec 12, 2013 at 4:10 AM, tdanielsmu...@googlemail.com wrote: LGTM, although it is difficult to visualise the effect of the various colours without seeing them. https://codereview.appspot.com/36480048/ I'll see what I can do about getting representative screenshots.

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

2013-12-12 Thread Carl Peterson
On Thu, Dec 12, 2013 at 1:06 AM, carlopeter...@gmail.com wrote: Message: Patch for initial solution to issue 3714, regarding color-coding of manuals. Please review this at https://codereview.appspot.com/36480048/ I have submitted follow-up patches to Rietveld to address comments regarding

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

2013-12-12 Thread Carl . D . Sorensen
On 2013/12/12 22:50:05, Carl P. wrote: issue 3714, adjust sidebar colors for legibility and contrast I think that the index sidebar colors are too dark. They dominate the page, in my opinon. In the current design, the sidebar color and the highlight box fill color are the same. Why not keep

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

2013-12-12 Thread Graham Percival
On Thu, Dec 12, 2013 at 05:58:33PM -0500, Carl Peterson wrote: For those who need a visual of these changes, I've uploaded screenshots to the Google Code issue. https://code.google.com/p/lilypond/issues/detail?id=3714. Woah, why are you changing the whole background? It looks a bit

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

2013-12-11 Thread carlopeterson
Reviewers: , Message: Patch for initial solution to issue 3714, regarding color-coding of manuals. This patch color codes more-or-less according to David K.'s proposal on the lilypond-user. Does not currently distinguish between dev and stable in the styling, but the functionality is there to

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

2013-12-11 Thread lemzwerg
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

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

2013-12-11 Thread James
Carlo, On 12/12/13 06:06, carlopeter...@gmail.com wrote: Reviewers: , Message: Patch for initial solution to issue 3714, regarding color-coding of manuals. . Make sure the tracker is set to patch-new else it won't get reviewed properly (if at all) and certainly won't get tested and