Skip to content
This repository has been archived by the owner on Nov 21, 2018. It is now read-only.

Add in random logo from iojs/logos to page header #12

Closed
wants to merge 5 commits into from
Closed

Add in random logo from iojs/logos to page header #12

wants to merge 5 commits into from

Conversation

therebelrobot
Copy link
Contributor

also added github fork banner to corner

@therebelrobot
Copy link
Contributor Author

So I tried to keep the js on the front end to a minimum, to prevent the need for anyone later needing to learn a whole framework to work with it, also went with browserify because it more "node-like", allowing any iojs dev to hop in and understand how it works.

I'm pulling from the Github public API the contents of iojs/logos, all of which should be licensed in CC0.

The caveat of using the public API with no authentication is you run the risk of hitting rate limits, hence the store dependency, to store the api returns for future use. This of course can be removed if the rate limit is not a concern, I was just planning for any immediate issues that may arise from it.

As for the rest of the dependencies, domready is being used to simply listen for the dom ready event, lodash is being used to work with the api collections sanely, moment is being used in connection with the API to determine when to access it again after rate limiting, superagent is being used for external calls, and zepto-browserify is being used to access the dom (Zepto because jQuery is a bloated beast, and zepto-browserify because i tried using just zepto, but it's not browserifyable currently). All of these dependencies can be replaced or removed, they just got the job done quickly and efficiently for what I was trying.

@therebelrobot
Copy link
Contributor Author

I'm probably going to close this myself for now, with no direction yet from #14 , and the fact that I worked on this in my master branch. #n00bmistake

snostorm pushed a commit that referenced this pull request Mar 12, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants