[Code style] Cleaned out comments

open #250
open #274
scss, html scoured for commented
dead code, leftover TO-DOs and
other unnecessary comments;

platform/commonUI/browse/res/templates/browse/object-header.html

@@ -19,7 +19,6 @@
  this source code distribution or the Licensing information page available
  at runtime from the About dialog for additional information.
 -->
-<!--<div class='object-header'> Moved up to browse-object.html and frame.html. TO-DO: make sure this is Ok!-->
 <span class='type-icon ui-symbol flex-elem'>{{type.getGlyph()}}</span>
 <span class="l-elem-wrapper l-flex-row flex-elem grows">
     <span ng-if="parameters.mode" class='action flex-elem'>{{parameters.mode}}</span>
@@ -28,5 +27,4 @@
         key="'menu-arrow'"
         mct-object='domainObject'
         class="flex-elem"></mct-representation>
-</span>
+</span>
-<!--</div>-->

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

@@ -36,7 +36,6 @@
     .col {
         @include box-sizing(border-box);
         @include clearfix;
-        //		background: rgba(#ffcc00, 0.2);
         float: left;
         margin-left: $ueColMargin;
         padding-left: $interiorMargin;
@@ -97,9 +96,7 @@
 .l-flex-col {
     @include display-flex;
     @include flex-wrap(nowrap);
-    //@include flex(1 1 auto !important);
     .flex-elem {
-        //@include test(green, 0.1);
         min-height: 0; // Needed to allow element to shrink within parent
         position: relative;
         &:not(.grows) {

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

@@ -127,7 +127,6 @@ label.checkbox.custom {
         &:before {
             @include border-radius($basicCr * .75);
             background: $bg;
-            //border-bottom: 1px solid lighten($bg, 10%);
             @include box-shadow(inset rgba(black, 0.4) 0 1px 2px);
             box-sizing: border-box;
             content: " ";
@@ -202,7 +201,6 @@ label.checkbox.custom {
 
 /******************************************************** OBJECT-HEADER */
 .object-header {
-    //@include test();
     font-size: 1em;
 
     > .type-icon {
@@ -213,8 +211,6 @@ label.checkbox.custom {
     }
 
     .l-elem-wrapper {
-        //@include test(#66f, 0.2);
-        //@include justify-content(flex-start); // Commented out 11/6/15 open250
         mct-representation {
             // Holds the context-available item
             // Must have min-width to make flex work properly
@@ -228,19 +224,15 @@ label.checkbox.custom {
     }
 
     .title-label {
-        //@include test(green, 0.9);
         color: $colorObjHdrTxt;
         @include ellipsize();
-        //color: pushBack($colorBodyFg, 40%);
         @include webkitProp(flex, '0 1 auto');
         padding-right: 0.35em; // For context arrow. Done with em's so pad is relative to the scale of the text.
-        //position: relative;
     }
 
     .context-available {
         font-size: 0.7em;
         @include webkitProp(flex, '0 0 1');
-        //margin-right: $interiorMargin;
     }
 
     @include desktop {
@@ -303,7 +295,6 @@ label.checkbox.custom {
     @include border-radius($basicCr);
     @include boxIncised(0.3, 4px);
     background: $colorProgressBarOuter;
-    //border:1px solid $colorProgressBarOuter;
     .progress-amt {
         @include border-radius($basicCr);
         @include boxShdw();
@@ -343,8 +334,6 @@ label.checkbox.custom {
 .slider {
     $knobH: 100%; //14px;
     .slot {
-        //        @include border-radius($basicCr * .75);
-        //@include sliderTrack();
         width: auto;
         position: absolute;
         top: 0;
@@ -400,14 +389,11 @@ label.checkbox.custom {
     width: 230px;
     .l-month-year-pager {
         $pagerW: 20px;
-        //@include test();
-        //font-size: 0.8rem;
         height: $r1H;
         margin-bottom: $interiorMargin;
         position: relative;
         .pager,
         .val {
-            //@include test(red);
             @extend .abs;
         }
         .pager {
@@ -454,7 +440,6 @@ label.checkbox.custom {
         }
         li {
             @include flex(1 0);
-            //@include test();
             margin-left: 1px;
             padding: $interiorMargin;
             text-align: center;
@@ -512,8 +497,6 @@ label.checkbox.custom {
         @include background-image(linear-gradient(lighten($bg, $gr), $bg 20px));
         @include border-radius(2px);
         @include box-sizing(border-box);
-        //@include boxShdwSubtle();
-        //border-top: 1px solid lighten($bg, 20%);
         &:hover {
             @include background-image(linear-gradient(lighten($hc, $gr), $hc 20px));
         }

platform/commonUI/general/res/sass/helpers/_splitter.scss

@@ -43,7 +43,6 @@
         }
     }
     &:active {
-        //@include test();
         &:after {
             background-color: $colorSplitterActive !important;
         }
@@ -122,11 +121,3 @@
         }
     }
 }
-
-/*.browse-area .splitter {
-	top: 0; //$ueTopBarH + $interiorMarginLg;
-}
-
-.edit-area .splitter {
-	top: 0;
-}*/

platform/commonUI/general/res/sass/mobile/_layout.scss

@@ -46,7 +46,7 @@
 
     .create-btn-holder {
         // Hide the create button by default in mobile;
-        // This can be overridden by an example bundle to come
+        // This can be overridden by the examples/mobile bundle
         display: none;
     }
 
@@ -62,10 +62,6 @@
         // is hidden.
         .pane.left.treeview {
             @include trans-prop-nice(opacity, 150ms);
-            //right: 100% !important;
-            //width: auto !important;
-            //overflow-y: hidden;
-            //overflow-x: hidden;
             opacity: 0 !important;
         }
         .pane.right.items {
@@ -79,7 +75,6 @@
         // causing cut/copy/paste menu to
         // not appear. Should me moved in
         // future to properly work
-        //@include user-select(none);
 
         // Sets the left tree menu when the tree is shown.
         .pane.left.treeview {
@@ -107,7 +102,6 @@
     }
 
     .object-browse-bar {
-        //@include test();
         left: 45px !important;
         .context-available {
             opacity: 1 !important;
@@ -166,4 +160,4 @@
     .desktop-hide {
         display: none;
     }
-}
+}

platform/commonUI/general/res/sass/search/_search.scss

@@ -41,7 +41,6 @@
 		$textInputHeight: 19px; // This is equal to the default value, 19px
 		$iconEdgeM: 4px;
 		$iconD: $treeSearchInputBarH - ($iconEdgeM*2);
-        //@include test(red, 0.1);
 		font-size: 0.8em;
 		max-width: 250px;
 		position: relative;
@@ -128,7 +127,6 @@
 	}
 
 	.active-filter-display {
-		//order: 2;
 		$s: 0.65em;
 		$p: $interiorMargin;
 		@include box-sizing(border-box);

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

@@ -30,7 +30,6 @@
 		}
 	}
 	.object-top-bar {
-		//@include test(red);
 		font-size: 0.75em;
 		height: $ohH;
 		line-height: $ohH;

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

@@ -31,7 +31,7 @@
 }
 
 .contents {
-    $myM: 0; //$interiorMargin;
+    $myM: 0;
     box-sizing: border-box;
     position: absolute;
     top: $myM;
@@ -86,12 +86,10 @@
 		background: $colorFooterBg;
 		color: lighten($colorBodyBg, 30%);
 		font-size: .7rem;
-
 		.status-holder {
 			@include box-sizing(border-box);
 			@include absPosDefault($interiorMargin);
 			@include ellipsize();
-			//line-height: $ueFooterH - ($interiorMargin * 2);
 			right: 120px;
 			text-transform: uppercase;
 			z-index: 1;
@@ -138,7 +136,6 @@
     }
 
     .mini-tab-icon.toggle-pane {
-        //@include test(blue, 0.3);
         z-index: 5;
         @include desktop {
             $d: $uePaneMiniTabH;
@@ -250,7 +247,6 @@
 
 /***************************************************** OBJECT BROWSE BAR */
 .object-browse-bar {
-    //@include test(blue);
     @include absPosDefault(0, visible); // Must use visible to avoid hiding view switcher menu
     @include box-sizing(border-box);
     height: $ueTopBarH;
@@ -270,7 +266,6 @@
 // right representation.
 .pane-tree-hidden {
     // Sets the left tree menu when the tree is hidden.
-    //.pane.left.treeview,
     .tree-holder,
     .splitter-treeview,
     .holder-treeview-elements {
@@ -280,7 +275,6 @@
 
 .pane-tree-showing {
     // Sets the left tree menu when the tree is shown.
-    //.pane.left.treeview,
     .tree-holder,
     .splitter-treeview {
         @include trans-prop-nice(opacity, $dur: 250ms, $delay: 250ms);
@@ -320,7 +314,6 @@
     }
     .split-layout {
         .split-pane-component.pane {
-            //@include test(green);
             &.treeview.left {
                 min-width: $ueBrowseLeftPaneTreeMinW;
                 max-width: $ueBrowseLeftPaneTreeMaxW;

platform/features/imagery/res/templates/imagery.html

@@ -1,5 +1,4 @@
 <div class="t-imagery" ng-controller="ImageryController as imagery">
-    <!-- Main image -->
     <div
         class="l-image-main-wrapper"
         ng-mouseenter="showLocalControls = true;"

platform/search/res/templates/search.html

@@ -20,24 +20,16 @@
  at runtime from the About dialog for additional information.
 -->
 <div class="l-flex-col flex-elem grows holder holder-search"  ng-controller="SearchController as controller">
-    <!-- Search bar -->
     <div class="search-bar flex-elem" ng-controller="ClickAwayController as toggle">
-        <!-- Input field -->
         <input class="search-input"
                type="text"
                ng-model="ngModel.input"
                ng-keyup="controller.search()" />
-
-        <!-- Clear icon/button 'x' -->
         <a class="clear-icon"
            ng-class="{show: !(ngModel.input === '' || ngModel.input === undefined)}"
            ng-click="ngModel.input = ''; controller.search()"></a>
-
-        <!-- Menu icon/button 'v' -->
         <a class="menu-icon"
            ng-click="toggle.toggle()"></a>
-
-        <!-- Menu -->
         <mct-include key="'search-menu'"
                      class="menu-element search-menu-holder"
                      ng-class="{off: !toggle.isActive()}"
@@ -45,31 +37,19 @@
                      ng-click="toggle.setState(true)">
         </mct-include>
     </div>
-
-    <!-- Active filter display -->
     <div class="active-filter-display flex-elem"
          ng-class="{off: ngModel.filtersString === '' || ngModel.filtersString === undefined || !ngModel.search}"
          ng-controller="SearchMenuController as menuController">
-
         <a class="clear-icon clear-filters-icon"
            ng-click="ngModel.checkAll = true; menuController.checkAll()"></a>Filtered by: {{ ngModel.filtersString }}
-
     </div>
-
-    <!-- This div exists to determine scroll bar location -->
     <div class="search-results flex-elem grows vscroll"
          ng-class="{ off: !(loading || results.length > 0), loading: loading }">
-
-        <!--ng-hide="(ngModel.input === '' || ngModel.input === undefined)"-->
-
-        <!--<div class="hint" ng-show="results.length === 0">No results found</div>-->
-        <!--<div class="hint" ng-show="results.length > 0">Displaying {{ results.length }} result<span ng-show="results.length > 1">s</span></div>-->
         <mct-representation key="'search-item'"
                             ng-repeat="result in results"
                             mct-object="result.object"
                             ng-model="ngModel">
         </mct-representation>
-        <!-- Load more button -->
         <a class="load-more-button s-btn vsm" ng-if="controller.areMore()" ng-click="controller.loadMore()">More Results</a>
     </div>