From b06f420ea168bee0539f8c09a1ee1227d93b2000 Mon Sep 17 00:00:00 2001
From: jeffreytse <jeffreytse.mail@gmail.com>
Date: Fri, 2 Apr 2021 17:08:15 +0800
Subject: [PATCH] fix: weird inline mathjax processing (#50)

This issue is caused by the wrongly scanning no any mathjax in a
page while the `optimize` option is on, the core issue is that the
processor just check the node that has no chindren, but <ul> node
may has several children.
---
 .../processors/mathjax-processor.rb           | 25 +++++++++++--------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/lib/jekyll-spaceship/processors/mathjax-processor.rb b/lib/jekyll-spaceship/processors/mathjax-processor.rb
index be4a60d..0d5f207 100644
--- a/lib/jekyll-spaceship/processors/mathjax-processor.rb
+++ b/lib/jekyll-spaceship/processors/mathjax-processor.rb
@@ -107,17 +107,22 @@ def get_math_patterns()
 
     def scan_mathjax_expression(doc, &block)
       patterns = get_math_patterns()
-      doc.css('*').each do |node|
-        next if ['code', 'pre', 'figure'].include? node.name
-        next if node.ancestors('code, pre, figure').size > 0
-        next if node.children.size > 1
+      doc = doc.clone
+
+      # remove code, pre, figure nodes
+      doc.css('body code, body pre, body figure').each do |node|
+        node.remove
+      end
+
+      # remove scripting mathjax expression
+      doc.css('body script').each do |node|
+        next if node['type']&.match(/math\/tex/)
+        node.remove
+      end
+
+      # scan mathjax expressions
+      doc.css('body *').each do |node|
         patterns['include'].each do |pattern|
-          # check scripting mathjax expression
-          if node.name == 'script'
-            type = node['type']
-            next unless type
-            next unless type.match(/math\/tex/)
-          end
           # check normal mathjax expression
           node.content.scan(pattern) do |result|
             expr = result[0]