Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EWPP-3296: Theme the Media paragraph. #1270

Merged
merged 3 commits into from
Jun 15, 2023
Merged

EWPP-3296: Theme the Media paragraph. #1270

merged 3 commits into from
Jun 15, 2023

Conversation

22Alexandra
Copy link
Contributor

No description provided.

oe_theme.theme Show resolved Hide resolved
oe_theme.theme Outdated Show resolved Hide resolved
oe_theme.theme Outdated Show resolved Hide resolved
nagyad
nagyad previously approved these changes May 15, 2023
oe_theme.theme Outdated
if ($source instanceof MediaAvPortalVideoSource || $source instanceof OEmbed || $source instanceof Iframe) {
if ($source instanceof MediaAvPortalVideoSource) {
// Default video aspect ratio is set to 16:9 for AV Portal Video.
$variables['ratio'] = '16:9';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be messing up with the height of avportal v ideos

@22Alexandra 22Alexandra force-pushed the EWPP-3296 branch 2 times, most recently from d606ae9 to 33c424a Compare May 17, 2023 06:38
@22Alexandra 22Alexandra changed the base branch from 3.x to EWPP-3310 May 17, 2023 06:39
@22Alexandra 22Alexandra changed the base branch from EWPP-3310 to 3.x May 17, 2023 11:23
imanoleguskiza
imanoleguskiza previously approved these changes May 22, 2023
nagyad
nagyad previously approved these changes May 22, 2023
composer.json Outdated Show resolved Hide resolved
@22Alexandra 22Alexandra merged commit 7411110 into 3.x Jun 15, 2023
@22Alexandra 22Alexandra deleted the EWPP-3296 branch June 15, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants