Skip to content

Commit dfee983

Browse files
authored
Merge pull request #527 from hazendaz/main
[css] Add patch to retain \9 during css formatting as cssparser conversions fixes #522
2 parents 308dfe9 + 1db7bdc commit dfee983

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/main/java/net/revelc/code/formatter/css/CssFormatter.java

+3
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,9 @@ protected String doFormat(String code, LineEnding ending) throws IOException {
5757
CSSStyleSheetImpl sheet = (CSSStyleSheetImpl) parser.parseStyleSheet(source, null, null);
5858
String formattedCode = sheet.getCssText(formatter);
5959

60+
// Patch converted 'tab' back to '\9' for IE 7,8, and 9 hack. Cssparser switches it to 'tab'.
61+
formattedCode = formattedCode.replace("\t;", "\\9;");
62+
6063
if (code.equals(formattedCode)) {
6164
return null;
6265
}

src/test/java/net/revelc/code/formatter/css/CssFormatterTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@ void testDoFormatFile() throws Exception {
3232
// FIXME Handle linux vs windows since this formatter does not accept line endings
3333
if (System.lineSeparator().equals("\n")) {
3434
doTestFormat(new CssFormatter(), "someFile.css",
35-
"80042a8a00870195a43c6a80e8e0b2d13e3ac8fd3f52e10277f05a637af6bb5bdddcf5b939677be0b48b907a74861f3f3cd71be38edf4f740c7e9f31d79b1ee1");
35+
"72b2c33020774b407c2e49a849e47990941d3c80d982b1a4ef2e0ffed605b85e2680fca57cfdbc9d6cd2fc6fc0236dbeb915fd75f530689c7e90a3745316b6a3");
3636
} else {
3737
doTestFormat(new CssFormatter(), "someFile.css",
38-
"332e3475ccfcb029ba8db3fd1d9ebd7e44dad60d96866de50e0434c0075a41e138c3fc919410780e985c69e6975d86cf18c2048ab1311cb382218db82dfaa79e");
38+
"684255d79eb28c6f4cfa340b6930fe1cfd9de16a1c6abf5f54e8f6837694b599101ef247ed00b8aea5460aa64cda60b418cebefd8ea28d5e747ed9cf4c3a9274");
3939
}
4040
}
4141

0 commit comments

Comments
 (0)