[webgeom] spelling and better formating

Author: linev

ui5/geom/controller/GeomHierarchy.controller.js

@@ -463,7 +463,7 @@ sap.ui.define(['sap/ui/core/mvc/Controller',
             this.selectedStack = this.getStackByPath(this.fullModel, path);
       },
 
-      /** @summary Show special message insted of nodes hierarchy */
+      /** @summary Show special message instead of nodes hierarchy */
       showTextInBrowser(text) {
          let br = this.byId("treeTable");
          br.collapseAll();

ui5/geom/controller/GeomViewer.controller.js

@@ -8,7 +8,7 @@ sap.ui.define(['sap/ui/core/mvc/Controller',
 
    "use strict";
 
-   /** @summary Central geometry viewer contoller
+   /** @summary Central geometry viewer controller
     * @desc All TGeo functionality is loaded after main ui5 rendering is performed,
     * To start drawing, following stages should be completed:
     *    - ui5 element is rendered (onAfterRendering is called)
@@ -211,7 +211,7 @@ sap.ui.define(['sap/ui/core/mvc/Controller',
       },
 
       /** @summary function to accumulate and process all drawings messages
-       * @desc if not all scripts are loaded, messages are quied and processed later */
+       * @desc if not all scripts are loaded, messages are queued and processed later */
       checkDrawMsg(kind, msg) {
          if (kind) {
             if (!msg)
@@ -290,7 +290,7 @@ sap.ui.define(['sap/ui/core/mvc/Controller',
       onWebsocketMsg(handle, msg /*, offset */) {
 
          // binary data can be send only as addition to draw message
-         // here data can be placed in the queue and processed when all other prerequicities are done
+         // here data can be placed in the queue and processed when all other prerequisites are done
          if (typeof msg != "string")
             return console.error("Geom viewer do not uses binary messages len = " + mgs.byteLength);
 

ui5/geom/lib/ColorBox.js

@@ -14,7 +14,7 @@ sap.ui.define([
       },
 
       // the part creating the HTML:
-      renderer : function(oRm, oControl) { // static function, so use the given "oControl" instance instead of "this" in the renderer function
+      renderer(oRm, oControl) { // static function, so use the given "oControl" instance instead of "this" in the renderer function
          // if (!oControl.getVisible()) return;
 
          if (!oControl.getColor() || (oControl.getColor() == '#fff')) return;
@@ -31,7 +31,7 @@ sap.ui.define([
 
 /*
       // an event handler:
-      onclick : function(evt) {   // is called when the Control's area is clicked - no further event registration required
+      onclick(evt) {   // is called when the Control's area is clicked - no further event registration required
          sap.ui.require([
             'sap/ui/unified/ColorPickerPopover'
          ], function (ColorPickerPopover) {
@@ -45,8 +45,8 @@ sap.ui.define([
          }.bind(this));
       },
 
-      handleChange: function (oEvent) {
-         var newColor = oEvent.getParameter("colorString");
+      handleChange(oEvent) {
+         let newColor = oEvent.getParameter("colorString");
          this.setColor(newColor);
          // TODO: fire a "change" event, in case the application needs to react explicitly when the color has changed
          // but when the color is bound via data binding, it will be updated also without this event

ui5/geom/lib/GeomDrawing.js

@@ -12,7 +12,7 @@ sap.ui.define(['sap/ui/core/Control',
          }
       },
 
-      onBeforeRendering: function() {
+      onBeforeRendering() {
          // remove Canvas and painter from DOM
          if (this.geom_painter) {
             this.geom_painter.clear3dCanvas();
@@ -21,7 +21,7 @@ sap.ui.define(['sap/ui/core/Control',
       },
 
       // the part creating the HTML:
-      renderer : function(oRm, oControl) { // static function, so use the given "oControl" instance instead of "this" in the renderer function
+      renderer(oRm, oControl) { // static function, so use the given "oControl" instance instead of "this" in the renderer function
          oRm.write("<div");
          oRm.writeControlData(oControl);  // writes the Control ID and enables event handling - important!
          // oRm.addStyle("background-color", oControl.getColor());  // write the color property; UI5 has validated it to be a valid CSS color
@@ -34,7 +34,7 @@ sap.ui.define(['sap/ui/core/Control',
          oRm.write("</div>"); // no text content to render; close the tag
       },
 
-      onAfterRendering: function() {
+      onAfterRendering() {
          ResizeHandler.register(this, this.onResize.bind(this));
 
          if (this.geom_painter) {
@@ -54,7 +54,7 @@ sap.ui.define(['sap/ui/core/Control',
          }
       },
 
-      setGeomPainter: function(painter, skip_cleanup) {
+      setGeomPainter(painter, skip_cleanup) {
 
          if (this.geom_painter) {
             if (this.geom_skip_cleanup) {
@@ -74,15 +74,14 @@ sap.ui.define(['sap/ui/core/Control',
          }
       },
 
-      onResize: function() {
+      onResize() {
          if (this.resize_tmout) clearTimeout(this.resize_tmout);
          this.resize_tmout = setTimeout(this.onResizeTimeout.bind(this), 100); // minimal latency
       },
 
-      onResizeTimeout: function() {
+      onResizeTimeout() {
          delete this.resize_tmout;
-         if (this.geom_painter)
-            this.geom_painter.checkResize();
+         this.geom_painter?.checkResize();
       }
    });