SOLVED: Video Element preload attribute

Bugs
1

Browser: Chrome 110
OS: macOS / Windows / Linux / etc.
URL: Link to a page that illustrates this issue
Video: Short screen recording that illustrates this issue (free tool: jam.dev)

In the video element, when using external url or in my case, dynamic data (an url-acf-field), choosing preload: none (the default), in the code there will not be any preload attribute at all, but it should be preload=“none”.

2

I found in the /includes/elements/video.php file, you have only 2 options for preload, metadata and auto, and the “none” is just a placeholder. of course, when it checks, if a setting was made, to insert the preload-attribute, it won’t.
so please set preload=“none” as default if no other option was choosen, or add “none” as an option.

3

Hi Christian,
Welcome to the forum!

Yep, I agree that “none” is missing as an option and have created an improvement task.

The default value is different for each browser. The spec advises it to be set to metadata .
<video>: The Video Embed element - HTML: HyperText Markup Language | MDN

Best regards,
timmse

1 Like
4

Hi guys,

We’ve added this improvement in Bricks 1.10.2, now available as a one-click update in your WordPress Dashboard.

Please let us know if you are still experiencing issues.

You can see the full changelog here: Bricks 1.10.2 Changelog – Bricks

Best regards,
M