fix: constrain height of video preview

[silverkszlo]

📝 Summary

• Resolves: Height of embedded videos added to page is too big #8114

The height of embedded videos is too big. This PR adds styling constraints to the video-wrapper.

🖼️ Screenshots

🏚️ Before | 🏡 After

|

B | A

🏁 Checklist

• Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
• Sign-off message is added to all commits
• Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
• Documentation (README or documentation) has been updated or is not required

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[mejo-]

Code changes look good to me. I guess limiting the height of the container means that videos (e.g. portrait format videos) will be resized to fit into this container no matter what? In any case I think 400px is a reasonable maximum height for a container of an embedded object.

[max-nextcloud]

Looks good to me. Shall we backport this?

[max-nextcloud]

/backport to stable32 please

[max-nextcloud]

/backport to stable31 please

[backportbot]

The backport to stable31 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable31
git pull origin stable31

# Create the new backport branch
git checkout -b backport/8118/stable31

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 4c277995

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/8118/stable31

Error: Failed to check for changes with origin/stable31: No changes found in backport branch

---

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

[max-nextcloud]

Merging and creating backport PRs - we can discuss the backports there in case they are controversial.