> I do mix spaces and tabs in my IDE (oh the horror) so this output Problem solved: http://prettydiff.com/?m=beautify&s=http://www.cancelbubble.com/hackernews/javascript/global.js
Thanks, Austin Cheney, CISSP -----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of cancel bubble Sent: Wednesday, October 26, 2011 9:05 AM To: The JSMentors JavaScript Discussion Group Subject: [JSMentors] I "redesigned" Hacker News - would love a JS code review The "redesign": http://www.cancelbubble.com/hackernews/ I made a quick screencast (link top left) walking through the features. Hacker News thread just posted: http://news.ycombinator.com/item?id=3158343 Might have a little more context, but the screencast pretty much covers it as well. The JavaScript file: http://www.cancelbubble.com/hackernews/javascript/global.js I do mix spaces and tabs in my IDE (oh the horror) so this output, while not horrible, is not perfect (looks fine in the IDE). I also posted up a jsFiddle where I cleaned up some of the spacing issues: http://jsfiddle.net/wPwbF/ If you have suggestions for improvements, you could make them here so I can see what you're talking about and what you did. Thanks much! -- To view archived discussions from the original JSMentors Mailman list: http://www.mail-archive.com/[email protected]/ To search via a non-Google archive, visit here: http://www.mail-archive.com/[email protected]/ To unsubscribe from this group, send email to [email protected] -- To view archived discussions from the original JSMentors Mailman list: http://www.mail-archive.com/[email protected]/ To search via a non-Google archive, visit here: http://www.mail-archive.com/[email protected]/ To unsubscribe from this group, send email to [email protected]
