Skip to content

Commit

Permalink
Address review
Browse files Browse the repository at this point in the history
  • Loading branch information
sosukesuzuki committed Sep 8, 2021
1 parent fbf980a commit 675228d
Show file tree
Hide file tree
Showing 4 changed files with 39 additions and 49 deletions.
11 changes: 5 additions & 6 deletions src/language-html/embed.js
Expand Up @@ -6,12 +6,11 @@ const {
} = require("../document/index.js");
const printFrontMatter = require("../utils/front-matter/print.js");
const {
closing: {
printClosingTag,
printClosingTagSuffix,
needsToBorrowPrevClosingTagEndMarker,
},
opening: { printOpeningTagPrefix, printOpeningTag },
printClosingTag,
printClosingTagSuffix,
needsToBorrowPrevClosingTagEndMarker,
printOpeningTagPrefix,
printOpeningTag,
} = require("./print/tag.js");
const { printImgSrcset, printClassNames } = require("./syntax-attribute.js");
const {
Expand Down
13 changes: 6 additions & 7 deletions src/language-html/get-node-content.js
@@ -1,13 +1,12 @@
"use strict";

const {
closing: {
needsToBorrowParentClosingTagStartMarker,
printClosingTagStartMarker,
needsToBorrowLastChildClosingTagEndMarker,
printClosingTagEndMarker,
},
opening: { needsToBorrowParentOpeningTagEndMarker, printOpeningTagEndMarker },
needsToBorrowParentClosingTagStartMarker,
printClosingTagStartMarker,
needsToBorrowLastChildClosingTagEndMarker,
printClosingTagEndMarker,
needsToBorrowParentOpeningTagEndMarker,
printOpeningTagEndMarker,
} = require("./print/tag.js");

function getNodeContent(node, options) {
Expand Down
36 changes: 16 additions & 20 deletions src/language-html/print/tag.js
Expand Up @@ -362,24 +362,20 @@ function printOpeningTagEndMarker(node) {
}

module.exports = {
closing: {
printClosingTag,
printClosingTagStart,
printClosingTagStartMarker,
printClosingTagEndMarker,
printClosingTagSuffix,
printClosingTagEnd,
needsToBorrowLastChildClosingTagEndMarker,
needsToBorrowParentClosingTagStartMarker,
needsToBorrowPrevClosingTagEndMarker,
},
opening: {
printOpeningTag,
printOpeningTagStart,
printOpeningTagPrefix,
printOpeningTagStartMarker,
printOpeningTagEndMarker,
needsToBorrowNextOpeningTagStartMarker,
needsToBorrowParentOpeningTagEndMarker,
},
printClosingTag,
printClosingTagStart,
printClosingTagStartMarker,
printClosingTagEndMarker,
printClosingTagSuffix,
printClosingTagEnd,
needsToBorrowLastChildClosingTagEndMarker,
needsToBorrowParentClosingTagStartMarker,
needsToBorrowPrevClosingTagEndMarker,
printOpeningTag,
printOpeningTagStart,
printOpeningTagPrefix,
printOpeningTagStartMarker,
printOpeningTagEndMarker,
needsToBorrowNextOpeningTagStartMarker,
needsToBorrowParentOpeningTagEndMarker,
};
28 changes: 12 additions & 16 deletions src/language-html/printer-html.js
Expand Up @@ -41,22 +41,18 @@ const { insertPragma } = require("./pragma.js");
const { locStart, locEnd } = require("./loc.js");
const embed = require("./embed.js");
const {
closing: {
printClosingTagEndMarker,
printClosingTagSuffix,
printClosingTag,
printClosingTagEnd,
needsToBorrowPrevClosingTagEndMarker,
needsToBorrowLastChildClosingTagEndMarker,
needsToBorrowParentClosingTagStartMarker,
},
opening: {
printOpeningTagPrefix,
printOpeningTag,
printOpeningTagStart,
printOpeningTagStartMarker,
needsToBorrowNextOpeningTagStartMarker,
},
printClosingTagEndMarker,
printClosingTagSuffix,
printClosingTag,
printClosingTagEnd,
needsToBorrowPrevClosingTagEndMarker,
needsToBorrowLastChildClosingTagEndMarker,
needsToBorrowParentClosingTagStartMarker,
printOpeningTagPrefix,
printOpeningTag,
printOpeningTagStart,
printOpeningTagStartMarker,
needsToBorrowNextOpeningTagStartMarker,
} = require("./print/tag.js");
const getNodeContent = require("./get-node-content.js");

Expand Down

0 comments on commit 675228d

Please sign in to comment.