Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Move "Destroy Service" link and icon into the inspector footer. #365

Merged
merged 1 commit into from Jun 4, 2014

Conversation

kadams54
Copy link
Contributor

@kadams54 kadams54 commented Jun 4, 2014

This ticket moves the link into the footer for the service inspector. This change is consistent with the ghost inspector, which already has the same link in the footer.

QA

  1. Drag a charm to the canvas and deploy it.
  2. Drag a second charm to the canvas, but don't deploy it.
  3. Click on the deployed service and verify that the "Destroy Service" link is anchored to the bottom of the inspector (expand some of the unit info so that you activate scrolling).
  4. Click on the ghost service and verify that the location and appearance of this link is consistent between the two different types of inspectors.
  5. Verify that the link and icon work as expected in both types of inspectors.

@makyo
Copy link
Contributor

makyo commented Jun 4, 2014

👍 QA okay!

@kadams54
Copy link
Contributor Author

kadams54 commented Jun 4, 2014

Thanks! :shipit:

@jujugui
Copy link
Contributor

jujugui commented Jun 4, 2014

Test PASSed.
Refer to this link for build results: http://ci.jujugui.org:8080/job/juju-gui/1169/

@jujugui
Copy link
Contributor

jujugui commented Jun 4, 2014

Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge

jujugui added a commit that referenced this pull request Jun 4, 2014
This ticket moves the link into the footer for the service inspector. This change is consistent with the ghost inspector, which already has the same link in the footer.

### QA

1. Drag a charm to the canvas and deploy it.
2. Drag a second charm to the canvas, but don't deploy it.
3. Click on the deployed service and verify that the "Destroy Service" link is anchored to the bottom of the inspector (expand some of the unit info so that you activate scrolling).
4. Click on the ghost service and verify that the location and appearance of this link is consistent between the two different types of inspectors.
5. Verify that the link and icon work as expected in both types of inspectors.
@jujugui jujugui merged commit 9f372d3 into juju:develop Jun 4, 2014
@kadams54 kadams54 deleted the inspector-footer branch June 5, 2014 13:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants