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

make feature notification modal responsive #84

Merged
merged 2 commits into from Apr 25, 2016

Conversation

Projects
None yet
2 participants
@chinaowl
Contributor

chinaowl commented Apr 20, 2016

As a first step to making the whole site responsive, I started with the feature notification modal. (See #26.) Now it should be possible to close the modal in mobile.

Question: do I have to include the dist folder in this PR? dist/main.css was modified, but seeing as it's a generated file, I didn't want to include it.

image

Note: this was only tested in Chrome and Safari on iOS. Also, sorry about the extra merge commit...I'm not sure how to get rid of it.

@glauberramos glauberramos merged commit 1f81bee into funretro:master Apr 25, 2016

@glauberramos

This comment has been minimized.

Show comment
Hide comment
@glauberramos

glauberramos Apr 25, 2016

Collaborator

Thanks for the pull request :D

Collaborator

glauberramos commented Apr 25, 2016

Thanks for the pull request :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment