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 gitcoin.co/presskit to reflect Gitcoin logo and Gitcoin DAO brand updates #9696

Merged
merged 26 commits into from
Nov 19, 2021

Conversation

mlvein
Copy link
Contributor

@mlvein mlvein commented Nov 17, 2021

Description

Updated the following on gitcoin.co/presskit

  • updated new logo assets to use updated logo
  • mission and about links
  • updated colors section (colors and hex + RGB + hsla values)
  • added new download links to Gitcoin DAO badges
  • updated Illustration section to DAO Brand Guide
  • added download link to Gitcoin DAO Brand Guide PDF
Refers/Fixes
Testing

https://www.loom.com/share/d403e5ff0dfc4dedb67fe341d546830d

mlvein added 24 commits May 14, 2021 17:26
updated images for logo usage guide on press kit
update on logo, brand usage, and assets
updated html + scss to make it more up to spec to figma design
deleted unusued illustration svgs
- fixed offset for h2 on usage.html
- removed duplicate .presskit class on presskit.css
- moved resource.html section to top and changed content
- added links and new illustrations
- updated text sizes (font-bigger-1) and spacing
removed duplicate .list-group-item class on presskit.scss
- removed unused images/assets
- fixed misspelled image filename
added animated hero svg
hero_svg anim update
- updated logo text
- links padding (pt-3 pb-2) on link items
- added links to about and mission that opens to new tab
- added hover style
Gitcoin DAO Brand update
- new logo
- added links to Gitcoin DAO badges
- updated usage illustrations
- updated colors and hex, rgb, hsla values (in views.py)
- added DAO brand guide PDF download
Copy link
Contributor

@PixelantDesign PixelantDesign left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Could you help review @thelostone-mc, TY!
@melvinalvarez could we make sure thee GTC token banner isn't in the PR?

removed shared/top_nav.html which contains the persistent QL banner
@mlvein
Copy link
Contributor Author

mlvein commented Nov 17, 2021

Looks good to me! Could you help review @thelostone-mc, TY! @melvinalvarez could we make sure thee GTC token banner isn't in the PR?

got it - removed 'shared/top_nav.html' from presskit.html

Copy link
Contributor

@gdixon gdixon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thank you @melvinalvarez! 🚀

@thelostone-mc thelostone-mc merged commit a5dffcf into gitcoinco:master Nov 19, 2021
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.

4 participants