Cleaning up apps codelab

Cleaning up apps codelab

Adding proper navigation, rewriting introductions to flow better, 
covering JavaScript and AngularJS samples, and including new tabs for code snippets.

BUG=162992
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194252

[mkearney1, 2013-04-08 22:22:58]

Hey, Renato

Here's the codelab all cleaned up.

I'll send a link to the staged docs too.

It's a big review, but hopefully straight-forward enough. I went through the
labs several times, testing links, etc., so I'm hoping it's not too buggy.

Meggin

[Renato Mangini (chromium), 2013-04-09 17:32:32]

Great job, Meggin! Several comments along the content, but it is a solid change
list.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/static/css/site.css (right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/static/css/site.css:596: }
lgtm

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab.html (right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:1: <h1
id="get_ready_to_build_chrome_apps_">About Todo Codelab</h1>
I prefer a more generic title (About this codelab)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:10: </p>
"Once you've completed this codelab, ..."

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:20: </p>
This link is confusing: "Step 5" seems to be linking into the codelab. Can you
name the link "Create your first app tutorial"?

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:26: </p>
".... Other platforms can see installed apps on a new Chrome tab or by
navigating to chrome://extensions"

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html:7:
<li><p>Open your preferred text editor. We have an under-development <a
href="http://chrome-api.storage.googleapis.com/index.html">Chrome apps
plugin</a> for <a href="http://www.sublimetext.com">Sublime</a>.</p></li>
Can you change this plugin URL to the Youtube video?
http://www.youtube.com/watch?v=x_FTILqlbsg&hd=1

(the linked page is used to install the dev version of the plugin, not the
stable one)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html:36:
<p>Note: for the rest of this tutorial, we&#39;ll refer to the directory for the
cloned git repository as &lt;tutorial&gt;.</p>
I suggest sweep through the codelab and change "tutorial" to "codelab" to be
coherent.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab2_basic.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab2_basic.html:67: 
<p class="note"><b>Tip:</b>
  If you use the Sublime Text editor with <a href="youtubelink">our plugin</a>,
you can create these three files with a click (Chrome -> New App -> Hello World)
</p>

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:14: as
long as they are CSP compliant, can be used in a Chrome packaged app.
... Packaged App (upper case)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:22:
Chrome packaged apps don&#39;t enforce any specific framework or programming
style.
... Packaged Apps (upper case)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:62:
%7B%7BtodoText%7D%7D
&#123;&#123;todoText&#125;&#125;

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:96: <p
class="note"><b>Note:</b>  The ng-csp directive tells Angular to run in a
&quot;content security mode&quot;. You don&#39;t need this directive when using
Angular v1.1.0+. We&#39;ve included it here so that the sample works regardless
of the Angular version in use.</p>
<code>ng-csp</code>

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab5_data.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab5_data.html:179: <a
href="http://www.html5rocks.com/en/tutorials/file/filesystem/">HTML5
Drag-and-Drop API</a>.
http://www.html5rocks.com/en/tutorials/dnd/basics/

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab5_data.html:330:
<pre>
maybe add a Angular tab (no JavaScript) to be clear that nothing needs to change
on the JavaScript version?

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab5_data.html:341:
&#123;&#123;dropText&#125;&#125;
Needs tabs. The JavaScript version should be:

 <div id="dropText">
 </div>

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html:8:
<h2 id="the_event_page">The event page</h2>
Background page is another commonly used name. Should we mention this here?

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html:217:
The <a href="http://developer.chrome.com/trunk/apps/app_identity.html">identify
API</a> covered in lab 7 is still experimental.</p>
change text to something like "The next chapter covers a still experimental API.
If you don't want to play with experimental APIs, feel free to skip it - the
rest of the codelab is independent from it."

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html:11:
<p class="note"><b>Note:</b>
change class to "warning" instead of "note"

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html:66:
or <a
href="https://github.com/GoogleChrome/chrome-app-codelab/blob/master/lab7_useridentification/javascript/gapi_tasks.js">JavaScript
gapi_tasks.js</a>.
they are the same. You can link one only (and mention they are the same)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html:185:
<p>In <a href="app_codelab8_webresources.html">8 - Access Web Resources</a>,
This should be numbered "7" instead of "8". Please, also check the other pages.
It should be coherent with the side nav.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:65:
</pre>
As in the Angular snippet, needs some comments to identity where the inserted
code should go. For example:
<!-- right after the "dropText" div: -->
&lt;webview&gt;&lt;/webview&gt;
<!-- in the TODO template, right before the end of the li: -->
&lt;a style=&quot;display: none;&quot; href=&quot;&quot;&gt;(view url)&lt;/a&gt;

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:72:
<p>Set an appropriate width and height to the webview tag in
<code> on webview. Check other occurrences

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:73:
the style sheet (it has a zero size by default).
remove the "it has a zero size by default" because this was a bug and I'm almost
sure it is fixed now.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:87:
<p>We now only need to add a method to our controller:
add a method to our AngularJS controller or an event handler to our JavaScript
controller

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:188:
&lt;img style=&quot;max-height: 48px; max-width: 120px;&quot;&lt;/img&gt;
missing a ">" after img

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:423:
you should now have a thumbnail of every image URL dropped into the Todo list
app.</p>
Add "Notice that with this change we are not handling local images dropped from
the file manager. We leave this as a challenge for you."

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:432:
We are working on a library that will be much more robust and easier to use.</p>
Link to the apps-resource-loader, and change the text to reflect that the
library is stable to be used in a production project.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab_10_publishing.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab_10_publishing.html:62:
<p>Hint: if anyone wants to create a tool that lets users quickly create all the
required assets, developers would sincerely appreciate.</p>
I think this should be rephrased.

[mkearney, 2013-04-09 17:38:49]

Cool, thanks.

I'm waiting for gclient sync-- first one since blink, and then I can action
the review comments.

On a side note, have you sync'd since blink and did it take you a long time?

Meggin


On Tue, Apr 9, 2013 at 10:32 AM, <mangini@chromium.org> wrote:

>
> Great job, Meggin! Several comments along the content, but it is a solid
> change
> list.
>
>
> https://codereview.chromium.**org/13248004/diff/25001/**
>
chrome/common/extensions/docs/**static/css/site.css<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/static/css/site.css>
> File chrome/common/extensions/docs/**static/css/site.css (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
>
chrome/common/extensions/docs/**static/css/site.css#newcode596<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/static/css/site.css#newcode596>
> chrome/common/extensions/docs/**static/css/site.css:596: }
> lgtm
>
> https://codereview.chromium.**org/13248004/diff/25001/**
>
chrome/common/extensions/docs/**templates/articles/app_**codelab.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab.html>
> File chrome/common/extensions/docs/**templates/articles/app_**codelab.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab.html#newcode1<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab.html#newcode1>
> chrome/common/extensions/docs/**templates/articles/app_**codelab.html:1:
> <h1
> id="get_ready_to_build_chrome_**apps_">About Todo Codelab</h1>
> I prefer a more generic title (About this codelab)
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab.html#newcode10<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab.html#newcode10>
> chrome/common/extensions/docs/**templates/articles/app_**codelab.html:10:
> </p>
> "Once you've completed this codelab, ..."
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab.html#newcode20<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab.html#newcode20>
> chrome/common/extensions/docs/**templates/articles/app_**codelab.html:20:
> </p>
> This link is confusing: "Step 5" seems to be linking into the codelab.
> Can you name the link "Create your first app tutorial"?
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab.html#newcode26<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab.html#newcode26>
> chrome/common/extensions/docs/**templates/articles/app_**codelab.html:26:
> </p>
> ".... Other platforms can see installed apps on a new Chrome tab or by
> navigating to chrome://extensions"
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab1_setup.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab1_setup.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab1_setup.html#newcode7<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html#newcode7>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab1_setup.html:7:
> <li><p>Open your preferred text editor. We have an under-development <a
>
href="http://chrome-api.**storage.googleapis.com/index.**html<http://chrome-api.storage.googleapis.com/index.html>">Chrome
> apps
> plugin</a> for <a
> href="http://www.sublimetext.**com <http://www.sublimetext.com>
> ">Sublime</a>.</p></li>
> Can you change this plugin URL to the Youtube video?
>
http://www.youtube.com/watch?**v=x_FTILqlbsg&hd=1<http://www.youtube.com/watc...
>
> (the linked page is used to install the dev version of the plugin, not
> the stable one)
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab1_setup.html#newcode36<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html#newcode36>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab1_setup.html:36:
> <p>Note: for the rest of this tutorial, we&#39;ll refer to the directory
> for the cloned git repository as &lt;tutorial&gt;.</p>
> I suggest sweep through the codelab and change "tutorial" to "codelab"
> to be coherent.
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab2_basic.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab2_basic.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab2_basic.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab2_basic.html#newcode67<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab2_basic.html#newcode67>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab2_basic.html:67:
>
> <p class="note"><b>Tip:</b>
>   If you use the Sublime Text editor with <a href="youtubelink">our
> plugin</a>, you can create these three files with a click (Chrome -> New
> App -> Hello World)
> </p>
>
> https://codereview.chromium.**org/13248004/diff/25001/**
>
chrome/common/extensions/docs/**templates/articles/app_**codelab3_mvc.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**codelab3_mvc.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab3_mvc.html#newcode14<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html#newcode14>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab3_mvc.html:14:
> as long as they are CSP compliant, can be used in a Chrome packaged app.
> ... Packaged App (upper case)
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab3_mvc.html#newcode22<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html#newcode22>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab3_mvc.html:22:
> Chrome packaged apps don&#39;t enforce any specific framework or
> programming style.
> ... Packaged Apps (upper case)
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab3_mvc.html#newcode62<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html#newcode62>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab3_mvc.html:62:
> %7B%7BtodoText%7D%7D
> &#123;&#123;todoText&#125;&#**125;
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab3_mvc.html#newcode96<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html#newcode96>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab3_mvc.html:96:
> <p class="note"><b>Note:</b>  The ng-csp directive tells Angular to run
> in a &quot;content security mode&quot;. You don&#39;t need this
> directive when using Angular v1.1.0+. We&#39;ve included it here so that
> the sample works regardless of the Angular version in use.</p>
> <code>ng-csp</code>
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab5_data.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab5_data.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab5_data.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab5_data.html#newcode179<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab5_data.html#newcode179>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab5_data.html:179:
> <a
href="http://www.html5rocks.**com/en/tutorials/file/**filesystem/<http://www.html5rocks.com/en/tutorials/file/filesystem/>
> ">HTML5
> Drag-and-Drop API</a>.
>
http://www.html5rocks.com/en/**tutorials/dnd/basics/<http://www.html5rocks.co...
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab5_data.html#newcode330<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab5_data.html#newcode330>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab5_data.html:330:
> <pre>
> maybe add a Angular tab (no JavaScript) to be clear that nothing needs
> to change on the JavaScript version?
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab5_data.html#newcode341<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab5_data.html#newcode341>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab5_data.html:341:
> &#123;&#123;dropText&#125;&#**125;
> Needs tabs. The JavaScript version should be:
>
>  <div id="dropText">
>  </div>
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab6_lifecycle.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab6_lifecycle.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab6_lifecycle.html#**newcode8<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html#newcode8>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab6_lifecycle.html:8:
> <h2 id="the_event_page">The event page</h2>
> Background page is another commonly used name. Should we mention this
> here?
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab6_lifecycle.html#**newcode217<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html#newcode217>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab6_lifecycle.html:217:
> The <a
>
href="http://developer.chrome.**com/trunk/apps/app_identity.**html<http://developer.chrome.com/trunk/apps/app_identity.html>
> ">identify
> API</a> covered in lab 7 is still experimental.</p>
> change text to something like "The next chapter covers a still
> experimental API. If you don't want to play with experimental APIs, feel
> free to skip it - the rest of the codelab is independent from it."
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab7_useridentification.**html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab7_useridentification.**html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab7_useridentification.**html#newcode11<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html#newcode11>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab7_useridentification.**html:11:
> <p class="note"><b>Note:</b>
> change class to "warning" instead of "note"
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab7_useridentification.**html#newcode66<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html#newcode66>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab7_useridentification.**html:66:
> or <a
> href="https://github.com/**GoogleChrome/chrome-app-**
>
codelab/blob/master/lab7_**useridentification/javascript/**gapi_tasks.js<https://github.com/GoogleChrome/chrome-app-codelab/blob/master/lab7_useridentification/javascript/gapi_tasks.js>
> ">JavaScript
> gapi_tasks.js</a>.
> they are the same. You can link one only (and mention they are the same)
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab7_useridentification.**html#newcode185<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html#newcode185>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab7_useridentification.**html:185:
> <p>In <a href="app_codelab8_**webresources.html">8 - Access Web
> Resources</a>,
> This should be numbered "7" instead of "8". Please, also check the other
> pages. It should be coherent with the side nav.
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html#**newcode65<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html#newcode65>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html:65:
> </pre>
> As in the Angular snippet, needs some comments to identity where the
> inserted code should go. For example:
> <!-- right after the "dropText" div: -->
> &lt;webview&gt;&lt;/webview&**gt;
> <!-- in the TODO template, right before the end of the li: -->
> &lt;a style=&quot;display: none;&quot; href=&quot;&quot;&gt;(view
> url)&lt;/a&gt;
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html#**newcode72<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html#newcode72>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html:72:
> <p>Set an appropriate width and height to the webview tag in
> <code> on webview. Check other occurrences
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html#**newcode73<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html#newcode73>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html:73:
> the style sheet (it has a zero size by default).
> remove the "it has a zero size by default" because this was a bug and
> I'm almost sure it is fixed now.
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html#**newcode87<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html#newcode87>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html:87:
> <p>We now only need to add a method to our controller:
> add a method to our AngularJS controller or an event handler to our
> JavaScript controller
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html#**newcode188<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html#newcode188>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html:**188:
> &lt;img style=&quot;max-height: 48px; max-width:
> 120px;&quot;&lt;/img&gt;
> missing a ">" after img
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html#**newcode423<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html#newcode423>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html:**423:
> you should now have a thumbnail of every image URL dropped into the Todo
> list app.</p>
> Add "Notice that with this change we are not handling local images
> dropped from the file manager. We leave this as a challenge for you."
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab8_webresources.html#**newcode432<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html#newcode432>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab8_webresources.html:**432:
> We are working on a library that will be much more robust and easier to
> use.</p>
> Link to the apps-resource-loader, and change the text to reflect that
> the library is stable to be used in a production project.
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab_10_publishing.html<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab_10_publishing.html>
> File
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab_10_publishing.html
> (right):
>
> https://codereview.chromium.**org/13248004/diff/25001/**
> chrome/common/extensions/docs/**templates/articles/app_**
>
codelab_10_publishing.html#**newcode62<https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/docs/templates/articles/app_codelab_10_publishing.html#newcode62>
> chrome/common/extensions/docs/**templates/articles/app_**
> codelab_10_publishing.html:62:
> <p>Hint: if anyone wants to create a tool that lets users quickly create
> all the required assets, developers would sincerely appreciate.</p>
> I think this should be rephrased.
>
>
https://codereview.chromium.**org/13248004/<https://codereview.chromium.org/1...
>

[mkearney1, 2013-04-10 17:59:58]

Hopefully we are closer...

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/static/css/site.css (right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/static/css/site.css:596: }
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> lgtm

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab.html (right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:1: <h1
id="get_ready_to_build_chrome_apps_">About Todo Codelab</h1>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> I prefer a more generic title (About this codelab)

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:10: </p>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> "Once you've completed this codelab, ..."

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:20: </p>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> This link is confusing: "Step 5" seems to be linking into the codelab. Can you
> name the link "Create your first app tutorial"?

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab.html:26: </p>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> ".... Other platforms can see installed apps on a new Chrome tab or by
> navigating to chrome://extensions"

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html:7:
<li><p>Open your preferred text editor. We have an under-development <a
href="http://chrome-api.storage.googleapis.com/index.html">Chrome apps
plugin</a> for <a href="http://www.sublimetext.com">Sublime</a>.</p></li>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> Can you change this plugin URL to the Youtube video?
> http://www.youtube.com/watch?v=x_FTILqlbsg&hd=1
> 
> (the linked page is used to install the dev version of the plugin, not the
> stable one)

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html:36:
<p>Note: for the rest of this tutorial, we&#39;ll refer to the directory for the
cloned git repository as &lt;tutorial&gt;.</p>
I swept the codelab and changed references to 'tutorial'. But I also deleted
this sentence, as it's unnecessary. On 2013/04/09 17:32:32, Renato Mangini
(chromium) wrote:
> I suggest sweep through the codelab and change "tutorial" to "codelab" to be
> coherent.

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab2_basic.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab2_basic.html:67: 
I moved this up to start of lab, as it makes the most sense there.

On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> <p class="note"><b>Tip:</b>
>   If you use the Sublime Text editor with <a href="youtubelink">our
plugin</a>,
> you can create these three files with a click (Chrome -> New App -> Hello
World)
> </p>

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:14: as
long as they are CSP compliant, can be used in a Chrome packaged app.
We do not capitalize packaged apps throughout the docs-- it's not a proprietary
name, at least not yet.
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> ... Packaged App (upper case)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:22:
Chrome packaged apps don&#39;t enforce any specific framework or programming
style.
Same as above.

On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> ... Packaged Apps (upper case)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:62:
%7B%7BtodoText%7D%7D
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> &#123;&#123;todoText&#125;&#125;

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab3_mvc.html:96: <p
class="note"><b>Note:</b>  The ng-csp directive tells Angular to run in a
&quot;content security mode&quot;. You don&#39;t need this directive when using
Angular v1.1.0+. We&#39;ve included it here so that the sample works regardless
of the Angular version in use.</p>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> <code>ng-csp</code>

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab5_data.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab5_data.html:179: <a
href="http://www.html5rocks.com/en/tutorials/file/filesystem/">HTML5
Drag-and-Drop API</a>.
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> http://www.html5rocks.com/en/tutorials/dnd/basics/

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab5_data.html:330:
<pre>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> maybe add a Angular tab (no JavaScript) to be clear that nothing needs to
change
> on the JavaScript version?

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab5_data.html:341:
&#123;&#123;dropText&#125;&#125;
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> Needs tabs. The JavaScript version should be:
> 
>  <div id="dropText">
>  </div>

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html:8:
<h2 id="the_event_page">The event page</h2>
Above, I've said 'aka background script' for event page. I like this better than
background page. Also, I think it's ok to not repeat this every time. Yes,
familiar with background page might get confused, but I think we need to move
forward and be consistent.

On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> Background page is another commonly used name. Should we mention this here?

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab6_lifecycle.html:217:
The <a href="http://developer.chrome.com/trunk/apps/app_identity.html">identify
API</a> covered in lab 7 is still experimental.</p>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> change text to something like "The next chapter covers a still experimental
API.
> If you don't want to play with experimental APIs, feel free to skip it - the
> rest of the codelab is independent from it."

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html:11:
<p class="note"><b>Note:</b>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> change class to "warning" instead of "note"

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html:66:
or <a
href="https://github.com/GoogleChrome/chrome-app-codelab/blob/master/lab7_useridentification/javascript/gapi_tasks.js">JavaScript
gapi_tasks.js</a>.
I mentioned they are the same, but I kept both links there, as a developer will
take one path or the other in full, so it makes sense to have links in either
direction (even if content is the same).

On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> they are the same. You can link one only (and mention they are the same)

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab7_useridentification.html:185:
<p>In <a href="app_codelab8_webresources.html">8 - Access Web Resources</a>,
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> This should be numbered "7" instead of "8". Please, also check the other
pages.
> It should be coherent with the side nav.

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:65:
</pre>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> As in the Angular snippet, needs some comments to identity where the inserted
> code should go. For example:
> <!-- right after the "dropText" div: -->
> &lt;webview&gt;&lt;/webview&gt;
> <!-- in the TODO template, right before the end of the li: -->
> &lt;a style=&quot;display: none;&quot; href=&quot;&quot;&gt;(view
url)&lt;/a&gt;

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:72:
<p>Set an appropriate width and height to the webview tag in
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> <code> on webview. Check other occurrences

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:73:
the style sheet (it has a zero size by default).
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> remove the "it has a zero size by default" because this was a bug and I'm
almost
> sure it is fixed now.

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:87:
<p>We now only need to add a method to our controller:
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> add a method to our AngularJS controller or an event handler to our JavaScript
> controller

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:188:
&lt;img style=&quot;max-height: 48px; max-width: 120px;&quot;&lt;/img&gt;
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> missing a ">" after img

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:423:
you should now have a thumbnail of every image URL dropped into the Todo list
app.</p>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> Add "Notice that with this change we are not handling local images dropped
from
> the file manager. We leave this as a challenge for you."

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:432:
We are working on a library that will be much more robust and easier to use.</p>
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> Link to the apps-resource-loader, and change the text to reflect that the
> library is stable to be used in a production project.

Done.

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab_10_publishing.html
(right):

https://codereview.chromium.org/13248004/diff/25001/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab_10_publishing.html:62:
<p>Hint: if anyone wants to create a tool that lets users quickly create all the
required assets, developers would sincerely appreciate.</p>
I just deleted it-- it's not necessary.
On 2013/04/09 17:32:32, Renato Mangini (chromium) wrote:
> I think this should be rephrased.

[Renato Mangini (chromium), 2013-04-12 20:33:06]

Aside from a few nits, lgtm

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html
(right):

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html:7:
<li><p>Open your preferred text editor. We have an under-development <a
href="http://chrome-api.storage.googleapis.com/index.html">Chrome apps
plugin</a> for <a
href="http://www.youtube.com/watch?v=x_FTILqlbsg&hd=1">Sublime</a>.</p></li>
You should replace the chrome-api.storage.googleapis.com link by the youtube
one, not the sublimetext.com link.

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html
(right):

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:194:
</conteint>
typo. should be "content"

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/json/apps_sidenav.json (right):

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
chrome/common/extensions/docs/templates/json/apps_sidenav.json:49: "title":
"Todo App Codelab",
I'm not quite comfortable with this title in the sidenav. Can we have just
"Learn With a Codelab"?

[mkearney1, 2013-04-15 20:48:12]

Trying to commit now-- getting merge conflicts but hopefully will push past them
soon.

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html
(right):

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab1_setup.html:7:
<li><p>Open your preferred text editor. We have an under-development <a
href="http://chrome-api.storage.googleapis.com/index.html">Chrome apps
plugin</a> for <a
href="http://www.youtube.com/watch?v=x_FTILqlbsg&hd=1">Sublime</a>.</p></li>
On 2013/04/12 20:33:07, Renato Mangini (chromium) wrote:
> You should replace the http://chrome-api.storage.googleapis.com link by the
youtube
> one, not the http://sublimetext.com link.

Done.

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
File
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html
(right):

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
chrome/common/extensions/docs/templates/articles/app_codelab8_webresources.html:194:
</conteint>
On 2013/04/12 20:33:07, Renato Mangini (chromium) wrote:
> typo. should be "content"

Done.

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
File chrome/common/extensions/docs/templates/json/apps_sidenav.json (right):

https://codereview.chromium.org/13248004/diff/35003/chrome/common/extensions/...
chrome/common/extensions/docs/templates/json/apps_sidenav.json:49: "title":
"Todo App Codelab",
On 2013/04/12 20:33:07, Renato Mangini (chromium) wrote:
> I'm not quite comfortable with this title in the sidenav. Can we have just
> "Learn With a Codelab"?

Done.

[mkearney1, 2013-04-15 22:56:18]

Committed patchset #16 manually as r194252 (presubmit successful).