[2.0.0-beta2] The regex for attribute (values) props should allow all Tailwind allowed special characters

I may be OK to merge it but I am still a bit surprised than you prefer to opt-in the allowed characters rather than opt-out the forbidden characters.

Here, you are allowing any characters expected in a tailwind class name. But AttributeProp is not only for classes, it is also for any other HTML attribute:

class="[&_img]:rounded [&_*]:bg-primary" title="Hello, my world" href="https://www.drupal.org/?foo&bar"

So why not replacing the white list by something catching all characters except the forbidden ones?

• double quotes, because it is our delimiter
• https://html.spec.whatwg.org/#syntax-ambiguous-ampersand
• something else?

Source: https://html.spec.whatwg.org/#attributes-2

@pdureau

I've revised the regex for attribute values to exclude forbidden characters. Typically, these forbidden characters include quotes (") and encoded HTML characters such as &, <, etc.
Here is the updated regex for attribute values:

$double_quoted_value = '"(?!.*&[a-zA-Z0-9]+\;)[^"]*"';

hello guys, i added the code to only discard double quotes as requested (and not work around a whitelist as this was the case before i came in).
i also added a piece of code, to handle the component form unfolding in case of errors.
please Review Pierre.