Вёрстка. Пожалуйста оцените.

[MrBis]

Всем привет.

 

Оцените, пожалуйста качество вёрстки, укажите на ошибки и недочёты.

 

Небольшой опыт вёрстки есть (делал пару сайтов для себя).

Недочёты, описанные в теме  типичные ошибки постарался учесть.

Хотел, чтобы вёрстка стала моим доходом, но пока не знаю  соответствует ли мой уровень.

 

Вёрстка

Макет

 

P.S.: Благодарю за критику и замечания.

Edited June 23, 2014 by MrBis

[Great Rash]

 

• 
  
  URL заблокирован SpIDer Gate
  
  
  besaba.com
  
  
  
  
  Причина:
  
  Источник распространения вирусов
  
  Дата:
  
  ‎23.‎06.‎2014 ‏‎11:09
  
  
  
  

[MrBis]

 

 

• URL заблокирован SpIDer Gate
  besaba.com
  Причина: Источник распространения вирусов Дата: ‎23.‎06.‎2014 ‏‎11:09

 

 

Хм... странно.

Проверил локально - вирусов нет. проверил он-лайн - вирусов нет.

Возможно у вас сработал на iframe - это я видео с youtube вставлял... сейчас убрал, заменил просто на картинку.

[Great Rash]

Если это бесплатный хостинг, то имеет смысл проверять на вирусы домен, а не конечный хост. Подозреваю. что источником недоверия стал именно besaba.com, хотя может у них на хосте разместили какую-то шнягу, а в черный список попал домен целиком.

 

Вот результат проверки: у трёх ресурсов сайт в черном списке.

[MrBis]

Вот результат проверки: у трёх ресурсов сайт в черном списке.

 

Исправил - перезалил на другой поддомен. Проверил - всё хорошо, никто не ругается.

 

Ссылки в первом посте заменил.

На всякий случай дублирую тут:

 

Вёрстка

Макет

[Ironimus]

1. Используйте семантические блоки.

2. Вместо 

border-bottom: 6px solid transparent;border-left: 6px solid transparent;border-right: 6px solid transparent;border-top: 6px solid #111111;

лучше использовать

border: 6px solid transparent;border-top-color: #111111;

3. Разве в блоке .members не должна открываться картинка при клике на миниатюру?

4. Почему форма с email имеет рамку даже без фокуса?

 

А так, то очень даже хорошо.

[Great Rash]

Во, другое дело. Мне верстка понравилась, явных косяков не заметил, код опрятный, все блоки, на первый взгляд, по делу - ничего лишнего.

В ИЕ7 конечно развалилось, но не знаю была ли у вас цель такой совместимости добиться.

[Bassline]

Хотел, чтобы вёрстка стала моим доходом, но пока не знаю  соответствует ли мой уровень.

 

 

Для начала поищи макеты, состоящие минимум из четырех страничек, и желательно, чтобы различий между их содержимым было как можно больше. 

От верстки одиночных страничек толку не сильно много.

[MrBis]

1. Используйте семантические блоки.

Из HTML5 или что имеется ввиду?

 

 

2. Вместо 

border-bottom: 6px solid transparent;border-left: 6px solid transparent;border-right: 6px solid transparent;border-top: 6px solid #111111;

лучше использовать

border: 6px solid transparent;border-top-color: #111111;

 

запись в 4е строки - это скорее всего в инспекторе так разложилось.

У меня в коде:

border: 6px solid;border-color: transparent transparent #111;

Хотя согласен, ваш пример легче воспринимается.

 

3. Разве в блоке .members не должна открываться картинка при клике на миниатюру?

Должна, но я функционал не прорабатывал. Просто вёрстка.

Это уж просто в блоке .various-articles скрипт для табов очень уж просился.

 

4. Почему форма с email имеет рамку даже без фокуса?

Странно. Проверял во всех браузерах (под Win) - такого не наблюдал.

В каком браузере такой косяк? Что за ОС.

 

А так, то очень даже хорошо.

Спасибо. А если попридираться?

 

 

 

 

Во, другое дело. Мне верстка понравилась, явных косяков не заметил, код опрятный, все блоки, на первый взгляд, по делу - ничего лишнего.

В ИЕ7 конечно развалилось, но не знаю была ли у вас цель такой совместимости добиться.

 Да, IE7 не учитывал. Часто под него требуют совместимость?

 

 

 

 

Для начала поищи макеты, состоящие минимум из четырех страничек, и желательно, чтобы различий между их содержимым было как можно больше. 

От верстки одиночных страничек толку не сильно много.

Спасибо, за совет. Может есть на примете сайты с такими макетами?

Edited June 23, 2014 by MrBis

[Ironimus]

windows, opera12.16. Но это, наверное, мои настройки браузера, так как все другие формы для эмейла имеют такую же рамку. Можно, конечно, добавить border: none !important;, но это, думаю, лишнее.

Edited June 23, 2014 by Ironimus

[Great Rash]

Opera 12 - это старьё, под которое не стоит верстать. Как показывают мои данные (сайт с посещаемостью несколько тысяч уников в день), её доля очень мала - на уровне ИЕ7. Чем быстрее она отомрёт, тем лучше будет для всех. Не могу до сих пор никак понять откуда у неё взялась кучка ярых фанатов.

 

Для ИЕ7 требуют верстать очень редко, но требуют. Например в некоторых моих проектах такое требование есть, но это не показатель конечно.