Оцените, пожалуйста, верстку!

  1. Weblancer
  2. Форум фрилансеров
  3. Критика и оценка работ
Денис Л.
23 сообщения
#12 лет назад


Мне, как начинающему, очень хочется знать: хорошо, качественно ли сверстано, какие есть недочеты, какую бы оценку дали по 10-бальной шкале?
И, это нормально, что код CSS на 754 строки? или я переборщил?

Жду вашей критики! желательно побольше!
Сергей Л.
616 сообщений
#12 лет назад
Цитата ("websterjoy"):
http://websterjoy.tk/
И, это нормально, что код CSS на 754 строки? или я переборщил?

Нормально, бывает много больше.

У вас где то тег div не закрыт и еще много ошибок, валидатором пользоваться умеете?



Все хаки для ie вынесите в отдельный файл css и скармливайте только IE
Докйтайп уже давно пора использовать просто html - <!DOCTYPE html>
Два раза грузится библиотека jquery - зачем?
все css можно объединить в 1

Ну вот вам первое, что бросилось в глаза, размышляйте, исправляйте... Пока на 5 из 10 по моему мнению
Денис Л.
23 сообщения
#12 лет назад
art-apple, спасибо, поисправлял, но CSS из-за вендорных префиксов так и будет невалидным. А так, даже на HTML5 перевел верстку, давно уже хотел его опробовать.
Есть ли еще замечания? покритикуйте еще пожалуйста, как семантика html5 например, правильная ли?
Сергей Л.
616 сообщений
#12 лет назад
Я уверен что, если я копну глубже, то найду много ошибок. Но у меня не так много "бесплатного" времени.
Вот еще немного, прям с главной, есть у вас разметка:


            <section class="buy">

                <div class="counting">

                    <h1 class="cross">&times;</h1>

                    <div class="number">

                        <a href="#" class="triangle"></a>

                        <h1>37</h1>

                        <p class="fs12 gray">упаковок</p>

                        <a href="#" class="triangle2"></a>

                    </div>

                    <h1 class="equally">=</h1>

                    <h1 class="price2">3 885<span> ⃏</span></h1>

                </div>

                <a href="#" class="more">Еще?</a>

                <a href="#" class="cart fs12">Перейти в корзину</a>

            </section>

Это не правильно, сеошники вам руки оторвут за такое! Тег h1 - должен быть один на странице, как правило для самого важного заголовка
Злоупотреблять h2 тоже не рекомендую, но его можно использовать несколько раз

Что еще по семантике, похожие товары у вас каждый товар засунут в тег section - это не правильно, как вариант весь блок похожих товаров нужно ставить в section, а отдельный товар можно в article

ошибки в css вы не все исправили, вендорные префиксы - не являются ошибками, только предупреждениями.

Для начинающего в принципе неплохо в общем.
Олег Н.
297 сообщений
#12 лет назад
Если бы еще была соблюдена модульная сетка. Но то уже совсем другая история
Денис Л.
23 сообщения
#12 лет назад
Спасибо, обновил сайт, убрав эти дурацкие ошибки.
Галина К.
19 сообщений
#12 лет назад
Цитата ("websterjoy"):
http://websterjoy.tk/

Мне, как начинающему, очень хочется знать: хорошо, качественно ли сверстано, какие есть недочеты, какую бы оценку дали по 10-бальной шкале?
И, это нормально, что код CSS на 754 строки? или я переборщил?

Жду вашей критики! желательно побольше!


Посмотрите этот чеклист:
По нему иногда тестирую верстку.
Денис Л.
23 сообщения
#12 лет назад
gl3mk, спасибо, полезная статья!