Changed topo option labels to show current state

weaveworks · Sep 14, 2015 · 949105b · 949105b
1 parent a8246a6
commit 949105b
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/app/router.go b/app/router.go
@@ -128,6 +128,8 @@ func apiHandler(w http.ResponseWriter, r *http.Request) {
 	respondWith(w, http.StatusOK, APIDetails{Version: version})
 }
 
+// Topology option labels should tell the current state. The first item must
+// be the verb to get to that state
 var topologyRegistry = map[string]topologyView{
 	"applications": {
 		human:    "Applications",
@@ -144,17 +146,17 @@ var topologyRegistry = map[string]topologyView{
 		parent:   "",
 		renderer: render.ContainerWithImageNameRenderer{},
 		options: optionParams{"system": {
-			{"show", "Show system containers", false, nop},
-			{"hide", "Hide system containers", true, render.FilterSystem},
+			{"show", "System containers shown", false, nop},
+			{"hide", "System containers hidden", true, render.FilterSystem},
 		}},
 	},
 	"containers-by-image": {
 		human:    "by image",
 		parent:   "containers",
 		renderer: render.ContainerImageRenderer,
 		options: optionParams{"system": {
-			{"show", "Show system containers", false, nop},
-			{"hide", "Hide system containers", true, render.FilterSystem},
+			{"show", "System containers shown", false, nop},
+			{"hide", "System containers hidden", true, render.FilterSystem},
 		}},
 	},
 	"hosts": {