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

Feature image stretched on Legacy theme, when native image is being used. #7550

Closed
milindmore22 opened this issue May 31, 2023 · 1 comment · Fixed by #7560
Closed

Feature image stretched on Legacy theme, when native image is being used. #7550

milindmore22 opened this issue May 31, 2023 · 1 comment · Fixed by #7560
Assignees
Labels
Bug Something isn't working Changelogged Whether the issue/PR has been added to release notes.
Milestone

Comments

@milindmore22
Copy link
Collaborator

Bug Description

I received a few support topics topic 1 and topic 2 where use experience images being stretched on mobile devices.

I am able to reproduce the issue on my local.

We can prevent this by adding CSS

figure.amp-wp-article-featured-image img {
    height: auto;
}

Expected Behaviour

Feature image in legacy theme working correctly on mobile and tablet devices

Screenshots

image

PHP Version

No response

Plugin Version

2.4.1

AMP plugin template mode

Reader

WordPress Version

6.2.2

Site Health

No response

Gutenberg Version

No response

OS(s) Affected

No response

Browser(s) Affected

No response

Device(s) Affected

No response

Acceptance Criteria

No response

Implementation Brief

No response

QA Testing Instructions

No response

Demo

No response

Changelog Entry

No response

@pooja-muchandikar
Copy link

QA Passed ✅

Verified the issue, the featured native image is visible properly now on mobile and tablet.

Before fix:
image


After fix:
image

@thelovekesh thelovekesh added Enhancement New feature or improvement of an existing one P2 Low priority Reader Mode WS:Core Work stream for Plugin core and removed Enhancement New feature or improvement of an existing one P2 Low priority Reader Mode WS:Core Work stream for Plugin core labels Jul 13, 2023
@westonruter westonruter added the Changelogged Whether the issue/PR has been added to release notes. label Jul 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Changelogged Whether the issue/PR has been added to release notes.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants