Fixes #1171538 and #1171880

Fixes #1171538 and #1171880

- Dislpay the first 10 charms in each container on fullscreen view
- Clicks on charms from fullscreen open the fullscreen charm details
- Drive by: remove the link to the test file that no longer exists
- Update the svg code to work since the juju-gui svg was trying hard to show
up now.
- Add tests for the editorial code. Not sure how I didn't have those before.
- Remove sidebar-editorial json file no longer needed. Outdated api
- Added the interesting.json for the updated api calls for testing

https://code.launchpad.net/~rharding/juju-gui/fullscreen-1171538/+merge/160473

(do not edit description out of merge proposal)

[rharding, 2013-04-23 19:04:41]

Please take a look.

[rharding, 2013-04-23 19:07:27]

Please take a look.

[curtis, 2013-04-23 19:17:48]

Thank you for this fix. The image/svg+xml change is timely. Aaron and I
discussed support for svgz. It is a nice to have, but the charm needs to make
the mime-type clear. This change indicates we are trying to be explicit, though
we assume 'image/svg+xml' is the only type. LGTM.

[jeff.pihach, 2013-04-23 19:27:33]

LGTM with a few trivial comments

https://codereview.appspot.com/8923044/diff/3001/app/subapps/browser/views/ed...
File app/subapps/browser/views/editorial.js (right):

https://codereview.appspot.com/8923044/diff/3001/app/subapps/browser/views/ed...
app/subapps/browser/views/editorial.js:28: // How many of each charm container
do we show by default.
I'm wondering if these shouldn't be in an attribute

https://codereview.appspot.com/8923044/diff/3001/test/test_browser_app.js
File test/test_browser_app.js (right):

https://codereview.appspot.com/8923044/diff/3001/test/test_browser_app.js#new...
test/test_browser_app.js:152: app.destroy();
This should probably be wrapped in an if block just in case there is no app in a
future test.

https://codereview.appspot.com/8923044/diff/3001/test/test_browser_editorial.js
File test/test_browser_editorial.js (right):

https://codereview.appspot.com/8923044/diff/3001/test/test_browser_editorial....
test/test_browser_editorial.js:45: var fakeStore = new Y.juju.Charmworld0({});
Do these need to be destroyed?

[rharding, 2013-04-23 19:45:41]

Please take a look.

[rharding, 2013-04-23 19:56:17]

*** Submitted:

Fixes #1171538 and #1171880

- Dislpay the first 10 charms in each container on fullscreen view
- Clicks on charms from fullscreen open the fullscreen charm details
- Drive by: remove the link to the test file that no longer exists
- Update the svg code to work since the juju-gui svg was trying hard to show
up now.
- Add tests for the editorial code. Not sure how I didn't have those before.
- Remove sidebar-editorial json file no longer needed. Outdated api
- Added the interesting.json for the updated api calls for testing

R=curtis, jeff.pihach
CC=
https://codereview.appspot.com/8923044