News for hackers. Stuff that matters.
Re:Please review my weekend project. 100% client side webapp... by stevelosh on Wednesday May 11, 2011 @12:30PM (#2537072)
A few (minor) things:

The url is tvdinnr but the logo is tvdinner (notice the e). This makes it harder to remember the URL.

The search button isn't wide enough and looks broken (Firefox, OS X): http://d.pr/UHUT

When I search there's a weird red box in the upper right: http://d.pr/R72E

The "most discussed" videos have no comments, which seems strange.

My browser's back button works, but Cmd+Left doesn't. Is something trapping keypresses?

As someone else mentioned, there's a horizontal scrollbar when there doesn't need to be.

Re:Please review my weekend project. 100% client side webapp... by ljlolel on Wednesday May 11, 2011 @02:12PM (#2537560)
Thanks.

Yea, it's a drop-in temporary logo and the orange box is the AddThis failing for some reason.

I'm not a designer. I changed #content to auto. The search youtube button overflows when text gets too big (looks normal for me), but I want it to right-align with the video, and maximize search box length. How would I best do that?

No keypresses are captured. It registers hashchanges. The Cmd+Left is probably because it goes back to root tvdinnr.com but doesn't redirect again. I fixed that. Thank you.

The most discussed issue is probably just the youtube request failing. I should create error handlers for that.

Re:Please review my weekend project. 100% client side webapp... by qeorge on Wednesday May 11, 2011 @07:29PM (#2538849)
This CSS works for me:

http://www.pastie.org/1891062

Try bumping the font-size on #header #search up to see things resize.

Worth knowing, Helvetica is wider than Arial, which is why it might look fine on Windows and too wide on Mac.

Re:Please review my weekend project. 100% client side webapp... by JoeAltmaier on Wednesday May 11, 2011 @01:22PM (#2537280)
IE 9 - see the page, no action when hitting search