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

Added collapsable nav bar. Moved buttons to line up #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Dmart331
Copy link

Navigate to lib folder. Run npm install then bower install Really all I changed was adding the collapsable nav when the browser window is smaller than a computer. Viewing on the phone should be a little smoother now. Let me know if I completely messed something up.

@brollb
Copy link
Contributor

brollb commented Mar 29, 2017

It's showing up a little funny for me:
snap0006

  • The white gap below the header is a little odd
  • The css updated for the anchors are showing up a little on the small side
  • the added left margin to the logo looks better
  • The header fonts look a little on the smaller side (esp in the section headers)

@Dmart331
Copy link
Author

Dmart331 commented Mar 29, 2017 via email

@brollb
Copy link
Contributor

brollb commented Mar 29, 2017

no worries!

@Dmart331
Copy link
Author

Dmart331 commented Mar 29, 2017 via email

@brollb
Copy link
Contributor

brollb commented Apr 2, 2017

A couple comments:

  • The header text in the banner seems to be the same size as the subtitle beneath it
  • the logo is not vertically centered (looks a little funny)
    snap0009
  • when I make the screen smaller, the collapsing nav button also hides the logo
    snap0011

snap0010

The main screenshot seems much more responsive though!

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