Embedded video does not respect the media_subpath Front Matter variable

[steveng57]

Checklist

• I have read and accepted the contributing guidelines.

How did you create the site?

Generated from chirpy-starter

Describe the bug

When you embed a video using the include statement as follows:

{% include embed/video.html src="IMG_0695.mp4" muted = true title = "Walnut Table - 180° View" poster = "thumbnails/IMG_0695.jpeg"%}

the code ignores the media_subpath variable, looks for the mp4 file in the root of the project instead of where the media_subpath folder is pointing to.

Steps To Reproduce

Create a page and include the video.html file as above...set put the video in say /assets/posts/videos and set the media_subpath Front Matter page variable to that...

Expected Behavior

All media should be relative to the media_subpath variable, images and video ( and their posters).

In the file /_includes/embed/video.html, lines 6 thru 10 are as follows.

{% unless video_url contains '://' %}
  {%- capture video_url -%}
    {% include media-url.html src=video_url %}
  {%- endcapture -%}
{% endunless %}

It appears they should be...

{% unless video_url contains '://' %}
  {%- capture video_url -%}
    {% include media-url.html src=video_url subpath=page.media_subpath  %}
  {%- endcapture -%}
{% endunless %}

Environment

• Ruby: 3.2.1
• Jekyll: 4.3.3
• Chirpy: 7.0.0

Anything else?

No response

[github-actions]

🎉 This issue has been resolved in version 7.0.1 🎉

The release is available on:

• v7.0.1
• GitHub release

Your semantic-release bot 📦🚀

[steveng57]

Thanks for the fix!
Nice.

[kungfux]

Thanks for the fix!
Nice.

Thanks for reporting!
You reminded me there was an unresolved thing in new features.

[github-actions]

🎉 This issue has been resolved in version 7.1.0 🎉

The release is available on:

• v7.1.0
• GitHub release

Your semantic-release bot 📦🚀