Merge pull request #664 from contactashish13/issue-254-pro

Open headers tab by default if credentials are saved

Author: rodica-andronache

classes/Visualizer/Render/Layout.php

@@ -148,6 +148,9 @@ public static function _renderJsonScreen( $args ) {
 			$headers['method'] = 'get';
 		}
 		$methods = apply_filters( 'visualizer_json_request_methods', array( 'GET', 'POST' ) );
+
+		// open the headers by default?
+		$headers_open = $headers && array_key_exists( 'auth', $headers ) && ( array_key_exists( 'username', $headers['auth'] ) && ! empty( $headers['auth']['username'] ) ) || ( ! empty( $headers['auth'] ) && is_string( $headers['auth'] ) );
 		?>
 		<div id="visualizer-json-screen" style="display: none">
 			<div class="visualizer-json-form">
@@ -170,7 +173,7 @@ class="visualizer-input json-form-element">
 						<button class="button button-secondary button-small" id="visualizer-json-fetch"><?php esc_html_e( 'Fetch Endpoint', 'visualizer' ); ?></button>
 
 						<div class="visualizer-json-subform">
-							<h3 class="viz-substep"><?php _e( 'Headers', 'visualizer' ); ?></h3>
+							<h3 class="viz-substep <?php echo $headers_open ? 'open' : ''; ?>"><?php _e( 'Headers', 'visualizer' ); ?></h3>
 							<div class="json-wizard-headers">
 								<div class="json-wizard-header">
 									<div><?php _e( 'Request Type', 'visualizer' ); ?></div>
@@ -213,7 +216,7 @@ class="json-form-element">
 											type="text"
 											id="vz-import-json-auth"
 											name="auth"
-											value="<?php echo ( ! empty( $headers ) && array_key_exists( 'auth', $headers ) ? $headers['auth']['auth'] : '' ); ?>"
+											value="<?php echo ( array_key_exists( 'auth', $headers ) && is_string( $headers['auth'] ) ? $headers['auth'] : '' ); ?>"
 											placeholder="<?php esc_html_e( 'e.g. SharedKey <AccountName>:<Signature>', 'visualizer' ); ?>"
 											class="visualizer-input json-form-element">
 									</div>

js/frame.js

@@ -376,6 +376,11 @@
             active: false,
             collapsible: true
         });
+        
+        // open the accordions by default if they are indicated with the 'open' class.
+        $('.visualizer-json-subform .viz-substep.open').each(function(i, e){
+            $('.visualizer-json-subform').accordion( "option", "active", i );
+        });
 
         // toggle between chart and create/modify parameters
         $( '#json-chart-button' ).on( 'click', function(){