Merge pull request #264 from contactashish13/issue-263

When Add New is used from within a selected chart type tab, that char…

Author: rodica-andronache

classes/Visualizer/Module/Admin.php

@@ -535,6 +535,7 @@ public function renderLibraryPage() {
 						array(
 							'action'  => Visualizer_Plugin::ACTION_CREATE_CHART,
 							'library' => 'yes',
+							'type'      => isset( $_GET['type'] ) ? $_GET['type'] : '',
 						), $ajaxurl
 					),
 					'edit'   => add_query_arg(

classes/Visualizer/Module/Chart.php

@@ -209,7 +209,7 @@ public function renderChartPages() {
 		// check chart, if chart not exists, will create new one and redirects to the same page with proper chart id
 		$chart_id = isset( $_GET['chart'] ) ? filter_var( $_GET['chart'], FILTER_VALIDATE_INT ) : '';
 		if ( ! $chart_id || ! ( $chart = get_post( $chart_id ) ) || $chart->post_type != Visualizer_Plugin::CPT_VISUALIZER ) {
-			$default_type = 'line';
+			$default_type = isset( $_GET['type'] ) && ! empty( $_GET['type'] ) ? $_GET['type'] : 'line';
 			$source       = new Visualizer_Source_Csv( VISUALIZER_ABSPATH . DIRECTORY_SEPARATOR . 'samples' . DIRECTORY_SEPARATOR . $default_type . '.csv' );
 			$source->fetch();
 			$chart_id = wp_insert_post(

js/frame.js

@@ -3,6 +3,13 @@
 /* global alert */
 
 (function ($) {
+    $(window).load(function(){
+        // scroll to the selected chart type.
+        if($('label.type-label.type-label-selected').length > 0) {
+            $('label.type-label.type-label-selected')[0].scrollIntoView();
+        }
+    });
+
     $(document).ready(function () {
         init_permissions();