Skip to content
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

Update Passport icon and product image #4035

Merged

Conversation

sethforprivacy
Copy link

Updates the product image and logo for Passport to better align with the design principles of the website and be a bit higher quality.

Tested both locally before PRing and they look great, are the correct sizes, and have been compressed with optipng -o7.

@sethforprivacy
Copy link
Author

Rebased to exclude the doc changes made in #4034 from this PR to allow them to be separately reviewed/merged.

@sethforprivacy sethforprivacy changed the title Update-passport-images Update Passport icon and product image May 1, 2023
@sethforprivacy
Copy link
Author

@Cobra-Bitcoin @crwatkins just for clarity we're planning on sharing that we've been added and give a shoutout this week, would be awesome if we could get this merged before then!

This PR should be fully inline with your spec and requirements for images, FWIW.

@crwatkins
Copy link
Contributor

LGTM. Thanks.

@sethforprivacy
Copy link
Author

LGTM. Thanks.

Awesome, thanks for the quick review!

@Cobra-Bitcoin
Copy link
Contributor

Very nice. Thank you.

@Cobra-Bitcoin Cobra-Bitcoin merged commit 4e6d955 into bitcoin-dot-org:master May 1, 2023
1 check passed
@sethforprivacy
Copy link
Author

Thanks for the quick merge!

@sethforprivacy sethforprivacy deleted the update-passport-images branch May 1, 2023 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants