search

nodejitsu / browsenpm.org

Browse packages, users, code, stats and more the public npm registry in style.
browsenpm.org
MIT License
44 stars 9 forks source link

Header images should be in header #7

Closed indexzero closed 10 years ago

indexzero commented 10 years ago

@Swaagie I've updated the header items by commenting out the ones we are not using yet, but we need the header images:

screen shot 2014-04-26 at 11 52 00 pm

Swaagie commented 10 years ago

these images should be served via our versions cdn. As soon as we have browsenpm.org setup, we need to tell the cdn that it has a new resource and consequently it will fetch the source, see https://github.com/nodejitsu/nodejitsu-app/commit/5f59da73dfeb30b3fb528a0034936cfd5ad223f6. If you'd like to see how it looks locally (before our cdn caches it), change this line to '' https://github.com/nodejitsu/contour/blob/master/assets/npm/defaults.styl#L17

Swaagie commented 10 years ago

Fixed for now by not depending on the CDN, can always do that later