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

W3-A3__sass-and-jquery Zach Gordon/Lyle Scott (FINAL) #23

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

Conversation

LyleCharlesScott
Copy link

@blackfalcon, @miketierney, @jedfoster

We did a significant amount of learning and refactoring. It was uphill, as we kept trying to use SASS and it repeatedly broke things that were working previously! For the most part we took it as a learning exercise and took the practice. For example, _responsive-nav.scss section broke the nav bar repeatedly when we tried to use SASS shortcuts, although it was working fine previously with CSS. So we never used our refactoring attempts on that section.

desktop-nav
desktop-sign-up
graph-animation
mobile nav
mobile-graph-animation
mobile-sign-up
portfolio-filter

The JS code for the pricing tables was done by Lyle. The code for the progress bars in the about section and the filter in the Portfolio section were both done by Zach.

LyleCharlesScott and others added 30 commits September 8, 2015 09:23
Add title and design from different branch
@LyleCharlesScott LyleCharlesScott changed the title A2-W3 Sass & JQuery (WIP) W3-A3__sass-and-jquery (WIP) Sep 18, 2015
@LyleCharlesScott LyleCharlesScott changed the title W3-A3__sass-and-jquery (WIP) W3-A3__sass-and-jquery Zach Gordon/Lyle Scott (FINAL) Sep 18, 2015
@LyleCharlesScott
Copy link
Author

@ZachKGordon

Breadcrumbs to find the pull request.

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.

1 participant