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

Fixed disappearing menu. #373

Closed
wants to merge 3 commits into from
Closed

Fixed disappearing menu. #373

wants to merge 3 commits into from

Conversation

ZenHarbinger
Copy link
Contributor

Fix for issue 372

#372

Force .author_urls to show() when width is wide enough.
Force .author_urls to hide() when not.

@ZenHarbinger
Copy link
Contributor Author

Switch from using window width to button visibility for if the author urls are shown.

@ZenHarbinger
Copy link
Contributor Author

Sorry for the redundant pulls.
Cleaned up _main.js
Added code straight to main.min.js

@mmistakes
Copy link
Owner

Thanks. I merged this in before you updated the pull so if you wouldn't mind resubmitting it since I updated some JS dependencies that are likely missing in your build of main.min.js. If you pull from master you should get them and then I can merge in the updated PR.

cjmadsen pushed a commit to cjmadsen/cjmadsen.github.io that referenced this pull request Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants