Jump to content

chekit

Newbie
  • Posts

    5
  • Joined

  • Last visited

About chekit

  • Birthday 04/17/1986

Information

  • Sex
    Мужчина
  • From
    Королёв

chekit's Achievements

Explorer

Explorer (1/14)

0

Reputation

  1. Тогда можно ещё отцентрировать (установив повторение по X на градиенты в кросовках и футере), потому что сейчас получается "дыра" справа. (
  2. Я бы увеличил отступы от краёв в основном блоке, увеличил межстрочный интервал в ненумерованном списке, не оборачивал бы в h2 добрую половину текста (вообще этот момент весьма удивил - у вас в h2 лежат параграфы(!!!)), да и реклама очень мешает... Так же неплохо бы добавить картинки в статьи, можно сократить количество подразделов, "раскрасить" их соответствующими иконками в меню навигации.
  3. Всегда пожалуйста
  4. Мне кажется лучше убрать h1 с картинки в логотипе. Либо же использовать h1, написав название сайта и подменив его картинкой, так как сейчас h1, по сути, не имеет никакого смысла. В основном меню совершенно несемантичные классы "one", "two" и т.д. Не совсем правильно используются атрибуты, например, лучше использовать background-image вместо background, раз уж вы используете из расширенного написания только часть с картинкой "Read more" одеть в p и использовать text-align вместо "сухого" margin-left и т.д. Очень много вещей можно исправить, сделав саму разметку более семантичной, а стили логичными.
  5. Добрый день. 1. В header я бы разнёс логотип и навигацию с помощью float: left, float: right, соответственно и в разметке перенёс logo на первое место 2. Основной заголовок "Lorem ipsum dolor sit amet, consectetur adipisicing elit" я бы обернул в h1 3. Для разделителя под "Lorem ipsum dolor sit amet, consectetur adipisicing elit" использовал бы либо hr, либо псевдо-элемент ::after вместо img 4. Я бы не использовал тэг small, а заголовок статьи и подзаголовок офрмил бы как h2, h3, соответственно 5. img под заголовками в .article положил бы в div (спорный момент ) 6. "Learn More" одел бы в p 7. Так как под колонками с article у вас опять разделитель, то точно бы использовал вместо img hr 8. В блоке с соц. сетями я бы использовал ненумерованный список, в котором бы перечислил, соответственно, эти сети и подставил бы картинки в ссылки подменой через text-indent 9. В блоке News Updates я бы использовал div с классом newsups вместо p, а вот в самом диве бы уже использовал p (опять-таки спорный момент) 10. Кстати, после пункта 8 глянул ваше основное меню (класс menu), там бы я тоже советовал использовать ненумерованный список и, опять-таки, не использовал бы small. Вот, такое моё мнение ))) Прав ли я?! Вообще в остальном всё очень хорошо, как мне кажется
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy