Hacker News Board Hacker News, browsed like a classic message board
Live data from Hacker News

Interactive Résumé of Robby Leonardi

rleonardi.com

Post reply on HN
Print view View original

11–20 of 22 posts

Re: Interactive Résumé of Robby Leonardi

#11

Post by at-fates-hands »

onion2k wrote: post #3

It's a fun and cute resume, but delving in to the code you see things like; function blinkRobbyEyes() { if (layersMovement != "not moving 2") { $(robbyEyesCloseDiv).fadeTo(0,1); $(robbyEyesCloseDiv).stop().delay(300).animate({opacity: 0}, 0, function() {}); } } "if (layersMovement != "not moving 2")" is a boolean that should be passed as a parameter if you're ever going to write a working unit test against that funct…

I've always found this interesting. If you're a developer, it should be your number one priority to make sure the code you write for your own site be top notch. God knows how much we all complain about other people's code, you should know however cool it is, developers are going to crack the hood and see how good the code is.

Great idea and quite original. I still give him rube cred for having such a nice concept despite the flaws in his code.

Re: Interactive Résumé of Robby Leonardi

#12

Post by jebus989 »

Dirlewanger wrote: post #10

No, it's really not. It's tacky and takes too long to get all of the relevant details...and they're not even all in one place. Everything a resume should do, this does not. This being included in a sampling of workpieces is a different story. Not to mention it's a repost.

We know this isn't designed to convey work experience, skillset in an efficient and clear way, it's just attention-grabbing hit bait so he can make some claim of "going viral" on his real CV.

Re: Interactive Résumé of Robby Leonardi

#15

Post by Banzai10 »

onion2k wrote: post #3

It's a fun and cute resume, but delving in to the code you see things like; function blinkRobbyEyes() { if (layersMovement != "not moving 2") { $(robbyEyesCloseDiv).fadeTo(0,1); $(robbyEyesCloseDiv).stop().delay(300).animate({opacity: 0}, 0, function() {}); } } "if (layersMovement != "not moving 2")" is a boolean that should be passed as a parameter if you're ever going to write a working unit test against that funct…

Agreed that the code is not very good, the idea of the animated resume was great, despite some technical issues it may have.

My first impression would be that he is a very creative professional with good designing skills. Regarding the programming I totally agree that it have a lot of flaws and lack of basic testing capabilities.

Re: Interactive Résumé of Robby Leonardi

#17

Post by rimantas »

at-fates-hands wrote: post #11
onion2k wrote: post #3

It's a fun and cute resume, but delving in to the code you see things like; function blinkRobbyEyes() { if (layersMovement != "not moving 2") { $(robbyEyesCloseDiv).fadeTo(0,1); $(robbyEyesCloseDiv).stop().delay(300).animate({opacity: 0}, 0, function() {}); } } "if (layersMovement != "not moving 2")" is a boolean that should be passed as a parameter if you're ever going to write a working unit test against that funct…

I've always found this interesting. If you're a developer, it should be your number one priority to make sure the code you write for your own site be top notch. God knows how much we all complain about other people's code, you should know however cool it is, developers are going to crack the hood and see how good the code is. Great idea and quite original. I still give him rube cred for having such a nice concept des…

You know, you just gave me an idea: not to have top quality code on personal web site, but have a comment instead "I can explain what's wrong with this code or any other you will show me and how to fix it". I think that gives opportunity for much more interesting discussion :)
Post reply on HN