7.0.0 broke comments on nested elements

[pgorny]

The change in #163 naively that was included in 7.0.0 expects the node contents to be something that can be included in a regex, but the problem is that both CSS and Regex share some syntax elements, including [] used for attribute selector and character classes, respectively.

This means that a simple code like this:

/* Comment */
&[data-roots-all] {
  display: none;
}

will now fail with

SyntaxError: Invalid regular expression: //* Comment */ *
*&[data-roots-all] {
   display: none;
}/: Range out of order in character class

This is due to the fact that [data-roots-all] is not treated as a literal, but rather a character class - and thus s-a class trips it up.

[ai]

@Ulyanov-programmer can you look and fix?

[Ulyanov-programmer]

I'll get on it as soon as possible. It will be necessary to add similar situations to the tests...

[Plumbiu]

It seems that comments cause a lot of problems:

.md {
  color: red;
  & > * {
    margin-bottom: 12px;
  }
  /* table */
  table {}
}

generated:

.md {
  color: red;
}
  .md .md * {
    margin-bottom: 12px;
  }
  /* table */
  table {
  }

[Ulyanov-programmer]

It seems that comments cause a lot of problems:

I've already written about it to @ai in PR but for some reason he hasn't read about it.
I have already solved this problem, but for some reason the author of the repository did not consider it necessary to include this solution.

[ai]

@Plumbiu the comment was moved correctly.

Where is the problem?

[Ulyanov-programmer]

@Plumbiu the comment was moved correctly.

Where is the problem?

Nesting does not work)

[ai]

I have already solved this problem, but for some reason the author of the repository did not consider it necessary to include this solution

Seems like I missed that there was a problem with selectors.

Can you send PR because it is regression?

[Ulyanov-programmer]

I have already solved this problem, but for some reason the author of the repository did not consider it necessary to include this solution

Seems like I missed that there was a problem with selectors.

Can you send PR because it is regression?

#167

[ai]

The fix was released in 7.0.2.