ADI-Labs / density

wireless density API
13 stars 26 forks source link

Add Navbar. Refactor logo and about info. Fix map bug. Minor padding adjustments. #161

Closed raymondxu closed 9 years ago

raymondxu commented 9 years ago

Review: @evantarrh @danrschlosser @natebrennand

Closes #143, #146, #150, #152, #160

Nav bar (screenshots updated to show new ordering): screen shot 2015-04-17 at 4 37 49 pm

About page is currently a little bland, but the page is there now. We can add more info later. screen shot 2015-04-17 at 4 37 55 pm

Mobile views: img_0376 img_0379 img_0378 img_0377

Map bug is fixed by waiting until the SVG is loaded before calling the JS.

  function checkReady() {
    if (document.getElementById('map').getSVGDocument() == null) {
      setTimeout(checkReady, 5);
    }
    else {
      run();
    }
  }
natebrennand commented 9 years ago

can you include mobile screenshots as well / ack that you checked them?

raymondxu commented 9 years ago

On it

raymondxu commented 9 years ago

Also will change the order to 'HOME - MAP - ABOUT - API' as per @danrschlosser's opinion

jzf2101 commented 9 years ago

:+1:

swbae31 commented 9 years ago

:+1: