Update the Style Guide

Update the Style Guide

Staged at:
https://sg-dot-dart-lang.appspot.com/articles/style-guide/

BUG= https://github.com/dart-lang/www.dartlang.org/issues/1237, https://github.com/dart-lang/www.dartlang.org/issues/1221, https://github.com/dart-lang/www.dartlang.org/issues/860
R=rnystrom@google.com

Committed: d312ffa75130ab440a88b6ecb9a4c60cc52e6a83

[Kathy Walrath, 2015-02-25 01:21:26]

kathyw@google.com changed reviewers:
+ rnystrom@google.com

[Bob Nystrom, 2015-02-26 00:19:53]

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
File src/site/articles/style-guide/index.markdown (right):

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
src/site/articles/style-guide/index.markdown:1063: whenTrue : whenFalse;
Let's make this one:

return someCondition
  ? whenTrue : whenFalse;

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
src/site/articles/style-guide/index.markdown:1067: {% endprettify %}
Also, maybe swap the order of the bad examples so they match the order of the
guidelines?

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
src/site/articles/style-guide/index.markdown:1625: we recommend always
specifying a library name.
We do?

[Kathy Walrath, 2015-02-26 16:15:14]

ptal

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
File src/site/articles/style-guide/index.markdown (right):

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
src/site/articles/style-guide/index.markdown:1063: whenTrue : whenFalse;
On 2015/02/26 00:19:52, Bob Nystrom wrote:
> Let's make this one:
> 
> return someCondition
>   ? whenTrue : whenFalse;

Done.

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
src/site/articles/style-guide/index.markdown:1067: {% endprettify %}
On 2015/02/26 00:19:52, Bob Nystrom wrote:
> Also, maybe swap the order of the bad examples so they match the order of the
> guidelines?

Done.

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
src/site/articles/style-guide/index.markdown:1625: we recommend always
specifying a library name.
On 2015/02/26 00:19:52, Bob Nystrom wrote:
> We do?

I was surprised, too, but that was in seanegan's suggested wording
(https://github.com/dart-lang/www.dartlang.org/issues/635#issuecomment-74520529),
which you said sounded good to you.

I've taken it out.

[Bob Nystrom, 2015-02-26 20:52:03]

LGTM, thanks!

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
File src/site/articles/style-guide/index.markdown (right):

https://chromiumcodereview.appspot.com/959553002/diff/1/src/site/articles/sty...
src/site/articles/style-guide/index.markdown:1625: we recommend always
specifying a library name.
On 2015/02/26 16:15:14, Kathy Walrath wrote:
> On 2015/02/26 00:19:52, Bob Nystrom wrote:
> > We do?
> 
> I was surprised, too, but that was in seanegan's suggested wording
>
(https://github.com/dart-lang/www.dartlang.org/issues/635#issuecomment-74520529),
> which you said sounded good to you.
> 
> I've taken it out. 

My reading comprehension was bad. :(

I like the rest of Sean's comments but I don't think there's much value in
encouraging library names. They're 90% pointless. When the language does require
them (because you import two unnamed libraries), it will let you know.

[Kathy Walrath, 2015-02-26 23:36:46]

Committed patchset #2 (id:20001) manually as
d312ffa75130ab440a88b6ecb9a4c60cc52e6a83 (presubmit successful).