Freemius / wordpress-sdk

https://freemius.com
GNU General Public License v3.0
267 stars 77 forks source link

Add-on banner image size issues #423

Open dgwyer opened 4 years ago

dgwyer commented 4 years ago

🐞 bug report

Source: https://freemiusdev.slack.com/archives/CHW7SSTQD/p1592341490425900

Behavior:

Fix:

Versions: (*)

dgwyer commented 4 years ago

@fajardoleo Do you think there would be any issues with just updating the values in add-ons.scss to these new values:

Main banner width: 821 -> 772 Card banner width: 310 -> 300

This would bring the SCSS inline with the image dimensions uploaded via the Developer Dashboard.

dgwyer commented 4 years ago

I've tested the card banner change which doesn't seem to have any impact for single/multiple add-ons. However, the iframe width was changed from 772px to 821px around a year ago (by Leo).

At 772px width it breaks the iframe so I'd recommend instead altering the banner image size on the Developer Dashboard to accept an 821px wide image. (Open new Asana ticket to address this)

I'll submit a PR for the card banner image to render 300px wide instead of 310px.