Angular parser strips single quotes from ngClass

Prettier 1.15.3 Playground link

--parser angular

Input:

<div [ngClass]="{ 'x': y }"></div>

Output:

<div [ngClass]="{ x: y }"></div>

Expected behavior: The single quotations around the x should not be stripped out.

[artemzakharov]

@duailibe No, I am not worried that it will break the app - I am worried that it will lead to what some (including myself) would consider inconsistent styling, where someone who isn’t familiar with the otherwise somewhat obscure rules regarding multiple, dash separated word variable names in JS would be confused at why some ngClass examples get stripped of quotes, and others don’t. This is further amplified by the fact that while the parser strips quotes when they aren’t needed, it doesn’t add them where they are needed, creating further confusion for someone who might modify a single word class into a multi word one, and suddenly run into a compilation error.

At the end of the day, I’m just worried that newbies are going to get confused, but its possible that I’m worrying too much, so it would help if any other Angular devs could give their thoughts.

[creadicted]

Whitespace around will not lead to the removal of the quotes. [ngClass]="{ 'active': isActive }"> here the quotes will be removed [ngClass]="{ ' active ': isActive }"> here not