-
Notifications
You must be signed in to change notification settings - Fork 7
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
[WIP] overhaul to use SASS, Compass, etc #82
base: master
Are you sure you want to change the base?
Conversation
…eaks. codebase is solid tho.
…s, applied material textures, fixed up css conflicts and funny padding oddities, made a mobile layout, applied google's material typography specs to font
In terms of functionality, things seem to work but I noticed that the result message after you share is blank. Presumably it's looking for a div with a particular ID that got removed. As for layout, I really like the look/aesthetic but some layout bugs have been introduced. Here are some screenshots from my testing showing problems on some screen sizes: |
Thank you I really appreciate the feedback. I had to resolve some other issues this week but I will have lots of time today and this weekend to resolve all these issues. |
Hi, I forked this off of master today and made sure it had all the same functionality (and even discovered a bug in the process.) |
I created this PR so that I could make comments (since you can't use the issue tracker on a fork).