add hangouts

add hangouts


BUG=
TEST=

[sethladd, 2012-05-22 04:48:05]

staged at http://hangouts.dart-lang.appspot.com/hangouts/

[kathyw, 2012-05-22 15:46:34]

Not sure about the format or titles... but it is nice to have the videos onsite.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/_includes/nav...
File src/site/_includes/nav.html (right):

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/_includes/nav...
src/site/_includes/nav.html:38: {% nav_link /hangouts/ %}Hangouts{% endnav_link
%}
Unless we intend to make this page more than a bunch of links to videos, this
would probably be clearer as "Hangout Videos".

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/dart...
File src/site/hangouts/dartisans-ep-5.markdown (right):

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/dart...
src/site/hangouts/dartisans-ep-5.markdown:3: title: "Dartisans Ep. 5 - Meet the
Dart VM team"
same title comments as in the hangout index

(this is really long for a page title! maybe we need the ability to specify a
subtitle?)

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
File src/site/hangouts/index.markdown (right):

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:3: title: "More Resources"
title should be "Hangout Videos"

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:7: # Hangout videos
-> Hangout Videos

(or use page.title)

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:9: This is a collection of videos about Dart
and its creators, recorded
"This is a collection of" -> "This page collects"

Will this have more than Dartisan hangouts? If not, we should probably mention
Dartisan in the first sentence.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:13: * [Dartisans Ep. 5 - Meet the Dart VM
team](dartisans-ep-5.html)
Let's not abbreviate "episode" -- either omit it or spell it out (in lowercase).

Also, " - " should probably be ": ". (" - " is fairly widely used, but it's not
standard English.)

So how about:

[Dartisans 5: Meet the Dart VM team]
...

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:17: * Dartisans Ep. 1 (lost to the bit bucket
in the sky)
Should we have dates and/or speakers listed here? That would give context and
improve searchability. Maybe this page should be more like the Presentations or
Articles page.

In fact... Maybe we shouldn't have a separate page for each video. Instead, have
the description here and have either a _blank link to the YouTube video or the
embedded video (in which case you'd probably want a TOC). Having the videos
visible on this site is nice, in that you don't lose people from the site. I
just don't know if having separate pages adds anything.

[sethladd, 2012-05-22 17:28:47]

please take another look.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/_includes/nav...
File src/site/_includes/nav.html (right):

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/_includes/nav...
src/site/_includes/nav.html:38: {% nav_link /hangouts/ %}Hangouts{% endnav_link
%}
On 2012/05/22 15:46:34, kathyw wrote:
> Unless we intend to make this page more than a bunch of links to videos, this
> would probably be clearer as "Hangout Videos".

Done.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/dart...
File src/site/hangouts/dartisans-ep-5.markdown (right):

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/dart...
src/site/hangouts/dartisans-ep-5.markdown:3: title: "Dartisans Ep. 5 - Meet the
Dart VM team"
will leave HTML title, but will split h1 into h1 and h2

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
File src/site/hangouts/index.markdown (right):

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:3: title: "More Resources"
On 2012/05/22 15:46:34, kathyw wrote:
> title should be "Hangout Videos"

Done.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:7: # Hangout videos
On 2012/05/22 15:46:34, kathyw wrote:
> -> Hangout Videos
> 
> (or use page.title)

Done.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:9: This is a collection of videos about Dart
and its creators, recorded
On 2012/05/22 15:46:34, kathyw wrote:
> "This is a collection of" -> "This page collects"
> 
> Will this have more than Dartisan hangouts? If not, we should probably mention
> Dartisan in the first sentence.

Done.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:13: * [Dartisans Ep. 5 - Meet the Dart VM
team](dartisans-ep-5.html)
On 2012/05/22 15:46:34, kathyw wrote:
> Let's not abbreviate "episode" -- either omit it or spell it out (in
lowercase).
> 
> Also, " - " should probably be ": ". (" - " is fairly widely used, but it's
not
> standard English.)
> 
> So how about:
> 
> [Dartisans 5: Meet the Dart VM team]
> ...

Done.

https://chromiumcodereview.appspot.com/10387230/diff/1/src/site/hangouts/inde...
src/site/hangouts/index.markdown:17: * Dartisans Ep. 1 (lost to the bit bucket
in the sky)
I like having a page per video, for analytics.

[kathyw, 2012-05-22 18:13:17]

Strike the "Ep." from each page's title, delete an apostrophe, and I think we're
good to go.

https://chromiumcodereview.appspot.com/10387230/diff/6001/src/site/hangouts/d...
File src/site/hangouts/dartisans-ep-5.markdown (right):

https://chromiumcodereview.appspot.com/10387230/diff/6001/src/site/hangouts/d...
src/site/hangouts/dartisans-ep-5.markdown:12: engineers. We learn about their
experience building VM's and what's

VM's -> VMs

(VM's probably isn't "wrong", but we don't use the ABC's form of pluralization
anywhere else. I think it isn't Google/common style.)

[kathyw, 2012-05-22 18:18:20]

lgtm