From 09aebfcc9800ae360bbd969a3749d0e4de661b2a Mon Sep 17 00:00:00 2001
From: Marcus Quinn <marcus@agentdesign.co.uk>
Date: Mon, 24 Mar 2025 22:33:07 +0000
Subject: [PATCH] Fix numbered list indentation to match bullet lists with
 dedicated class and targeted CSS

---
 admin/css/wp-allstars-admin.css         | 16 ++++++++--------
 admin/includes/class-readme-manager.php |  6 +++---
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/admin/css/wp-allstars-admin.css b/admin/css/wp-allstars-admin.css
index 858c2f2..6cd104a 100644
--- a/admin/css/wp-allstars-admin.css
+++ b/admin/css/wp-allstars-admin.css
@@ -1025,18 +1025,18 @@ body.wp-admin .button.pricing-button:hover,
 
 #readme .wp-allstars-markdown-content ol.wp-allstars-list,
 #readme .wp-allstars-markdown-content ol {
-    padding-left: 25px; /* Same padding as bullet lists for consistent indentation */
+    padding-left: 25px; /* Same as bullets */
     margin: 10px 0;
     list-style-type: decimal;
 }
 
-/* Adjust for the wider numbers to align text with bullet lists */
-#readme .wp-allstars-markdown-content ol.wp-allstars-list li,
-#readme .wp-allstars-markdown-content ol li {
-    padding-left: 0; /* Remove extra padding */
-    margin-left: -5px; /* Subtle adjustment to align text with bullet lists */
-    text-indent: -5px; /* Adjust text indent to align with bullet text */
-    padding-right: 10px; /* Prevent text from being too close to the edge */
+/* Special styling for numbered lists to align with bullet lists */
+#readme .wp-allstars-markdown-content ol.numbered-list {
+    padding-left: 19px; /* Adjusted to better align with bullets */
+}
+
+#readme .wp-allstars-markdown-content ol.numbered-list li {
+    margin-left: 4px; /* Fine-tune the alignment */
 }
 
 #readme .wp-allstars-markdown-content li {
diff --git a/admin/includes/class-readme-manager.php b/admin/includes/class-readme-manager.php
index 481bf26..a9ce3be 100644
--- a/admin/includes/class-readme-manager.php
+++ b/admin/includes/class-readme-manager.php
@@ -97,14 +97,14 @@ class WP_Allstars_Readme_Manager {
         // First group bullet items into unordered lists
         $markdown = preg_replace('/((?:<li class="bullet-item">.*?<\/li>\n)+)/s', '<ul class="wp-allstars-ul">$1</ul>', $markdown);
         
-        // Then group numbered items into ordered lists
-        $markdown = preg_replace('/((?:<li class="number-item">.*?<\/li>\n)+)/s', '<ol class="wp-allstars-ol">$1</ol>', $markdown);
+        // Then group numbered items into ordered lists with a special class for indentation
+        $markdown = preg_replace('/((?:<li class="number-item">.*?<\/li>\n)+)/s', '<ol class="wp-allstars-ol numbered-list">$1</ol>', $markdown);
         
         // Clean up the classes from the final output
         $markdown = str_replace('class="bullet-item"', '', $markdown);
         $markdown = str_replace('class="number-item"', '', $markdown);
         $markdown = str_replace('class="wp-allstars-ul"', 'class="wp-allstars-list"', $markdown);
-        $markdown = str_replace('class="wp-allstars-ol"', 'class="wp-allstars-list"', $markdown);
+        $markdown = str_replace('class="wp-allstars-ol numbered-list"', 'class="wp-allstars-list numbered-list"', $markdown);
         
         // Links
         $markdown = preg_replace('/\[(.*?)\]\((.*?)\)/s', '<a href="$2" target="_blank">$1</a>', $markdown);