ashutoshdikshit

feedback.js

Day 1

Form.html

Excellent. Inline CSS and JS is bad practice. Always create files and include them.

Day 2:
Tweets.html

1.
Inline CSS and JS is bad practice. Always create files and include them.

2.
Hooking inline js to html elements is bad practice. Its always good to follow "Unobtrusive JS" approach. It is clear separation of html from javascript code. MVC on the front-end tier.
Read more on this -> http://en.wikipedia.org/wiki/Unobtrusive_JavaScript

3.
$('<li>'+data[i].text+'</li>').appendTo("#list");

# This will add "li" to the DOM inside loop. Not good for performance.
# Its better to build a string inside a loop and add the whole string to the DOM outside of loop.