thinkshout / feeding-texas

Home of the Feeding Texas website. Development by ThinkShout.
feedingtexas.org
5 stars 12 forks source link

QA: SNAPshot Texas page hero image height #124

Closed jcdwyer1 closed 10 years ago

jcdwyer1 commented 10 years ago

http://feedingtexas.org.s3-website-us-east-1.amazonaws.com/learn/communities/snapshot-texas.html

The height of this banner image is larger than the others in this directory.

jcdwyer1 commented 10 years ago

The banner image also obscures the zip code widget text.

heypaxton commented 10 years ago

c2046bd

heypaxton commented 10 years ago

Related to #137

heypaxton commented 10 years ago

48fd5f7

heypaxton commented 10 years ago

@jcdwyer1 This is live on staging and ready for review. stage.feedingtexas.org.s3-website-us-east-1.amazonaws.com/learn/communities/snapshot-texas.html

jcdwyer1 commented 10 years ago

Thanks, @heypaxton but when I changed the banner image to make it consistent with other pages in the subdirectory it happened again. Am I doing something wrong with the banner tag?

snapshot2

jcdwyer1 commented 10 years ago

Related: The same issue is occurring on the Get Help page. Both pages utilize the zip code search widget. http://feedingtexas.org.s3-website-us-east-1.amazonaws.com/help/

heypaxton commented 10 years ago

@jcdwyer1 There seems to be a difference in whats in staging and live. I'll update you once this is pushed to live.

heypaxton commented 10 years ago

@jcdwyer1 this is on staging and should be on pushed to live in a short moment. For now you can review at the following URL: http://stage.feedingtexas.org.s3-website-us-east-1.amazonaws.com/help/

jcdwyer1 commented 10 years ago

Perfect, thank you. This page has a lonely secondary link navigation issue similar to #149, which I assume will go away once @caxy4 adds logic removing it?

alexellison commented 10 years ago

I assume will go away once @caxy4 adds logic removing it?

Correct. Pushing this and the rest of my morning's work live shortly.

alexellison commented 10 years ago

Closing as the lonley secondary nav link has been removed.