fix: qa issues

Author: HardeepAsrani

classes/Visualizer/Render/Layout.php

@@ -1049,7 +1049,7 @@ class="dashicons dashicons-lock"></span></h2>
 											<?php
 											$is_wp_source = 'visualizer_source_query_wp' === $source_of_chart;
 											$hours        = get_post_meta( $chart_id, Visualizer_Plugin::CF_DB_SCHEDULE, true );
-											$bttn_label   = $is_wp_source ? __( '1. Modify Data Source', 'visualizer' ) : __( '1. Choose Data Source', 'visualizer' );
+											$bttn_label   = $is_wp_source ? '1. ' . __( 'Modify Data Source', 'visualizer' ) : '1. ' . __( 'Choose Data Source', 'visualizer' );
 											?>
 
 											<!-- Step 1: Choose / modify data source -->
@@ -1058,7 +1058,7 @@ class="dashicons dashicons-lock"></span></h2>
 											<!-- Step 2: Sync schedule toggle -->
 											<div id="vz-wp-sync-step">
 												<button type="button" id="vz-wp-sync-btn" class="vz-import-step-toggle" aria-expanded="<?php echo $is_wp_source ? 'true' : 'false'; ?>">
-													<?php _e( '2. Set Sync Schedule', 'visualizer' ); ?>
+													<?php echo '2. ' . __( 'Set Sync Schedule', 'visualizer' ); ?>
 													<span class="dashicons dashicons-arrow-down-alt2"></span>
 												</button>
 												<div id="vz-wp-sync-options"<?php echo $is_wp_source ? '' : ' style="display:none"'; ?>>
@@ -1218,7 +1218,7 @@ class="dashicons dashicons-lock"></span></h2>
 									<!-- Step 2: Sync schedule toggle -->
 									<div id="vz-db-sync-step">
 										<button type="button" id="vz-db-sync-btn" class="vz-import-step-toggle" aria-expanded="<?php echo $is_db_source ? 'true' : 'false'; ?>">
-											<?php _e( '2. Set Sync Schedule', 'visualizer' ); ?>
+											<?php echo '2. ' . __( 'Set Sync Schedule', 'visualizer' ); ?>
 											<span class="dashicons dashicons-arrow-down-alt2"></span>
 										</button>
 										<div id="vz-db-sync-options"<?php echo $is_db_source ? '' : ' style="display:none"'; ?>>

js/frame.js

@@ -371,6 +371,8 @@
     function init_filter_import() {
         $( '#db-filter-save-button' ).on( 'click', function(){
             $('#vz-filter-wizard').submit();
+            $( '#vz-wp-sync-options' ).hide();
+            $( '#vz-wp-sync-btn' ).attr( 'aria-expanded', false );
         });
     }
 
@@ -452,11 +454,20 @@
             $( this ).attr( 'aria-expanded', ! isOpen );
         } );
 
+        $( document ).on( 'click', '#vz-wp-sync-btn', function(){
+            var $options = $( '#vz-wp-sync-options' );
+            var isOpen   = $options.is( ':visible' );
+            $options.toggle();
+            $( this ).attr( 'aria-expanded', ! isOpen );
+        } );
+
         $( '#db-chart-save-button' ).on( 'click', function(){
             // submit only if a query has been provided.
             if($('#db-query-form .visualizer-db-query').val().length > 0){
                 $('#viz-db-wizard-params').val($('#db-query-form').serialize());
                 $('#vz-db-wizard').submit();
+                $( '#vz-db-sync-options' ).hide();
+                $( '#vz-db-sync-btn' ).attr( 'aria-expanded', false );
             }else{
                 $('#canvas').unlock();
             }