search

ga-wdi-boston / html-css-layout-bootstrap-sass

Other
1 stars 163 forks source link

017/solution #33

Closed Jcornmanhomonoff closed 7 years ago

Jcornmanhomonoff commented 7 years ago

Update solution.

Jcornmanhomonoff commented 7 years ago

I did not make these changes for lm01 as I ran out of time. I can address them for 017 if you like. I made a few small changes that rearrange the HTML so that it follows the README instructions more closely.

jrhorn424 commented 7 years ago

@Jcornmanhomonoff I'll go ahead and update the solution and close this. If you'd like to address my comments for 017 delivery, that's great. I'll open an issue and assign you. If you don't get to it this time, no big deal.

jrhorn424 commented 7 years ago

@Jcornmanhomonoff Please target solution instead of master next time. Thanks!