Change to always show the sensor controls. (#526)

sim/visuals/board.ts

@@ -466,7 +466,7 @@ namespace pxsim.visuals {
                     const isSelected = EV3View.isPreviousInputSelected(index, node.id) || view.getSelected();
                     if (isSelected && !view.getSelected()) view.setSelected(true);
                     const control = isSelected ? this.getControlForNode(node.id, index, !node.modeChange()) : undefined;
-                    const closeIcon = control ? this.getCloseIconView() : undefined;
+                    const closeIcon = control && view.hasClose() ? this.getCloseIconView() : undefined;
                     this.layoutView.setInput(index, view, control, closeIcon);
                     view.updateState();
                     if (control) control.updateState();

sim/visuals/controls/closeIcon.ts

@@ -17,7 +17,6 @@ namespace pxsim.visuals {
             return this.closeGroup;
         }
 
-
         buildDom(): SVGElement {
             this.content = svg.elt("svg", { width: "100%", height: "100%"}) as SVGSVGElement;
             this.content.appendChild(this.getInnerView());

sim/visuals/nodes/colorSensorView.ts

@@ -1,7 +1,7 @@
-/// <reference path="./moduleView.ts" />
+/// <reference path="./sensorView.ts" />
 
 namespace pxsim.visuals {
-    export class ColorSensorView extends ModuleView implements LayoutElement {
+    export class ColorSensorView extends SensorView implements LayoutElement {
 
         private control: ColorGridControl;
 

sim/visuals/nodes/gyroSensorView.ts

@@ -1,7 +1,7 @@
-/// <reference path="./moduleView.ts" />
+/// <reference path="./sensorView.ts" />
 
 namespace pxsim.visuals {
-    export class GyroSensorView extends ModuleView implements LayoutElement {
+    export class GyroSensorView extends SensorView implements LayoutElement {
 
         constructor(port: number) {
             super(GYRO_SVG, "gyro", NodeType.GyroSensor, port);

sim/visuals/nodes/infraredview.ts

@@ -1,7 +1,7 @@
 /// <reference path="./moduleView.ts" />
 
 namespace pxsim.visuals {
-    export class InfraredView extends ModuleView implements LayoutElement {
+    export class InfraredView extends SensorView implements LayoutElement {
 
         constructor(port: number) {
             super(INFRARED_SVG, "infrared", NodeType.InfraredSensor, port);

sim/visuals/nodes/sensorView.ts

@@ -0,0 +1,19 @@
+/// <reference path="./moduleView.ts" />
+
+namespace pxsim.visuals {
+
+    export abstract class SensorView extends ModuleView implements LayoutElement {
+
+        constructor(xml: string, prefix: string, id: NodeType, port: NodeType) {
+            super(xml, prefix, id, port);
+        }
+
+        public getSelected() {
+            return true;
+        }
+
+        public hasClose() {
+            return false;
+        }
+    }
+}

sim/visuals/nodes/ultrasonicView.ts

@@ -1,7 +1,7 @@
-/// <reference path="./moduleView.ts" />
+/// <reference path="./sensorView.ts" />
 
 namespace pxsim.visuals {
-    export class UltrasonicSensorView extends ModuleView implements LayoutElement {
+    export class UltrasonicSensorView extends SensorView implements LayoutElement {
 
         constructor(port: number) {
             super(ULTRASONIC_SVG, "ultrasonic", NodeType.UltrasonicSensor, port);

sim/visuals/view.ts

@@ -212,6 +212,10 @@ namespace pxsim.visuals {
             }
         }
 
+        public hasClose() {
+            return true;
+        }
+
         protected setChangedState() {
             this.changed = true;
         }

theme/blockly-toolbox.less

@@ -58,6 +58,11 @@ span.blocklyTreeLabel {
     border: 1px solid #ebebeb !important;
 }
 
+/* Monaco flyout */
+.monacoFlyout {
+    border: 0 !important;
+}
+
 /* Mobile */
 @media only screen and (max-width: @largestMobileScreen) {
     #blocklyTrashIcon {