From adbefb46226365c6d3ab09f559f5524e82d02059 Mon Sep 17 00:00:00 2001
From: KernelDeimos <eric.alex.dube@gmail.com>
Date: Mon, 6 May 2024 16:44:23 -0400
Subject: [PATCH] Cleanup log messages

---
 src/UI/Components/Flexer.js     | 3 ---
 src/UI/Components/QRCode.js     | 2 --
 src/UI/Components/StringView.js | 1 -
 src/UI/UIComponentWindow.js     | 1 -
 src/UI/UIWindow.js              | 2 --
 src/UI/UIWindow2FASetup.js      | 5 ++---
 src/UI/UIWindowLogin.js         | 3 ++-
 src/util/Component.js           | 2 --
 8 files changed, 4 insertions(+), 15 deletions(-)

diff --git a/src/UI/Components/Flexer.js b/src/UI/Components/Flexer.js
index 47b9b46c4..50deac895 100644
--- a/src/UI/Components/Flexer.js
+++ b/src/UI/Components/Flexer.js
@@ -27,14 +27,12 @@ export default class Flexer extends Component {
     }
 
     on_ready ({ listen }) {
-        console.log('Flexer on_ready called');
         for ( const child of this.get('children') ) {
             child.setAttribute('slot', 'inside');
             child.attach(this);
         }
 
         listen('gap', gap => {
-            console.log('gap called', gap);
             $(this.dom_).find('div').first().css('gap', gap);
         });
     }
@@ -45,6 +43,5 @@ export default class Flexer extends Component {
 if ( ! window.__component_flexer ) {
     window.__component_flexer = true;
 
-    console.log('this is here');
     customElements.define('c-flexer', Flexer);
 }
diff --git a/src/UI/Components/QRCode.js b/src/UI/Components/QRCode.js
index 3dfb9f1e1..859a629ed 100644
--- a/src/UI/Components/QRCode.js
+++ b/src/UI/Components/QRCode.js
@@ -40,9 +40,7 @@ export default class QRCodeView extends Component {
     }
 
     on_ready ({ listen }) {
-        console.log('QRCodeView on_ready called');
         listen('value', value => {
-            console.log('got value', value);
             // $(this.dom_).find('.qr-code').empty();
             new QRCode($(this.dom_).find('.qr-code').get(0), {
                 text: value,
diff --git a/src/UI/Components/StringView.js b/src/UI/Components/StringView.js
index 613341c3b..2ad4228ec 100644
--- a/src/UI/Components/StringView.js
+++ b/src/UI/Components/StringView.js
@@ -28,7 +28,6 @@ export default class StringView extends Component {
     on_ready ({ listen }) {
         // TODO: listener composition, to avoid this
         const either = ({ heading, text }) => {
-            console.log('---', { heading, text });
             const wrapper_nodeName = heading ? 'h' + heading : 'span';
             $(this.dom_).find('span').html(`<${wrapper_nodeName}>${
                 this.get('no_html_encode') ? text : html_encode(text)
diff --git a/src/UI/UIComponentWindow.js b/src/UI/UIComponentWindow.js
index ad2a1b647..3e8cf9306 100644
--- a/src/UI/UIComponentWindow.js
+++ b/src/UI/UIComponentWindow.js
@@ -21,7 +21,6 @@ export default async function UIComponentWindow (options) {
 
     options.component.attach(placeholder);
     options.component.focus();
-    console.log('UIComponentWindow', options.component);
 
     return win;
 }
diff --git a/src/UI/UIWindow.js b/src/UI/UIWindow.js
index ac86fe005..dbcbbf38d 100644
--- a/src/UI/UIWindow.js
+++ b/src/UI/UIWindow.js
@@ -2816,9 +2816,7 @@ $.fn.close = async function(options) {
             }
         }
 
-        console.log('deos ')
         if ( this.on_before_exit ) {
-            console.log('this happens??');
             if ( ! await this.on_before_exit() ) return false;
         }
 
diff --git a/src/UI/UIWindow2FASetup.js b/src/UI/UIWindow2FASetup.js
index 83b48a382..b76fafd95 100644
--- a/src/UI/UIWindow2FASetup.js
+++ b/src/UI/UIWindow2FASetup.js
@@ -115,8 +115,6 @@ const UIWindow2FASetup = async function UIWindow2FASetup () {
                         new CodeEntryView({
                             _ref: me => code_entry = me,
                             async [`property.value`] (value, { component }) {
-                                console.log('value? ', value)
-
                                 if ( ! await check_code_(value) ) {
                                     component.set('error', 'Invalid code');
                                     component.set('is_checking_code', false);
@@ -172,7 +170,8 @@ const UIWindow2FASetup = async function UIWindow2FASetup () {
     stepper.values_['done'].sub(value => {
         if ( ! value ) return;
         $(win).close();
-        console.log('WE GOT HERE')
+        // Write "2FA enabled" in green in the console
+        console.log('%c2FA enabled', 'color: green');
         promise.resolve(true);
     })
 
diff --git a/src/UI/UIWindowLogin.js b/src/UI/UIWindowLogin.js
index f60a8ab58..3b69cfeb4 100644
--- a/src/UI/UIWindowLogin.js
+++ b/src/UI/UIWindowLogin.js
@@ -227,7 +227,8 @@ async function UIWindowLogin(options){
                                             $(win).close();
                                             p.resolve();
                                         } catch (e) {
-                                            console.log('error object', e)
+                                            // keeping this log; useful in screenshots
+                                            console.log('2FA Login Error', e);
                                             component.set('error', i18n(error_i18n_key));
                                             component.set('is_checking_code', false);
                                         }
diff --git a/src/util/Component.js b/src/util/Component.js
index 6943c9643..115648ee5 100644
--- a/src/util/Component.js
+++ b/src/util/Component.js
@@ -56,7 +56,6 @@ export class Component extends HTMLElement {
             }
         }
         this.addEventListener('focus', () => {
-            console.log('got the event?');
             if ( this.on_focus ) {
                 this.on_focus();
             }
@@ -87,7 +86,6 @@ export class Component extends HTMLElement {
     }
 
     connectedCallback () {
-        console.log('connectedCallback called')
         this.on_ready && this.on_ready(this.get_api_());
     }