update org description, remove client section, fix partner images#4
update org description, remove client section, fix partner images#4citizendrummer wants to merge 1 commit intomasterfrom
Conversation
…es, remove sitecake
|
@arelia @DerekTownsend @briberri could someone review these changes then merge? Thanks! |
|
@citizendrummer good job with the clean up. For future pull requests, it might be easier to review if you break the changes up into separate commits or even separate PRs. That way, if anything breaks it's easier to determine which change caused bugs. It would be easier to review and approve the Sitecake changes because the PR would only include |
|
@citizendrummer Also, include a link to a test site so that we can see how the changes will affect the site, like @jabari-austin did here: #3 |
| @@ -8,13 +8,10 @@ | |||
| } | |||
|
|
|||
| .img-partners { | |||
| display:inline-block; | |||
| margin:25px; | |||
| display: block; | |||
There was a problem hiding this comment.
this is the type of change I'd want to see on a test site first, will you please add one?
|
Thanks @arelia. I updated the PR with link to test site. Do you want me to redo all the changes separately? |
2 participants
update organization description
remove client section
fix partner image styles
remove sitecake files and markup
https://preview.c9users.io/citizendrummer/cs-new-mission/index.html