code review 6816072: go.talks/present: separate actions from slide and artic...

go.talks/present: separate actions from slide and article templates

Also includes some CSS cleanups, and adds the ability to edit code
in articles.

[adg, 2012-11-01 15:53:43]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go.talks

[francesc, 2012-11-01 16:00:08]

On 2012/11/01 15:53:43, adg wrote:
> Hello mailto:golang-dev@googlegroups.com,
> 
> I'd like you to review this change to
> https://code.google.com/p/go.talks

What if one day we decide that the playground should look different in slides
and articles, or any new format we may create?

I don't think having some repetition is harmful in this case.

[adg, 2012-11-01 16:35:42]

When you template.ParseFiles("a", "b"), a block that's defined in "a" can
be redefined in "b", which will replace the block from "a". So if in the
future we want to define custom action blocks for a specific template, we
can just do it in that specific template file.


On 2 November 2012 03:00, <campoy@golang.org> wrote:

> On 2012/11/01 15:53:43, adg wrote:
>
>> Hello mailto:golang-dev@**googlegroups.com <golang-dev@googlegroups.com>,
>>
>
>  I'd like you to review this change to
>> https://code.google.com/p/go.**talks <https://code.google.com/p/go.talks>
>>
>
> What if one day we decide that the playground should look different in
> slides and articles, or any new format we may create?
>
> I don't think having some repetition is harmful in this case.
>
>
https://codereview.appspot.**com/6816072/<https://codereview.appspot.com/6816...
>

[francesc, 2012-11-01 16:57:46]

On 2012/11/01 16:35:42, adg wrote:
> When you template.ParseFiles("a", "b"), a block that's defined in "a" can
> be redefined in "b", which will replace the block from "a". So if in the
> future we want to define custom action blocks for a specific template, we
> can just do it in that specific template file.
> 
> 
> On 2 November 2012 03:00, <mailto:campoy@golang.org> wrote:
> 
> > On 2012/11/01 15:53:43, adg wrote:
> >
> >> Hello mailto:golang-dev@**googlegroups.com
<mailto:golang-dev@googlegroups.com>,
> >>
> >
> >  I'd like you to review this change to
> >> https://code.google.com/p/go.**talks <https://code.google.com/p/go.talks>
> >>
> >
> > What if one day we decide that the playground should look different in
> > slides and articles, or any new format we may create?
> >
> > I don't think having some repetition is harmful in this case.
> >
> >
>
https://codereview.appspot.**com/6816072/%3Chttps://codereview.appspot.com/68...>
> >

I didn't know about that. If that's the case: LGTM

[adg, 2012-11-01 17:02:23]

*** Submitted as
http://code.google.com/p/go/source/detail?r=c232a1e8b8f9&repo=talks ***

go.talks/present: separate actions from slide and article templates

Also includes some CSS cleanups, and adds the ability to edit code
in articles.

R=golang-dev, campoy
CC=golang-dev
http://codereview.appspot.com/6816072