chore: line number dom elements updated to use span (#8)

readmeio · Sep 16, 2020 · 85aeb0e · 85aeb0e
1 parent df2236e
commit 85aeb0e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/__tests__/codeMirror.test.js b/__tests__/codeMirror.test.js
@@ -155,7 +155,7 @@ describe('highlight mode', () => {
   });
 
   it('should return line numbers by default', () => {
-    expect(node.find('p').first().hasClass('cm-lineNumber')).toBe(true);
+    expect(node.find('span').first().hasClass('cm-lineNumber')).toBe(true);
   });
 
   it('should convert variable regex matches to a component instance', () => {

diff --git a/src/codeMirror/index.jsx b/src/codeMirror/index.jsx
@@ -11,9 +11,9 @@ import 'codemirror/mode/meta';
 // Pre output conversion
 // CodeMirror "Gutter" -> Apply line numbers to styled lines
 const defaultLineJsx = line => (
-  <p key={`ln-${line}`} className="cm-lineNumber">
+  <span key={`ln-${line}`} className="cm-lineNumber">
     {line}
-  </p>
+  </span>
 );
 
 // Wrap children elements, seperated by line into divs