Incorporated most of the comments. Will upload new patch set in a minute
https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi File Documentation/web/download.itexi (right): https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi#newcode38 Documentation/web/download.itexi:38: On 2013/12/13 06:00:09, J_lowe wrote:
Is this whole section conforming to the correct line length - 72 (or
66) chars?
At least here in Rietveld it looks sloppy with weird line breaks.
Maybe this is
just the way it is displayed here but can we make sure please.
Done. https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi#newcode40 Documentation/web/download.itexi:40: @subheading Before you proceed ... On 2013/12/13 06:00:09, J_lowe wrote:
I don't like those elipses and they aren't really needed (or being
used
correctly, they are for missing words and anyway there is a texinfo
command for
'dots'.) can we just have 'Before you proceed' and be done?
I'm OK with this kind of edits. But I'd like to point out that I "copied" the "chatty" style from quite some existing texts on the website: Just a few examples: - "I want to see some music!" - "Yep, it's free." Done. https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi#newcode42 Documentation/web/download.itexi:42: ... you should be aware of some facts and concepts. On 2013/12/13 06:00:09, J_lowe wrote:
Remove this '... you should be aware of ...' line. Subheading is
enough. Done. https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi#newcode50 Documentation/web/download.itexi:50: If you want you can go to @ref{Text input} and learn more about LilyPond's On 2013/12/13 06:00:09, J_lowe wrote:
What happens if they 'don't want?' Remove the 'If you want you can go
to'
Just keep it simple. I.e 'See @ref{Text input} to learn more.'
Done. https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi#newcode53 Documentation/web/download.itexi:53: @subsubheading I'm new to LilyPond and don't have a dedicated editor yet On 2013/12/13 06:00:09, J_lowe wrote:
I personally don't like these 'first person' question-type headings,
but if we
have to have them can we just remove the 'I'm new to LilyPond' and get
to the
point.
Something like 'Which dedicated LilyPond Editor' or similar.
I think in this case we *should* get to this colloquial style. It seems to be necessary because too many people have got this wrong, presumably because they didn't really bother to read. However I've switched to a "You" address. Done. https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi#newcode55 Documentation/web/download.itexi:55: If you don't already have an editing environment for LilyPond you should On 2013/12/13 06:00:09, J_lowe wrote:
You had already subheaded this section as '[I] ... don't have a
dedicated
editor...' so these first words are redundant. Just skip to the point.
"See @ref{Editing} to find a list of LilyPond editors. Frescobaldi or
Denemo are
recommended for beginners."
Done. https://codereview.appspot.com/40510046/diff/20001/Documentation/web/download.itexi#newcode64 Documentation/web/download.itexi:64: to install a new LilyPond version you can proceed as described below. On 2013/12/13 06:00:09, J_lowe wrote:
Is this section even needed?
I think yes. This should step in the way of a new user who'd (out of a habit) jump directly to the Download buttons. https://codereview.appspot.com/40510046/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel