Documentation text and sample are inconsistent: parentheses around RegExp literals

[alecglassford]

Describe the bug

On https://svelte.dev/docs/basic-markup#text-expressions (source markdown), there is an aside that reads:

If you're using a regular expression (RegExp) literal notation, you'll need to wrap it in parentheses.

This is followed by a markup sample:

<div>{/^[A-Za-z ]+$/.test(value) ? x : y}</div>

Note that in the sample, the RegExp literal is not wrapped in parentheses. Therefore this section of the doc becomes confusing to users: do you need to wrap RegExp literals in parentheses or not?

Reproduction

Navigate to https://svelte.dev/docs/basic-markup#text-expressions

Logs

N/A

System Info

N/A

Severity

annoyance

[alecglassford]

A little digging, in case it's helpful:

This part of the doc was added in #5640, and you'll see in this PR that the RegExp literal in the sample originally was wrapped in parentheses:

<div>{(/^[A-Za-z ]+$/).test(value) ? x : y}</div>

The parentheses were later removed in #8435 (specifically in this squashed commit).

• If the parentheses removal was purposeful, then the (now-outdated) aside text should also be removed.
• If the parentheses removal was accidental, then the parentheses should likely be added back to the sample.
  • However, if the need for parentheses has coincidentally/independently gone away since docs: mention usage of regexp literal in text expressions #5640, then the aside text should be removed and the sample should stay the same. 🤪