[Fronted] Minor final sanding and polishing

WTD-839
.t-view-switcher > .view-switcher;
Added CSS transition to hide/show of view-switcher;
Moved .view-switcher hide/show to _frame.scss;
Tweaked .frame:hover to raise z-index to show complete border;

platform/commonUI/general/res/sass/_constants.scss

@@ -145,3 +145,6 @@ $imageThumbPad: 1px;
 // Bubbles
 $bubbleArwSize: 10px;
 $bubblePad: $interiorMargin;
+
+// Timing
+$controlFadeMs: 100ms;

platform/commonUI/general/res/sass/_hide-non-functional.scss

@@ -36,7 +36,7 @@
 			// Don't pad in from top and bottom
 			//top: 0; bottom: 0;
 			.object-browse-bar {
-				.t-btn.key-window {
+				.btn.key-window {
 					// Hide the Open in New Window button
 					display: none;
 				}

platform/commonUI/general/res/sass/controls/_controls.scss

@@ -275,28 +275,28 @@ label.checkbox.custom {
 	$p: $interiorMarginSm * 2;
 	$c: $colorBodyFg;
 	@include btnSubtle($colorBodyBg);
-/*	height: $h;
-	line-height: $h;
-	&.dropdown {
-		padding-left: $p;
-		padding-right: $p;
-	}*/
+	/*	height: $h;
+		line-height: $h;
+		&.dropdown {
+			padding-left: $p;
+			padding-right: $p;
+		}*/
 
 	&:not(.disabled):hover {
 		color: lighten($c, 20%);
 	}
 
-/*	&.context-available {
-		// An element like the invoke-menu triangle;
-		// Indicates that this element has a dropdown menu available;
-		// Currently unused
-		$c: $colorKey;
-		color: $c;
-		padding: 0 5px;
-		&:hover {
-			color: lighten($c, 10%);
-		}
-	}*/
+	/*	&.context-available {
+			// An element like the invoke-menu triangle;
+			// Indicates that this element has a dropdown menu available;
+			// Currently unused
+			$c: $colorKey;
+			color: $c;
+			padding: 0 5px;
+			&:hover {
+				color: lighten($c, 10%);
+			}
+		}*/
 
 	span.l-click-area {
 		// In markup, this element should not enclose anything.
@@ -348,6 +348,10 @@ label.checkbox.custom {
 	}
 }
 
+.view-switcher {
+	@include trans-prop-nice-fade($controlFadeMs);
+}
+
 /******************************************************** OBJECT-HEADER */
 .object-header {
 	display: inline-block;
@@ -374,37 +378,6 @@ label.checkbox.custom {
 	}
 }
 
-/******************************************************** VIEW-CONTROLS */
-
-/*.view-controls .view-type {
-// UNUSED?
-	$d: 20px;
-	$p: 5px;
-	@include border-radius($controlCr);
-	box-sizing: border-box;
-	display: inline-block;
-	margin-left: $interiorMargin;
-	height: $d;
-	line-height: $d;
-	padding-left: $p;
-	padding-right: $p;
-	&.cur {
-		background: lighten($colorBodyBg, $ltGamma);
-	}
-}*/
-
-.frame .t-view-switcher {
-	// Hide the name when the view switcher is in a frame context
-	.name {
-		display: none;
-	}
-}
-
-.edit-mode .top-bar .control-set.edit-view-controls {
-	//	Used in templates/edit-view-controls.html
-	margin-right: $interiorMargin * 10;
-}
-
 /******************************************************** SLIDERS */
 
 .slider {

platform/commonUI/general/res/sass/user-environ/_frame.scss

@@ -27,6 +27,7 @@
 		border: 1px solid $bc;
 		&:hover {
 			border-color: lighten($bc, 10%);
+			z-index: 2;
 		}
 		.contents {
 //			overflow: hidden;
@@ -46,6 +47,26 @@
 		bottom: $myM;
 		left: $myM;
 	}
+	&.frame-template {
+		// Hide the view switcher by default when it's in an element that's in a frame context
+		// Frame template is used because we need to target the lowest nested frame
+		// This has the effect of hiding the view switcher in nested frames in edit mode, which is desirable currently (as it's non-functional)
+		.view-switcher {
+			//display: none;
+			opacity: 0;
+		}
+		&:hover .view-switcher {
+			// Show the view switcher on frame hover
+			//display: inline-block !important;
+			opacity: 1;
+		}
+	}
+	.view-switcher {
+		// Hide the name when the view switcher is in a frame context
+		.name {
+			display: none;
+		}
+	}
 }
 
 .edit-main .frame.child-frame.panel {
@@ -53,5 +74,4 @@
 		border-color: $colorKey;
 		@include boxShdwLarge();
 	}
-
 }