Fix ESLint warnings (pyscript#294)

woxtu · web-flow · commit 6f18a66d534e · 2022-05-09T15:16:25.000-05:00
* Use builtin methods

* Fix formatting
diff --git a/pyscriptjs/src/components/pybutton.ts b/pyscriptjs/src/components/pybutton.ts
@@ -12,25 +12,27 @@ export class PyButton extends BaseEvalElement {
     mount_name: string;
     constructor() {
         super();
-        this.defaultClass = ['p-2', 'text-white', 'bg-blue-600', 'border', 'border-blue-600','rounded']
+
+        this.defaultClass = ['p-2', 'text-white', 'bg-blue-600', 'border', 'border-blue-600', 'rounded'];
+
         if (this.hasAttribute('label')) {
             this.label = this.getAttribute('label');
         }
 
         // Styling does the same thing as class in normal HTML. Using the name "class" makes the style to malfunction
         if (this.hasAttribute('styling')) {
-             let klass = this.getAttribute('styling');
-            if (klass.trim() == ''){
-                this.class = this.defaultClass
-            }else{
-                klass = klass.trim()
-                const newClassArray = klass.split(' ');
+            const klass = this.getAttribute('styling').trim();
+            if (klass === '') {
+                this.class = this.defaultClass;
+            } else {
                 // trim each element to remove unnecessary spaces which makes the button style to malfunction
-                this.class = (() => {const concatenatedString = []; for (let i = 0; i < newClassArray.length; i++) {if (newClassArray[i].trim() !== '')(concatenatedString.push(newClassArray[i].trim()));} return concatenatedString;})();
+                this.class = klass
+                    .split(' ')
+                    .map(x => x.trim())
+                    .filter(x => x !== '');
             }
-        }
-        else {
-            this.class = this.defaultClass
+        } else {
+            this.class = this.defaultClass;
         }
     }
 
