Merge pull request #24 from infininight/master

The Italic Conundrum

Author: noniq

Syntaxes/Markdown (GitHub Italics).tmLanguage

@@ -7,7 +7,7 @@
 	<key>hideFromUser</key>
 	<true/>
 	<key>injectionSelector</key>
-	<string>(L:text.html.markdown)</string>
+	<string>L:(text.html.markdown - (markup.italic.markdown | markup.bold.markdown | markup.raw))</string>
 	<key>name</key>
 	<string>Markdown (GitHub Italics)</string>
 	<key>patterns</key>
@@ -27,7 +27,7 @@
 				precedence and thus prevent the main Markdown grammar’s pattern for italic text from
 				matching these underscores.)</string>
 			<key>match</key>
-			<string>(_)(?=\w)(?&lt;=\w)</string>
+			<string>(?&lt;=\w)(_)(?=\w)</string>
 			<key>name</key>
 			<string>markup.other</string>
 		</dict>

Tests/Italics.mdown

@@ -0,0 +1,35 @@
+# Should Match
+
+_italic_
+
+_italic italic_
+
+_italic_regular
+
+
+# Should Not Match
+
+regular_regular_regular
+
+
+# Should Not Match But Does
+
+regular _italic_regular
+
+
+# Other Cases (Should not match)
+
+__double underscores__
+
+    _raw_
+    
+    raw_raw_raw
+
+Lorem `ipsum_dolor_sit` amet
+
+
+# Exceptional Cases
+
+$$y_i = \alpha_{j[i]} + \beta x_i + \epsilon_i$$
+
+This is MathJax code which should be parsed separately, but shows just how loose the main spec is about what can be in italic.