Media embed is stripping width and height from embedding media

@wiifm ,
thank you for your patch. We try to check it as soon as possible. Please be patient.

I'm running into the same issue with version 7.x-1.16. For this version, I needed to patch the dialog JS instead—it was stripping out any number values that included px or &.

Reroll #5 for latest 7.x-1.x-dev

The idea from #7 seems to be working just fine, although I think is too wide as \S* matches any non-whitespace character (equal to [^\r\n\t\f\v ]).

For the width/height we can have digits or digits with the unit, eg width="300" or width="0.50vw" or eventually width="50%".

So there is a need just for: [a-zA-Z0-9\.\%] or even a whitelist of all possible units. That would require more updates though, so I would just stick to easier but not too wide regex.

See attached proposal.

While I understand the purpose of this patch from 9 years ago, I don't believe it has much value today.

Embedded media in modern websites should not have hard-coded width and height settings, otherwise it is impossible to properly scale on mobile devices. The much better approach is to standardized aspect ratios that will display correctly on any device size.

This can easily be done as a preprocess for any media such that it is seamless to content contributors.