midterm

[k-kawa-gucci]

No description provided.

[epilande]

Your nav styles are under your Base Typography section. It would be better to move this down to Site Specific css section.

[epilande]

I've added some feedback.

Other comments:

• I see you're using two different stylesheets for your home & about page. It's not necessary to use two, it would be better to use one stylesheet, that way you can reuse styles, & it will be easier to maintain.
• Your images have white background. We want to save out the images without the white background (instead of white, make it a transparent background), that way we can use it on any color background.
• Currently you have section01 grouping the hero/banner & 3 columns section. It would be easier to style them if you separate the 3 columns into a section02.

[k-kawa-gucci]

Thank you ill apply the changes

On Tue, Mar 22, 2016 at 12:35 PM, Emmanuel Pilande <notifications@github.com

wrote:

I've added some feedback.

Other comments:

• I see you're using two different stylesheets for your home & about
  page. It's not necessary to use two, it would be better to use one
  stylesheet, that way you can reuse styles, & it will be easier to maintain.
• Your images have white background. We want to save out the images
  without the white background (instead of white, make it a transparent
  background), that way we can use it on any color background.
• Currently you have section01 grouping the hero/banner & 3 columns
  section. It would be easier to style them if you separate the 3 columns
  into a section02

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#1 (comment)