Reworked general settings section by moving title and font settings into separate groups

Author: Eugene Manuilov

classes/Visualizer/Render/Sidebar.php

@@ -152,13 +152,17 @@ protected function _renderChartTitleSettings() {
 	protected function _renderGeneralSettings() {
 		self::_renderGroupStart( esc_html__( 'General Settings', Visualizer_Plugin::NAME ) );
 			self::_renderSectionStart();
-				self::_renderSectionDescription( esc_html__( 'Configure title, font styles and legend positioning settings for the chart.', Visualizer_Plugin::NAME ) );
+				self::_renderSectionDescription( esc_html__( 'Configure title, font styles, tooltip, legend and else settings for the chart.', Visualizer_Plugin::NAME ) );
+			self::_renderSectionEnd();
 
+			self::_renderSectionStart( esc_html__( 'Title', Visualizer_Plugin::NAME ), false );
 				$this->_renderChartTitleSettings();
+			self::_renderSectionEnd();
 
+			self::_renderSectionStart( esc_html__( 'Font Styles' ), false );
 				echo '<div class="section-item">';
 					echo '<a class="more-info" href="javascript:;">[?]</a>';
-					echo '<b>', esc_html__( 'Font Family And Size', Visualizer_Plugin::NAME ), '</b>';
+					echo '<b>', esc_html__( 'Family And Size', Visualizer_Plugin::NAME ), '</b>';
 
 					echo '<table class="section-table" cellspacing="0" cellpadding="0" border="0">';
 						echo '<tr>';

classes/Visualizer/Render/Sidebar/Graph.php

@@ -93,7 +93,12 @@ public function __construct( $data = array() ) {
 	 * @access protected
 	 */
 	protected function _renderChartTitleSettings() {
-		parent::_renderChartTitleSettings();
+		self::_renderTextItem(
+			esc_html__( 'Chart Title', Visualizer_Plugin::NAME ),
+			'title',
+			$this->title,
+			esc_html__( 'Text to display above the chart.', Visualizer_Plugin::NAME )
+		);
 
 		self::_renderSelectItem(
 			esc_html__( 'Chart Title Position', Visualizer_Plugin::NAME ),
@@ -103,6 +108,13 @@ protected function _renderChartTitleSettings() {
 			esc_html__( 'Where to place the chart title, compared to the chart area.', Visualizer_Plugin::NAME )
 		);
 
+		self::_renderColorPickerItem(
+			esc_html__( 'Chart Title Color', Visualizer_Plugin::NAME ),
+			'titleTextStyle[color]',
+			isset( $this->titleTextStyle['color'] ) ? $this->titleTextStyle['color'] : null,
+			'#000'
+		);
+
 		echo '<div class="section-delimiter"></div>';
 
 		self::_renderSelectItem(
@@ -112,8 +124,6 @@ protected function _renderChartTitleSettings() {
 			$this->_positions,
 			esc_html__( 'Determines where to place the axis titles, compared to the chart area.', Visualizer_Plugin::NAME )
 		);
-
-		echo '<div class="section-delimiter"></div>';
 	}
 
 	/**

readme.txt

@@ -65,6 +65,7 @@ Pay attention that to turn your shortcodes into graphs, your theme has to have `
 * Implemented new settings for pie chart like pie hole, start angle and slice offset
 * Implemented ability to select a color for chart title and legend items
 * Fixed number formatting settings for linear charts, from now it works only for axis labels
+* Reworked general settings section by moving title and font settings into separate groups
 
 = 1.3.0.2 =
 * Replaced links to github wiki