Fix a few small issues with map-tile attributes

Clean up map-feature connectedCallback (remove unused bind)

Author: Peter Rushforth

src/map-feature.js

@@ -166,7 +166,6 @@ export class HTMLFeatureElement extends HTMLElement {
   constructor() {
     // Always call super first in constructor
     super();
-    this._calculatePosition = calculatePosition.bind(this);
   }
 
   connectedCallback() {

src/map-tile.js

@@ -18,32 +18,41 @@ export class HTMLTileElement extends HTMLElement {
   /* jshint ignore:end */
   get row() {
     /* jshint ignore:start */
-    return +this.#initialRow;
+    return this.#hasConnected ? +this.#initialRow : +this.getAttribute('row');
     /* jshint ignore:end */
   }
   set row(val) {
+    /* jshint ignore:start */
+    if (this.#hasConnected) return; // Ignore after connection
+    /* jshint ignore:end */
     var parsedVal = parseInt(val, 10);
     if (!isNaN(parsedVal)) {
       this.setAttribute('row', parsedVal);
     }
   }
   get col() {
     /* jshint ignore:start */
-    return +this.#initialCol;
+    return this.#hasConnected ? +this.#initialCol : +this.getAttribute('col');;
     /* jshint ignore:end */
   }
   set col(val) {
+    /* jshint ignore:start */
+    if (this.#hasConnected) return; // Ignore after connection
+    /* jshint ignore:end */
     var parsedVal = parseInt(val, 10);
     if (!isNaN(parsedVal)) {
       this.setAttribute('col', parsedVal);
     }
   }
   get zoom() {
     /* jshint ignore:start */
-    return this.#initialZoom;
+    return this.#hasConnected ? +this.#initialZoom : +this.getAttribute('zoom');;
     /* jshint ignore:end */
   }
   set zoom(val) {
+    /* jshint ignore:start */
+    if (this.#hasConnected) return; // Ignore after connection
+    /* jshint ignore:end */
     var parsedVal = parseInt(val, 10);
     if (!isNaN(parsedVal) && parsedVal >= 0 && parsedVal <= 25) {
       this.setAttribute('zoom', parsedVal);
@@ -75,13 +84,10 @@ export class HTMLTileElement extends HTMLElement {
       switch (name) {
         case 'row':
           return String(this.#initialRow); /* jshint ignore:line */
-          break;
         case 'col':
           return String(this.#initialCol); /* jshint ignore:line */
-          break;
         case 'zoom':
           return String(this.#initialZoom); /* jshint ignore:line */
-          break;
       }
     }
     return super.getAttribute(name);