[10.x] Remove width and height settings from the formatter

Width and height attributes are still required on the video element to prevent Cumulative Layout Shift (CLS).

But I agree, we don't need this in the formatter settings. Its just confusing.

So lets stick with the height and with properties on the video-player SDCs, fix the docs:

* - width: Player width in pixels (overridden by CSS, required to prevent CLS).
* - height: Player height in pixels (overridden by CSS, required to prevent
* CLS).

I've tested a bit in order to reduce CLS and FOUC.

At the moment, we only set the height and width of the video HTML element (and on the video-player-rmote SDCs mediaComponent). I think we should set them for the video player element too. So additonaly:

{# Always set width and height to prevent CLS #}
{% set attributes = attributes.setAttribute('width', width|default('640')) %}
{% set attributes = attributes.setAttribute('height', height|default('360')) %}
{# Only required on the video-player SDC, can be remove on video-player-remote because we dont print the iframe element #}
{% set mediaAttributes = mediaAttributes.setAttribute('width', width|default('640')) %}
{% set mediaAttributes = mediaAttributes.setAttribute('height', height|default('360')) %}

We should also add this to the poster image (turn off JS to see the problem):

// Hide the poster image to prevent FOUC.
$poster['#attributes']->addClass('hidden');

The Video.js player doesn't care about this class, grabs the poster image and everything works fine.

We also should add some CSS to reduce FOUC:

video-player {
  display: block;
  video {
    width: 100%;
  }
}
youtube-video,
vimeo-video {
  display: block;
}

Done