fix: improve tooltip button generation and remove unnecessary console…

… logs
bluewave-labs · Jan 8, 2025 · ac2cef4 · ac2cef4
1 parent 4af6604
commit ac2cef4
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/jsAgent/hint.js b/jsAgent/hint.js
@@ -9,16 +9,20 @@ bw.hint = {
 
         for (let i = 0; i < hintData.length; i++) {
             const item = hintData[i];
-            console.log(item);
 
             const tooltip = bw.hint.generateTooltip(item);
             const header = bw.hint.generateHeader(item);
 
             const contentContainer = bw.hint.generateContentContainer(item);
             const content = bw.hint.generateContent(item);
-            const button = bw.hint.generateButton(item);
+
             contentContainer.appendChild(content);
-            contentContainer.appendChild(button);
+
+            if(item.action !== 'no action'){
+                const button = bw.hint.generateButton(item);
+                contentContainer.appendChild(button);
+            }
+
 
             tooltip.appendChild(header);
             tooltip.appendChild(contentContainer);
@@ -184,8 +188,7 @@ bw.hint = {
 
         button.addEventListener('click', () => {
             if(btnEvent == 'no action'){
-                //bw.hint.hideModal();
-                console.log('no action');
+
             }
             else if(btnEvent == 'open url'){
                 location.href = item.actionButtonUrl;

diff --git a/jsAgent/popup.js b/jsAgent/popup.js
@@ -55,7 +55,7 @@ bw.popup = {
                     ${bw.popup.addHeader(option.header, option.headerBackgroundColor, option.headerColor, option.padding)}
                     <div class="modal-body" style='padding: ${option.padding}px ${option.padding}px; display: flex; justify-content: space-between; flex-direction: column; box-sizing: border-box; font-family: "Inter", sans-serif; font-size: 13px; min-height: 227px; '>
                         ${option.content}
-                        ${bw.popup.addButton(option.actionButtonText, option.buttonBackgroundColor, option.buttonTextColor, option.padding, `bw-popup-btn`)}
+                        ${option.closeButtonAction !== 'no action' ?  bw.popup.addButton(option.actionButtonText, option.buttonBackgroundColor, option.buttonTextColor, option.padding, `bw-popup-btn`): ''}
                     </div>
                 </div>
             </div>`;