lp://staging/~gary/juju-gui/doc-cleanup

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://staging/juju-gui/experimental at revision 685

Juju GUI Hackers: Pending requested 2013-05-23

Diff: 667 lines (+263/-302)

4 files modified

CHANGES.yaml (+3/-3)
Makefile (+2/-2)
app/views/utils.js (+59/-92)
undocumented (+199/-205)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

685. By Gary Poster on 2013-05-23

add some documentation, remove unused code, and clean up doc generation.

684. By Richard Harding on 2013-05-23

Move the featureFlag processor into window scope.

- In order to check for feature flags the processing must be done before the
app starts
- Adds some tests to verify we parse the url correctly with our regex/etc.
- Update the app.js to not deal with feature flags. It's done in index.html.
- Update the simulateEvents to be triggered by the feature flag existing vs
the custom route.

R=bcsaller, gary.poster, jeff.pihach
CC=
https://codereview.appspot.com/9682046

683. By Jeff Pihach on 2013-05-22

Adds the new inspector widget into the tree

This is the new widget which will be used to render the
inspector and various other nested views for the coming
UI changes. No QA necessary as it has no client facing
changes in this branch.

R=bac, gary.poster
CC=
https://codereview.appspot.com/9673044

682. By Brad Crittenden on 2013-05-22

Disable building relations if charm is not loaded.

The potential endpoints for a service are not available until the charm is
loaded. This branch fixes a bug where an attempt was made to access the
non-existent endpoints and then blithely proceeded along. In addition, the UI
is changed so that the 'Build Relation' submenu is disabled if the charm is
not loaded (a proxy for the endpoints not being there) and the
'long-click-to-begin-relation-building' is also disabled.

The whole relation building experience is up for redesign. At that time we
may chose to do something that provides a better user experience, such as
showing a spinner until the service is fully available. For now, the actual
time the actions will be disabled should be a matter of a few seconds.

A feature flag is introduced and can be used with the sandbox to artifically
set the charm loading delay for testing. Use it as follows for a ten second
delay before the charm is reported as loaded:

http://localhost:8888/:flags:/charmLoadDelay=10000

R=teknico, frankban
CC=
https://codereview.appspot.com/9650043

681. By Benjamin Saller on 2013-05-22

Allow Multi URL fragments per namespace

Router can take combineFlags to its constructor or to parse/combine methods.
This allows configuration of how to process namespace collisions in the URL
(overwrite or merge). The router itself can dispatch multi URL fragments per
namespace as well now.

R=jeff.pihach, gary.poster
CC=
https://codereview.appspot.com/9621045

680. By Francesco Banconi on 2013-05-22

Improve CI test runner.

bin/test-charm refactoring.
Improved error handling. Ensure the environment is
destroyed when an error occurs during deployment.

Also added a way to reuse the juju environment
while coding/debugging.

Fixed the Firefox version used in Saucelabs.
Running with a specific Firefox version is still
required, but now we know why: the default Saucelab
version (11) is not supported by Juju GUI.

This branch also includes some lint/pep8 fixes.

R=gary.poster, teknico
CC=
https://codereview.appspot.com/9642043

679. By j.c.sackett on 2013-05-22

Replaces spinner.gif with spin.js spinner

This branch is the first part of standardizing our spinners in jujugui,
matching the ones used in the charmbrowser with the one when the canvas is
loading.

- Wraps a min version of spin.js in a YUI module.
- Uses that module in overlay-indicator.js

NB: This means there is a dupe of spin.js in the code tree. The spinner.gif is
still in the tree as well. I will do a follow up branch removing all of that,
but I wanted the user-facing bits working and solid before doing the cleanup.

R=jeff.pihach, matthew.scott
CC=
https://codereview.appspot.com/9095044

678. By Benji York on 2013-05-21

Set version back to unreleased and tweak the release instructions a little.

677. By Benji York on 2013-05-21

add changelog for 0.5.0 release

676. By Gary Poster on 2013-05-21

Fix assorted annoyances

- Make simulation turned off by default, but available if you use :flags:/simulateEvents in the URL.

- fix the sizing of the service page. we're going to replace these pages soon, but the experience should not be awful before then.

- switch the default charmworld URL to manage.charmworld.com. It is a better default (more reliable) than staging these days.

R=frankban, teknico
CC=
https://codereview.appspot.com/9593044