Support handling body tag across multiple lines (#70)

ashmaroli · web-flow · commit 66ee59d69b23 · 2020-02-26T11:17:36.000-05:00
Merge pull request 70
diff --git a/lib/jekyll-mentions.rb b/lib/jekyll-mentions.rb
@@ -8,7 +8,7 @@ class Mentions
     GITHUB_DOT_COM = "https://github.com"
     BODY_START_TAG = "<body"
 
-    OPENING_BODY_TAG_REGEX = %r!<body(.*?)>\s*!.freeze
+    OPENING_BODY_TAG_REGEX = %r!<body(.*?)>\s*!m.freeze
 
     InvalidJekyllMentionConfig = Class.new(Jekyll::Errors::FatalException)
 
diff --git a/spec/fixtures/_layouts/multiline_body_tag.html b/spec/fixtures/_layouts/multiline_body_tag.html
@@ -0,0 +1,14 @@
+<!DOCTYPE HTML>
+<html lang="en-US">
+  <head>
+    <meta charset="UTF-8">
+    <title>{{ page.title }}</title>
+    <meta name="viewport" content="width=device-width,initial-scale=1">
+    <link rel="stylesheet" href="/css/screen.css">
+  </head>
+  <body
+    class="wrap"
+  >
+    {{ content }}
+  </body>
+</html>
diff --git a/spec/fixtures/multiline-body-tag.md b/spec/fixtures/multiline-body-tag.md
@@ -0,0 +1,6 @@
+---
+layout: multiline_body_tag
+title: Multi-line Body Tag
+---
+
+test @TestUser test
diff --git a/spec/mentions_spec.rb b/spec/mentions_spec.rb
@@ -91,6 +91,10 @@ def para(content)
     )
   end
 
+  it "works with HTML body tag markup across multiple lines" do
+    expect(find_by_title(site.pages, "Multi-line Body Tag").output).to include(para(result))
+  end
+
   context "when jekyll-mentions is set to false" do
     it "should not replace the @TestUser with the link to @TestUser" do
       expect(disabled_mentioning_page.output).not_to include(result)
