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

Revamp GUI to be homogeneous #112

Merged
merged 12 commits into from
Jun 19, 2014
Merged

Conversation

ElDeveloper
Copy link
Member

The main changes to note are:

  • Re-organization of the navigation bar.
  • Addition of favicon.
  • Addition of explanatory text about Qiita to index.html
  • Indentation and styling of the HTML code in sitebase.html
  • Cleaner overall look of the site.

@ElDeveloper
Copy link
Member Author

Do not merge unless #106 is merged!

@ElDeveloper
Copy link
Member Author

@squirrelo would be nice if you could take a look!

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.18%) when pulling 03c9334 on ElDeveloper:madne-css into bcf4a24 on biocore:master.

@ElDeveloper
Copy link
Member Author

In addition to the changes I mentioned above, I've also made the "create study" step a simple modal view.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.18%) when pulling a3a632f on ElDeveloper:madne-css into bcf4a24 on biocore:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.18%) when pulling a3a632f on ElDeveloper:madne-css into bcf4a24 on biocore:master.

@antgonza
Copy link
Member

+1

1 similar comment
@teravest
Copy link
Contributor

👍

@@ -1,21 +0,0 @@
{% extends sitebase.html%}
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this getting deleted?

Copy link
Member Author

Choose a reason for hiding this comment

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

I made a modal view out of this template.

@ElDeveloper
Copy link
Member Author

BTW what about changing the Qiita logo to have a clear version i. e. instead of:

screen shot 2014-06-19 at 9 42 51 am

Changing it to:

screen shot 2014-06-19 at 9 42 22 am


I personally like the white font with the dark background more.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5aaa333 on ElDeveloper:madne-css into baf3432 on biocore:master.

@ElDeveloper
Copy link
Member Author

Should be ready to be merged qiitas.

antgonza added a commit that referenced this pull request Jun 19, 2014
@antgonza antgonza merged commit 4491e7c into qiita-spots:master Jun 19, 2014
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.

5 participants