[GALLERY] Article page launch blocking final

ampproject / ampstart

AMP Start source code and templates .

https://ampstart.com/

Apache License 2.0

419 stars 152 forks source link

[GALLERY] Article page launch blocking final #786

Closed spacedino closed 6 years ago

spacedino commented 6 years ago

2 bugs

[ ] somehow now the carousel doesnt flush with the left edge of the page on page load for certain device sizes...is this worth looking into? Galaxy 5s below 340x640

SHOULD BE

[ ] The pineapple image is too big - it didnt get cropped or sized correctly. Mobile should be cropped. Desktop should be scaled down and cropped. It's too overwhelming.

CURRENT

SHOULD BE

spacedino commented 6 years ago

Moving this to non-launch blocking. The way this was implemented makes it not worth fixing right now.Marking as resolved for now after talk with @camelburrito

cc @ericlindley-g