lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Web: Download: Add introductory text (issue 40510046)


From: lilyliska
Subject: Re: Web: Download: Add introductory text (issue 40510046)
Date: Fri, 13 Dec 2013 10:59:33 +0000

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/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]