article-json-html-render
Advanced tools
Comparing version 2.0.0 to 2.1.0
@@ -42,2 +42,3 @@ 'use strict'; | ||
var Embed = _ref2.Embed; | ||
var renderEmptyTextNodes = _ref2.renderEmptyTextNodes; | ||
@@ -90,3 +91,3 @@ var types = { | ||
var typeFn = types[type]; | ||
if (!typeFn || isText(type) && !hasContent(item)) { | ||
if (!typeFn || !renderEmptyTextNodes && isText(type) && !hasContent(item)) { | ||
return ''; | ||
@@ -93,0 +94,0 @@ } |
@@ -20,3 +20,4 @@ 'use strict'; | ||
blockList: (0, _blockList2.default)({ | ||
Embed: (0, _embed2.default)(opts) | ||
Embed: (0, _embed2.default)(opts), | ||
renderEmptyTextNodes: opts.renderEmptyTextNodes | ||
}) | ||
@@ -23,0 +24,0 @@ }); |
@@ -13,3 +13,3 @@ import element from 'magic-virtual-element'; | ||
const setup = ({Embed}) => { | ||
const setup = ({Embed, renderEmptyTextNodes}) => { | ||
const types = { | ||
@@ -32,3 +32,3 @@ blockquote: ({children}) => <Blockquote items={children} />, | ||
const typeFn = types[type]; | ||
if (!typeFn || (isText(type) && !hasContent(item))) { | ||
if (!typeFn || (!renderEmptyTextNodes && isText(type) && !hasContent(item))) { | ||
return ''; | ||
@@ -35,0 +35,0 @@ } |
@@ -7,3 +7,4 @@ import setupArticle from './components/article'; | ||
blockList: setupBlocklist({ | ||
Embed: setupEmbed(opts) | ||
Embed: setupEmbed(opts), | ||
renderEmptyTextNodes: opts.renderEmptyTextNodes | ||
}) | ||
@@ -10,0 +11,0 @@ }); |
{ | ||
"name": "article-json-html-render", | ||
"version": "2.0.0", | ||
"version": "2.1.0", | ||
"description": "Base for html-based article-json renderer, such as [article-json-to-amp](https://www.npmjs.com/package/article-json-to-amp)", | ||
@@ -5,0 +5,0 @@ "main": "dist/index.js", |
24
test.js
@@ -167,2 +167,26 @@ import test from 'tape'; | ||
test('text with no content, opts.renderEmptyTextNodes = true', t => { | ||
const Article = setupArticle({embeds: {}, renderEmptyTextNodes: true}); | ||
const items = [ | ||
'paragraph', | ||
'header1', 'header2', 'header3', 'header4', 'header5', 'header6' | ||
].map(type => ({ | ||
type, children: [] | ||
})); | ||
const actual = renderString(tree(<Article items={items} />)); | ||
const expected = renderString(tree(<article> | ||
<p></p> | ||
<h1></h1> | ||
<h2></h2> | ||
<h3></h3> | ||
<h4></h4> | ||
<h5></h5> | ||
<h6></h6> | ||
</article>)); | ||
t.equal(actual, expected); | ||
t.end(); | ||
}); | ||
test('blockquote', t => { | ||
@@ -169,0 +193,0 @@ const Article = setupArticle({embeds: {}}); |
New author
Supply chain riskA new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.
Found 1 instance in 1 package
27992
26
761
0