Код в значительной степени в порядке - несколько вещей, которые вы можете сделать:
1.) Вам не нужно определять свойства, которые установлены по умолчанию в браузере: font-weight: normal;
уже является значением браузера по умолчанию для тела, поэтому вы можете пропустить его, если не меняете его внешний вид.
2.) margin: 0px;
с ним не требуется пиксель - делайте margin: 0;
3.) Идентификаторы имен и классов с именами, относящимися к контенту, но не с макетом: #rightnav
может быть справа от вашего текущего макета CSS, но однажды вы можете передумать и поставить его на левую сторону и идентификатор вроде теряет актуальность. #subnav
может быть лучшим выбором.
4.) Не совсем понимаю, что вы хотели выполнить с этим кусочком кода (поскольку у меня нет времени настроить пример с живым сайтом):
padding-bottom: 1000px;
margin-bottom: -1000px;
но выглядит немного некрасиво, хотя это совершенно правильно и может сделать работу.
5.) <img src="images/background.jpg" alt="background" id="bg" />
- Если изображение является фоновым и не связано с содержимым, используйте свойство css background-image
, чтобы применить его.
Я не буду комментировать метатеги, так как у меня недостаточно знаний об этом.