Как объективно оценить сайт?

Question

[alexru3k]

Всем привет! Не так давно начала изучать html css js. Сделала свой первый одностраничник, но сомневаюсь в его так сказать пригодности. Хотелось бы как-то оценить проделанную работу, вроде все работает и выглядит как хотелось, но может с точки зрения специалиста написано все "коряво". В общем я буду очень признательна за критику с указанием промахов и недостатков или достоинств) Сайт снеговички.рф

Answer 1

[Lord_Dantes]

Возможно стоило сделать форму на ajax? Там немного дела, да и лучше будет однозначно!
Ну, а если вам нужно на отдельную страницу, то хоть сделайте "Вернутся на сайт обратно" или подобное))
CSS - лучше подключать в шапку сайта то есть между
А вот скрипты в footer, вообщем наоборот все нужно делать.
Почему? Потому-что сначала лучше загрузить стили, а в конце когда пользователь просматривает сайт, он уже моментально получил js.
Ведь зачем сайту скрипты если там нет стилей?)
+ еще после выхода с мобильной версии сайта, появляется горизонтальный скролл

Также стоит заметить, что отступы лучше делать сверху вниз, а не наоборот, так как будут опускаться все элементы,а так будет только тот которому вы задали этот отступ.
Сайт годный, разве что стоило задать вопросы по дизайну, но это отдельная история.

Comments

[xtress]

alexru3k, Ну и да: под большие мониторы абсолютно не адаптировано. У товарища вообще не видно меню сайта и первого описания. Монитор, судя по скрину, скорее всего 27+ (либо разрешение больше fullHD), ибо у меня на 27 1920х1080 - меню и первое описание просто съехали, но частично видны

[Lord_Dantes]

xtress, нет что вы)) У меня full-hd, я просто зашёл и вышел с адаптива парой кликов, я просто уменьшил экран в процентах.

[alexru3k]

ух.. буду исправлять, спасибо

Answer 2

[азаза]

оформление жуть
модальники вылезают один на другого
сделайте валидацию на заполненность полей

Answer 3

[Юрий Самойлов]

Шрифты не подгружаются. Светлый текст на фоне со светлым рисунком местами смотрится неразборчиво.
Мобильное меню не работает.

Comments

[alexru3k]

спасибо, исправила. почему-то если подключение скриптов переношу из head в body они перестают работать. Пришлось обратно в head разместить

[Lord_Dantes]

alexru3k, скрипты нужно подключать перед закрывающим тегом

[alexru3k]

прям перед закрывающим тегом body прописывала подключение:

</footer>
		</div>
		<script type="text/javascript" src="/main.js"></script>
        <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.7.1/jquery.min.js"  type="text/javascript"></script>
		<script src="https://code.jquery.com/jquery-1.11.0.min.js" type="text/javascript"></script>
	</body>
</html>