HNGlance

Top Best New Show

Show HN: a simple YouTube jukebox built with AngularJS

https://github.com/jgthms/juketube

by bbx • 13 years ago

2 8 13 years ago

GI

Related Stories

Loading related stories...

Comments

8

tharshan095• 13 years ago

Great work! Lots of little improvements to be made but its a great start. I am glad its in AngularJS and open source, looking forward to see it under the hood.

bbx3• 13 years ago

Thanks. Can you tell me what kind of improvements can be made? I'm not very acquainted with AngularJS best practices.

tharshan095• 13 years ago

Well if you plan on expanding the app, you should split the app.js into separate files. The VideosController does not need an init() function. You can simply take those lines out of the init function and delete the call to the init function and leave them separate - they will be called when the app is loaded. Nice work otherwise!

Source preview

github.com

HNGlance

© 2024-2026 HN Glance. Story content belongs to its original sources.

Privacy
Terms