search

cascadiajs / 2015.cascadiajs.com

The official repo for the CascadiaJS 2015 conference
MIT License
54 stars 29 forks source link

Feature: speaker rotation #372

Closed alexlande closed 9 years ago

alexlande commented 9 years ago

This adds a quick random speaker display to the homepage. Picks a random speaker for each day + displays them. All of the speakers are in the page HTML but are hidden by default.

/cc @davethegr8 @crtr0

alexlande commented 9 years ago

Ugh, forgot about images. They're all getting downloaded. Hold off on merging this, will fix.

alexlande commented 9 years ago

Ok, added lazy loading for the homepage speaker images so only the active one is loaded.

davethegr8 commented 9 years ago

I'm curious - for you, what's the pros and cons to putting all the content in the document and showing one vs filling in data from a json file?

alexlande commented 9 years ago

Based the decision entirely on simplicity. If we built them dynamically, you've got to do string concatenation or front-end templating, then you've got to keep that in sync with the backend template we're using, since they wouldn't be shared.

With this method the document size is slightly larger than the other way, but it should gzip really well so it shouldn't be a real issue.

davethegr8 commented 9 years ago

Fair enough, sounds like a reasonable solution. Thanks!