Remove log message not necessary (#16)

linagora · Jul 12, 2023 · 8366fa0 · 8366fa0
1 parent 6ff1827
commit 8366fa0
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 12 deletions.
diff --git a/lib/src/widgets/html_editor_widget_mobile.dart b/lib/src/widgets/html_editor_widget_mobile.dart
@@ -429,7 +429,6 @@ class _HtmlEditorWidgetMobileState extends State<HtmlEditorWidget> {
                               foreColor = document.queryCommandValue('foreColor');
                               backColor = document.queryCommandValue('hiliteColor');
                               if (!backColor) {
-                                 console.log("Browser not support queryCommandValue('hiliteColor')");
                                  backColor = document.queryCommandValue('backColor');
                               }
                               fontName = document.queryCommandValue('fontName');

diff --git a/lib/src/widgets/html_editor_widget_web.dart b/lib/src/widgets/html_editor_widget_web.dart
@@ -226,7 +226,6 @@ class _HtmlEditorWidgetWebState extends State<HtmlEditorWidget> {
        
         window.parent.addEventListener('message', handleMessage, false);
         document.onselectionchange = onSelectionChange;
-        console.log('done');
       
         function handleMessage(e) {
           if (e && e.data && e.data.includes("toIframe:")) {
@@ -321,12 +320,10 @@ class _HtmlEditorWidgetWebState extends State<HtmlEditorWidget> {
                 if (commandType === "hiliteColor") {
                   if (data["argument"] === null) {
                     if (!document.execCommand("hiliteColor", false)) {
-                      console.log("Browser not support document.execCommand('hiliteColor')");
                       document.execCommand("backColor", false);
                     }
                   } else {
                     if (!document.execCommand("hiliteColor", false, data["argument"])) {
-                      console.log("Browser not support document.execCommand('hiliteColor')");
                       document.execCommand("backColor", false, data["argument"]);
                     }
                   }
@@ -341,7 +338,6 @@ class _HtmlEditorWidgetWebState extends State<HtmlEditorWidget> {
               if (data["type"].includes("execSummernoteAPI")) {
                 var nameAPI = data["nameAPI"];
                 var value = data["value"];
-                console.log("nameAPI: " + nameAPI + " | value: " + value);
                 if (value === null) {
                   \$('#summernote-2').summernote(nameAPI);
                 } else {
@@ -370,7 +366,6 @@ class _HtmlEditorWidgetWebState extends State<HtmlEditorWidget> {
                       var texto;
                       var count = 0;
                       var value = data["case"];
-                      console.log(value);
                       var nodes = selected.nodes();
                       for (var i=0; i< nodes.length; ++i) {
                           if (nodes[i].nodeName == "#text") {
@@ -480,16 +475,13 @@ class _HtmlEditorWidgetWebState extends State<HtmlEditorWidget> {
             foreColor = document.queryCommandValue('foreColor');
             backColor = document.queryCommandValue('hiliteColor')
             if (!backColor) {
-               console.log("Browser not support queryCommandValue('hiliteColor')");
                backColor = document.queryCommandValue('backColor');
             }
             fontName = document.queryCommandValue('fontName');
           }
           const browserName = getBrowserName();
-          console.log('browserName: ' + browserName)
           if (browserName === "Firefox") {
             backColor = \$(focusNode.parentNode).css('background-color');
-            console.log('backColor-Firefox: ' + backColor);
           }
           var message = {
             'view': "$createdViewId", 

diff --git a/lib/src/widgets/toolbar_widget.dart b/lib/src/widgets/toolbar_widget.dart
@@ -131,7 +131,9 @@ class ToolbarWidgetState extends State<ToolbarWidget> {
   /// Updates the toolbar from the JS handler on mobile and the onMessage
   /// listener on web
   void updateToolbar(Map<String, dynamic> json) {
-    debugPrint('ToolbarWidget::updateToolbar::json: $json');
+    if (kDebugMode) {
+      print('ToolbarWidget::updateToolbar::json: $json');
+    }
     //get parent element
     String parentElem = json['style'] ?? '';
     //get font name
@@ -175,7 +177,9 @@ class ToolbarWidgetState extends State<ToolbarWidget> {
         _fontNameSelectedItem = 'sans-serif';
       });
     }
-    debugPrint('ToolbarWidget::updateToolbar::colorList: $colorList');
+    if (kDebugMode) {
+      print('ToolbarWidget::updateToolbar::colorList: $colorList');
+    }
     //update the fore/back selected color if necessary
     final foregroundColor = colorList[0];
     if (foregroundColor != null && foregroundColor.isNotEmpty) {

diff --git a/lib/utils/javascript_utils.dart b/lib/utils/javascript_utils.dart
@@ -35,7 +35,6 @@ class JavascriptUtils {
   static const String jsHandleUpdateBodyDirection = '''
     const nodeEditor = document.getElementsByClassName('note-editable')[0];
     const currentDirection = data['direction'];
-    console.log('currentDirection: ' + currentDirection);
     nodeEditor.style.direction = currentDirection.toString();
   ''';