-
Notifications
You must be signed in to change notification settings - Fork 10
Added logo dimensions #465
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
base: main
Are you sure you want to change the base?
Added logo dimensions #465
Conversation
✅ Deploy Preview for incomparable-tiramisu-91a96a ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| | **Link to the merchant's website or social network** | <Cnd /> | **Recommended**. Provides context on the merchant's activity and may be requested during the review. May be requested after a risk review if it wasn't provided before. | | ||
| | **Payment frequency** | <Cnd /> | Defines whether payments are mainly one-off or recurring (e.g. subscriptions). | | ||
| | **Logo** | <Opt /> | A `.png` file under 1 MB, displayed on the payment page and the mandate PDF. | | ||
| | **Logo** | <Opt /> | A `.png` file under 1 MB and with the recommended dimensions of 500x500 pixels, displayed on the payment page and the mandate PDF. | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please ask Pierre whether 500*500 is the recommended dimensions, or the maximum recommended dimensions? It makes a big difference for partner design teams :)
max-koro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Quick question we should ask Pierre before approval though 🙏
He's OOO, so we can ask Mia. |
Only updated the dimensions because the other logo details were added in a previous ticket.