Ну у вас там нет ничего — несколько десятков строк кода. Не страшного кода. Фтопку препода.
HTML
Нет 
meta с 
charset. Шанс на то, что сервер не отдаст кодировку в 
Content-Type, ничтожно мал, но соломку лучше положить. Да и для локального просмотра по 
file:// пригодится. Прямо перед 
title.
Декларация есть, стили перед скриптом.
Часть элементов без классов, что может превратить дальнейшее усложнение кода в сущий ад.
Блоки с классами 
clear и 
line — избыточный код. Это реализуется с помощью псевдоэлементов или самих элементов.
А вот 
a.submit — чушь. В форме нужна кнопка 
button.submit. У формы должен быть 
action, отправляться она должна по 
https:// и методом 
post.
По стилистике «Ваш» — архаизм. Выкиньте.
CSS
Не привязывайтесь к именам тэгов. Потом сильно пожалеете.
Шрифт задан а 
html — хорошо.
Задавать блочным элементам в нормальном потоке 
width: 100%; — моветон.
Не ставьте 
z-index с шагом больше, чем 1.
Устанавливать 
cursor для 
:hover — странно. Лучше не делать смену этого правила при наведении, ведь курсор имеет смысл только при наведении.
Для 
border-radius не нужно вендорных префиксов.
Размер кегля шрифта лучше привязывать либо к размеру кегля родительского элемента, либо корневого элемента (
rem).
Жонглирование странными значениями размеров говорит о том, что для вас тема построения визуальной модели пока сложна. Обязательно к 
прочтению.
JS
Ну тут вообще ничего пока нет. Но уже если пишете 
$(), то 
jQuery-то нужно подключить.