QA: Hero images on top-level pages (/learn, /work etc) do not stretch across full screen - Githubissues

thinkshout / feeding-texas

Home of the Feeding Texas website. Development by ThinkShout.

feedingtexas.org

5 stars 12 forks source link

QA: Hero images on top-level pages (/learn, /work etc) do not stretch across full screen #85

Closed jcdwyer1 closed 10 years ago

jcdwyer1 commented 10 years ago

Not sure if this is a code issue or the images are sized incorrectly. hero_stretch

Latest chrome

heypaxton commented 10 years ago

7a7589d

heypaxton commented 10 years ago

JC This has been updated and ready for you to review.

jcdwyer1 commented 10 years ago

New images look great, thank you!

Your selection for the home page brings up a new issue though - the text above the zip code lookup ("Get statistics for your community:") is very difficult to read. It wasn't easy to read with the old image either, so maybe this text just needs a different style?

heypaxton commented 10 years ago

Sure thing. I'll push an update with the upcoming code commits.

heypaxton commented 10 years ago

1a83274

heypaxton commented 10 years ago

@jcdwyer1 This is ready for you to review. I updated the text style and also added a treatment to the splash background image.

jcdwyer1 commented 10 years ago

Everything looks great, especially the splash page. The only image I might want to change is the sunflowers (not typically considered food in Texas), but we can talk about how I manage images on our call this afternoon.