Conversation
GWG logo assets added 9211839 |
Signed-off-by: iboonox <iheb.dev@gmail.com>
Signed-off-by: iboonox <iheb.dev@gmail.com>
Signed-off-by: iboonox <iheb.dev@gmail.com>
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.
@iboonox these look fantastic IMO 🏆
One note, the assets/logos/OGO/styleguide.svg
file looks like a different typeface than Lato is used in the SVG image itself. GitHub won't allow me to make a suggested change on a binary file – do you see what I mean?
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.
Never mind, I get it. While the GWG styleguide is vectorized, the OGO styleguide SVG is not, and I didn't have Lato font installed. We may want to do the same for the OGO styleguide, but we can do refinements in a follow-up PR. This is 💯 🏆
* Add OGO logo assets Signed-off-by: iboonox <iheb.dev@gmail.com> * Add GWG logo assets + change assets folder structure Signed-off-by: iboonox <iheb.dev@gmail.com> * Fix source folder path Signed-off-by: iboonox <iheb.dev@gmail.com>
@scottrigby: does this change with the naming change as well? |
@amye I believe we should be set with these designs:
If I understood what you meant correctly |
That is what I was asking yes! As for the 2nd, having all the artwork in one place is fine, put in a PR under 'other' in /artwork |
@amye OK perfect 👌 PRs made: I'll also post in the TOC project onboarding issue |
No description provided.