search

antirez / lamernews

Lamer News -- an HN style social news site written in Ruby/Sinatra/Redis/JQuery
http://lamernews.com
Other
1.35k stars 200 forks source link

Nested form elements? #90

Closed zhemao closed 13 years ago

zhemao commented 13 years ago

Why on earth is there a form element with name="f" inside another form element? This doesn't seem to serve any purpose.

beanz commented 13 years ago

My pull request #81 covers the nested comment issue. I believe your fix is incorrect - for a start you removed the form from the login case which wasn't broken? The correct fix is probably f919e7b528150b7a251922a329be541fb8af2102 since that is consistent with the other div/form and seems more appropriate given the css for these elements.

antirez commented 13 years ago

Thanks, already fixed merging #81