Move script position to performance settings

Author: Tsvetan Stoychev

plugins/basicrum/languages/basicrum-bg_BG.po

@@ -144,23 +144,23 @@ msgstr ""
 msgid "Milliseconds to delay the beacon after onload. Leave 0 to disable the delay."
 msgstr ""
 
-#: src/Admin/Settings/Page.php:253
+#: src/Admin/Settings/Page.php:269
 msgid "Developer Settings"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:272
+#: src/Admin/Settings/Page.php:246
 msgid "Script Position"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:278
+#: src/Admin/Settings/Page.php:252
 msgid "Where to insert the monitoring script."
 msgstr ""
 
-#: src/Admin/Settings/Page.php:280
+#: src/Admin/Settings/Page.php:254
 msgid "Header (wp_head)"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:281
+#: src/Admin/Settings/Page.php:255
 msgid "Footer (wp_footer)"
 msgstr ""
 
@@ -212,10 +212,10 @@ msgstr "Невалиден Brum Site ID. Копирайте го от админ
 msgid "Delay cannot exceed 30000 milliseconds. It has been capped at 30000."
 msgstr ""
 
-#: src/Admin/Settings/Page.php:260
+#: src/Admin/Settings/Page.php:276
 msgid "Development Mode"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:266
+#: src/Admin/Settings/Page.php:282
 msgid "Allow HTTP beacon URLs for local testing. Do not enable this on production sites."
 msgstr ""

plugins/basicrum/languages/basicrum.pot

@@ -143,32 +143,32 @@ msgstr ""
 msgid "Milliseconds to delay the beacon after onload. Leave 0 to disable the delay."
 msgstr ""
 
-#: src/Admin/Settings/Page.php:253
-msgid "Developer Settings"
+#: src/Admin/Settings/Page.php:246
+msgid "Script Position"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:260
-msgid "Development Mode"
+#: src/Admin/Settings/Page.php:252
+msgid "Where to insert the monitoring script."
 msgstr ""
 
-#: src/Admin/Settings/Page.php:266
-msgid "Allow HTTP beacon URLs for local testing. Do not enable this on production sites."
+#: src/Admin/Settings/Page.php:254
+msgid "Header (wp_head)"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:272
-msgid "Script Position"
+#: src/Admin/Settings/Page.php:255
+msgid "Footer (wp_footer)"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:278
-msgid "Where to insert the monitoring script."
+#: src/Admin/Settings/Page.php:269
+msgid "Developer Settings"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:280
-msgid "Header (wp_head)"
+#: src/Admin/Settings/Page.php:276
+msgid "Development Mode"
 msgstr ""
 
-#: src/Admin/Settings/Page.php:281
-msgid "Footer (wp_footer)"
+#: src/Admin/Settings/Page.php:282
+msgid "Allow HTTP beacon URLs for local testing. Do not enable this on production sites."
 msgstr ""
 
 #: src/Admin/Settings/Page.php:288

plugins/basicrum/readme.txt

@@ -71,7 +71,8 @@ Yes. Enable Development Mode under Basicrum's Developer Settings to preserve HTT
 
 1. Admin settings page - General settings with beacon URL and Brum Site ID.
 2. Privacy settings - Consent mode configuration with JS API documentation.
-3. Developer settings - Script position and debug options.
+3. Performance settings - Beacon timing and script position options.
+4. Developer settings - Development mode and loader debugging options.
 
 == Changelog ==
 

plugins/basicrum/src/Admin/Settings/Page.php

@@ -240,6 +240,22 @@ private function add_performance_section() {
 				'max'   => 30000,
 			)
 		);
+
+		add_settings_field(
+			'script_position',
+			esc_html__( 'Script Position', 'basicrum' ),
+			array( $this, 'render_radio_field' ),
+			self::SLUG,
+			'basicrum_section_performance',
+			array(
+				'id'      => 'script_position',
+				'label'   => __( 'Where to insert the monitoring script.', 'basicrum' ),
+				'options' => array(
+					'header' => __( 'Header (wp_head)', 'basicrum' ),
+					'footer' => __( 'Footer (wp_footer)', 'basicrum' ),
+				),
+			)
+		);
 	}
 
 	/**
@@ -267,22 +283,6 @@ private function add_developer_section() {
 			)
 		);
 
-		add_settings_field(
-			'script_position',
-			esc_html__( 'Script Position', 'basicrum' ),
-			array( $this, 'render_radio_field' ),
-			self::SLUG,
-			'basicrum_section_developer',
-			array(
-				'id'      => 'script_position',
-				'label'   => __( 'Where to insert the monitoring script.', 'basicrum' ),
-				'options' => array(
-					'header' => __( 'Header (wp_head)', 'basicrum' ),
-					'footer' => __( 'Footer (wp_footer)', 'basicrum' ),
-				),
-			)
-		);
-
 		add_settings_field(
 			'use_unminified_loaders',
 			esc_html__( 'Use Unminified Loaders', 'basicrum' ),