Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

Design update #344

Merged
merged 31 commits into from
Jul 26, 2016
Merged

Design update #344

merged 31 commits into from
Jul 26, 2016

Conversation

prayerslayer
Copy link
Contributor

@prayerslayer prayerslayer commented Jul 9, 2016

Ignore that the code is a mess for a second. I'd like to make our design more consistent (colors mostly), the logo less oldschool (until we get a "real" one) and the bootstrap theme less bootstrappy.

image

image

Comments welcome. If you check the PR out and run it yourself you see that the frontpage also has animations 😱

(If the theme reminds you of babel it's because of the color choice. Also I adopted their headline (X "is a" Y) because I like its simplicity.)

@prayerslayer
Copy link
Contributor Author

@mfellner @spaudanjo

@mfellner
Copy link
Contributor

Looks good! A few points:

  • The final logo is probably going to use a different primary color, can we easily change it?
  • GitHub should always be written with a capital H.
  • We probably want to use a catchier headline than "Zappr is a GitHub integration".
  • "approvals" in "Pull Request approvals" should not be underlined to be consistent with the other headlines.
  • Maybe we can also already put the "sync repos" button on the top of the repo list (I know...)?

@prayerslayer
Copy link
Contributor Author

prayerslayer commented Jul 11, 2016

The final logo is probably going to use a different primary color, can we easily change it?

Pretty much. The updated logo is a vector, so if we were to keep it (just because, idk) we have to change it there. In the stylesheet I'm currently using CSS variables, so it's easy there. (We will need to exchange them for something else, maybe a proper LESS/postcss pipeline, due to lacking browser support.)

GitHub should always be written with a capital H.

Will change.

We probably want to use a catchier headline than "Zappr is a GitHub integration".

I like that it's simple. When we go for something catchy, people may get confused about what Zappr actually is. What it does is described just underneath.

"approvals" in "Pull Request approvals" should not be underlined to be consistent with the other headlines.

Agreed, will change.

Maybe we can also already put the "sync repos" button on the top of the repo list (I know...)?

Oh, yesyesyes!

@prayerslayer
Copy link
Contributor Author

Updated the screenshots, note that they are from a bigger screen now.

@prayerslayer
Copy link
Contributor Author

Draft of logo/mascot from @kolja:

zappr

@prayerslayer
Copy link
Contributor Author

Updated initial comment in PR with screenshots.

@prayerslayer
Copy link
Contributor Author

Updated with new mascot 🎉 @mfellner @spaudanjo

@mfellner
Copy link
Contributor

👍

1 similar comment
@prayerslayer
Copy link
Contributor Author

👍

@prayerslayer prayerslayer merged commit f0dc18f into master Jul 26, 2016
@prayerslayer prayerslayer deleted the design-update branch July 26, 2016 17:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants