Skip to content

Rebranding styles #35

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

Merged
merged 5 commits into from
Nov 2, 2018
Merged

Rebranding styles #35

merged 5 commits into from
Nov 2, 2018

Conversation

ticidesign
Copy link
Contributor

@ticidesign ticidesign commented Oct 24, 2018

screen shot 2018-10-25 at 12 01 38 pm

@ticidesign ticidesign requested a review from ButenkoT October 24, 2018 02:56
index.html Outdated
</li>
<li class="facebook">
<span>Facebook:</span>
<a href="https://www.facebook.com/MusesCodeJSAustralia" target="_blank" rel="noopener noreferrer">MusesCodeJSAustralia</a>
Copy link
Member

Choose a reason for hiding this comment

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

facebook is MusesCodeJS

color: white;
display: flex;
flex-direction: row;
font-family: Georgia, serif;
font-family: 'Source Sans Pro', sans-serif;
Copy link
Member

Choose a reason for hiding this comment

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

our second font, and font for instructions should be 'Avenir'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Avenir ins't a google font https://fonts.google.com/?query=Avenir

Copy link
Member

Choose a reason for hiding this comment

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

You can put Avenir as 1st choice, then keep Source Sans Pro. It works even without adding google font link

@ButenkoT
Copy link
Member

Also can you add screenshot to description :) Thanks!

@ButenkoT
Copy link
Member

ButenkoT commented Oct 25, 2018

I think we can do footer of one of our main colors? What do you think? Or it looks too grey? @ticidesign

ButenkoT
ButenkoT previously approved these changes Nov 2, 2018
@ButenkoT ButenkoT merged commit 9d309ac into master Nov 2, 2018
@ButenkoT ButenkoT deleted the rebrand branch November 2, 2018 04:46
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.

2 participants