Skip to content
This repository was archived by the owner on Jun 18, 2024. It is now read-only.

mobile nav fix #42

Merged
merged 3 commits into from
May 17, 2013
Merged

mobile nav fix #42

merged 3 commits into from
May 17, 2013

Conversation

awolfe76
Copy link

@benbalter there is an issue with the navigation on mobile devices.
This fixes that. Should take a look and merge ASAP.

/*!
* Bootstrap v2.3.1
* Bootstrap v2.1.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Purposely downgraded?

@awolfe76
Copy link
Author

Yeah, for now.

Just wanted to get the fix first. Couldn't find what was going on, but going backwards to this older (but known working) version fixed it.

I'll do some more clean up, and re-upgrade, shortly.

Sound ok?

@awolfe76
Copy link
Author

That should get everything back in line.

@jpmckinney
Copy link
Contributor

FYI, here is a diff that ignores whitespace changes

@awolfe76
Copy link
Author

@benbalter can you pull down and run local to make sure? It's much cleaner now.

@benbalter
Copy link
Contributor

Looks good to me. 👍 to merge.

screen shot 2013-05-17 at 10 40 17 am

@haleyvandyck
Copy link
Contributor

Thanks again @awolfe76 for the help with this! 🇺🇸

haleyvandyck pushed a commit that referenced this pull request May 17, 2013
@haleyvandyck haleyvandyck merged commit 00c6c10 into project-open-data:master May 17, 2013
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.

5 participants