Skip to content

Fix issues with image display#3555

Merged
swissspidy merged 1 commit intodevelopfrom
fix/3429-fix_image_display
Oct 18, 2019
Merged

Fix issues with image display#3555
swissspidy merged 1 commit intodevelopfrom
fix/3429-fix_image_display

Conversation

@miina
Copy link
Copy Markdown
Contributor

@miina miina commented Oct 18, 2019

Summary

Fixes #3429

  • Fix issues with image positioning.
  • Uses display: block for images in the FE.

Checklist

  • My pull request is addressing an open issue (please create one otherwise).
  • My code is tested and passes existing tests.
  • My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).

Use display:block for figure element.
@googlebot googlebot added the cla: yes Signed the Google CLA label Oct 18, 2019
@miina miina requested review from barklund and swissspidy October 18, 2019 13:39
Copy link
Copy Markdown
Collaborator

@swissspidy swissspidy left a comment

Choose a reason for hiding this comment

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

LGTM!

@swissspidy swissspidy added this to the v1.4 milestone Oct 18, 2019
@swissspidy swissspidy merged commit 7ecb74a into develop Oct 18, 2019
@swissspidy swissspidy deleted the fix/3429-fix_image_display branch October 18, 2019 13:51
westonruter added a commit that referenced this pull request Oct 18, 2019
…twentytwenty-support

* 'develop' of github.com:ampproject/amp-wp: (73 commits)
  Mock HTTP request for Test_AMP_WordPress_TV_Embed_Handler
  Add test case
  Fix issues with image display (#3555)
  Fix infinite loop when uploading videos (#3553)
  Fix styling on remove button. (#3554)
  Test for XML processing instruction
  Undo over-eager rename of auto_accept_sanitization
  Rename auto-accept to accept-by-default
  Rename filter to amp_validation_error_auto_sanitized
  Fix jumping blocks when changing fonts (#3529)
  Update dependency browserslist to v4.7.1 (#3551)
  Make sure that isPageBlock always returns boolean.
  Fix and improve e2e tests.
  Remove unnessary check for block. Use shift over access first element of array.
  Use isPageBlock for cleaner code.
  Fix minor grammar issues
  Add docblock for `amp_is_sanitization_auto_accepted` filter
  Align arrows
  Update tests to use filter to set sanitization auto acceptance
  Make AMP_Validation_Manage::is_sanitization_auto_accepted() filterable
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Signed the Google CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Image Not Rendering Properly on Mobile and Desktop

3 participants