search

codeforamerica / codeforamerica.org-archive

The (former) @codeforamerica website
http://archive.codeforamerica.org
Other
176 stars 135 forks source link

Load more button on Apps page doesn't work; throws Javascript error. #790

Open krusynth opened 9 years ago

krusynth commented 9 years ago

Clicking the "Load More" button on the apps page throws Uncaught TypeError: Cannot read property 'offsetHeight' of null in Chrome 38.0.2125.111 / OS X 10.9.4.

migurski commented 9 years ago

Hi, can you share the URL you're looking at?

krusynth commented 9 years ago

On closer inspection, all four section pages have the js error on load (not on click), and the Load More button just is just returning to the top of the page (no event attached maybe?).

JayK31 commented 9 years ago

Doesn't seem to be a need for any of the Load More buttons at this point, as there are no more apps to load on any of the pages.

As for the error, in global.js line 77: var maxheight = document.getElementById('fade').offsetHeight + 15; It looks like there are no fade elements on any of those pages, thus throwing the JS error.