Skip to content

✨Convert the full-width responsive to container-width responsive for desktop users.#28565

Merged
calebcordry merged 10 commits intoampproject:masterfrom
Jiaming-X:container_width_responsive
May 29, 2020
Merged

✨Convert the full-width responsive to container-width responsive for desktop users.#28565
calebcordry merged 10 commits intoampproject:masterfrom
Jiaming-X:container_width_responsive

Conversation

@Jiaming-X
Copy link
Copy Markdown
Member

@Jiaming-X Jiaming-X commented May 26, 2020

Implementations for issue: #28403. Convert the full-width responsive to container-width responsive for desktop users.

PTAL. @charliereams, @calebcordry, @jridgewell

@amp-owners-bot
Copy link
Copy Markdown

Hey @jeffkaufman! These files were changed:

extensions/amp-ad-network-adsense-impl/0.1/amp-ad-network-adsense-impl.js
extensions/amp-ad-network-adsense-impl/0.1/responsive-state.js
extensions/amp-ad-network-adsense-impl/0.1/test/test-amp-ad-network-adsense-impl.js
extensions/amp-ad-network-adsense-impl/0.1/test/test-responsive-state.js

@Jiaming-X
Copy link
Copy Markdown
Member Author

Screenshot:
after the changes
https://screenshot.googleplex.com/jAjoUDNUD8x
before the changes
https://screenshot.googleplex.com/CaFwsBPadYD

@calebcordry calebcordry self-requested a review May 26, 2020 15:39
@amp-bundle-size amp-bundle-size bot requested a review from estherkim May 27, 2020 19:53
@Jiaming-X Jiaming-X force-pushed the container_width_responsive branch from 21086e4 to 9da3f75 Compare May 28, 2020 18:31
@Jiaming-X Jiaming-X force-pushed the container_width_responsive branch from 2db4bc9 to 4a9b46b Compare May 29, 2020 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants