lp://staging/~gary/juju-gui/syncMenu

Branch merges

Propose for merging

No branches dependent on this one.

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

Juju GUI Hackers: Pending requested 2013-09-17

Diff: 185 lines (+28/-48)

5 files modified

app/index.html (+4/-6)
app/views/environment.js (+5/-0)
app/views/topology/service.js (+13/-25)
test/test_environment_view.js (+5/-5)
test/test_service_module.js (+1/-12)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

1052. By Gary Poster on 2013-09-17

remove toggleServiceMenu because we do not need it anymore.

1051. By Gary Poster on 2013-09-17

relation menu should open and close with inspector

1050. By Gary Poster on 2013-09-17

Inspector shows up after ghost inspector

- Per UX, after ghost inspector finishes, real inspector starts
- Changed new service creation to not bounce the box around
- Made a couple of other small tweaks for sandbox edge cases and capitalization

R=matthew.scott
CC=
https://codereview.appspot.com/13246050

1049. By Richard Harding on 2013-09-17

Support for show/hide in viewlets, resize config

- Fixes #1221665
- Add support in viewlets for custom show/hide methods. Provide a default that
does the current work of showing/hiding the container.
- Add a custom show method in the ghost config and service config viewlets to
make sure that once the config textareas are visible (container.show is run)
that we force a resize in order to show them at their proper height.
- Add a quick check that we update the resizing clone node when data binding
updates a value in a resizing config field.

QA:

- Follow the instructions for the bug listed above. Deploy liferay and note
that in both the ghost and the inspector config views that the java-opts field
is larger than default and shows all content.

R=jeff.pihach
CC=
https://codereview.appspot.com/13366051

1048. By Jeff Pihach on 2013-09-16

Adds service name valid indicator

When deploying a charm the service name input box will
now show you a checkmark or x depending on other services
currently deployed.

R=gary.poster
CC=
https://codereview.appspot.com/13253056

1047. By Jeff Pihach on 2013-09-16

Remove ghost CSS duplication

Much of the CSS required for the inspector layout was duplicated for the ghost
inspector, even though the presentation is the same.

The result of this branch should be that the ghost inspector looks exactly the
same, but now with less code and less duplication and fragmentation for future
updates.

(Jeff - Landing for Huw with small fix)
Original: https://codereview.appspot.com/13715044

R=
CC=
https://codereview.appspot.com/13571046

1046. By Richard Harding on 2013-09-16

Add Go support for removing units from services.

- This was missed during the transition to the go backend
- Add the Go handler method to removing a unit.
- Add a test and a note that the equiv. python test is located in another
file.

R=gary.poster
CC=
https://codereview.appspot.com/13349047

1045. By Benji York on 2013-09-16

Factor out charmworld API helpers and rename

These changes are part of the move to adding v3 of the charmworld API.

R=bac, rharding
CC=
https://codereview.appspot.com/13592047

1044. By Gary Poster on 2013-09-16

Add message if no relations exist

R=
CC=
https://codereview.appspot.com/13253054

1043. By Gary Poster on 2013-09-13

Trivial spelling fix

R=
CC=
https://codereview.appspot.com/13250048