Skip to content

Commit 7448857

Browse files
committed
Changes suggested in code review
1 parent 50713a3 commit 7448857

File tree

4 files changed

+32
-8
lines changed

4 files changed

+32
-8
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ Please also have a look at our
3636

3737
### Removed
3838

39-
- Remove the IE hack in `Rule` (#995))
39+
- Remove the IE hack in `Rule` (#995)
4040
- Remove `OutputFormat::level()` (#874)
4141
- Remove expansion of shorthand properties (#838)
4242
- Remove `Parser::setCharset/getCharset` (#808)

src/Rule/Rule.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -86,13 +86,6 @@ public static function parse(ParserState $parserState): Rule
8686
$parserState->consume(':');
8787
$value = Value::parseValue($parserState, self::listDelimiterForRule($rule->getRule()));
8888
$rule->setValue($value);
89-
if ($parserState->getSettings()->bLenientParsing) {
90-
while ($parserState->comes('\\')) {
91-
$parserState->consume('\\');
92-
$parserState->consume();
93-
$parserState->consumeWhiteSpace();
94-
}
95-
}
9689
$parserState->consumeWhiteSpace();
9790
if ($parserState->comes('!')) {
9891
$parserState->consume('!');

tests/ParserTest.php

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1060,6 +1060,28 @@ public function unexpectedTokenExceptionLineNo(): void
10601060
}
10611061
}
10621062

1063+
/**
1064+
* @test
1065+
*/
1066+
public function ieHacksCauseExceptionInStrictMode(): void
1067+
{
1068+
$this->expectException(UnexpectedTokenException::class);
1069+
1070+
self::parsedStructureForFile('ie-hacks', Settings::create()->beStrict());
1071+
}
1072+
1073+
/**
1074+
* @test
1075+
*/
1076+
public function ieHacksAreRemovedInLenientMode(): void
1077+
{
1078+
$document = self::parsedStructureForFile('ie-hacks', Settings::create()->withLenientParsing(true));
1079+
1080+
$expected = 'p {padding-right: .75rem;background-image: none;color: red;'
1081+
. 'background-color: red;background-color: red;content: "red \\0";content: "red઼";}';
1082+
self::assertSame($expected, $document->render());
1083+
}
1084+
10631085
/**
10641086
* @depends files
10651087
*

tests/fixtures/ie-hacks.css

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
p {
2+
padding-right: .75rem \9;
3+
background-image: none \9;
4+
color:red\9\0;
5+
background-color:red \9 \0;
6+
background-color:red \9 \0 !important;
7+
content: "red \9\0";
8+
content: "red\0abc";
9+
}

0 commit comments

Comments
 (0)