fix: avoid removing space after /deep/ when used directly (close #1035)

yyx990803 · yyx990803 · commit c30331259f33 · 2017-11-03T14:31:43.000-04:00
diff --git a/lib/style-compiler/plugins/scope-id.js b/lib/style-compiler/plugins/scope-id.js
@@ -29,9 +29,9 @@ module.exports = postcss.plugin('add-id', ({ id }) => root => {
           }
           // /deep/ alias for >>>, since >>> doesn't work in SASS
           if (n.type === 'tag' && n.value === '/deep/') {
-            const next = n.next()
-            if (next.type === 'combinator' && next.value === ' ') {
-              next.remove()
+            const prev = n.prev()
+            if (prev && prev.type === 'combinator' && prev.value === ' ') {
+              prev.remove()
             }
             n.remove()
             return false

Original file line number	Diff line number	Diff line change
`@@ -29,9 +29,9 @@ module.exports = postcss.plugin('add-id', ({ id }) => root => {`
`29`	`29`	`}`
`30`	`30`	`// /deep/ alias for >>>, since >>> doesn't work in SASS`
`31`	`31`	`if (n.type === 'tag' && n.value === '/deep/') {`
`32`		`- const next = n.next()`
`33`		`- if (next.type === 'combinator' && next.value === ' ') {`
`34`		`- next.remove()`
	`32`	`+ const prev = n.prev()`
	`33`	`+ if (prev && prev.type === 'combinator' && prev.value === ' ') {`
	`34`	`+ prev.remove()`
`35`	`35`	`}`
`36`	`36`	`n.remove()`
`37`	`37`	`return false`