Алексей Б.
12 сообщений
#9 лет назад
 Добрый день друзья! Прошу оценить верстку
Алексей Бычков
1979 сообщений
#9 лет назад
Как по мне, если на кнопках / ссылках нет ховеров, то верстка уже уг )
Алексей Б.
12 сообщений
#9 лет назад
littlewd, да, есть такое. На верхней правой кнопке ховер есть, на остальные не делал так как интересует критика самой структуры сайта, блоков, css и т д )
Алексей Бычков
1979 сообщений
#9 лет назад
А, ну тогда спецов по верстке ждите)
Елена Б.
6863 сообщения
#9 лет назад
На верхнем меню он слишком медленный
и слишком мала область наведения
Алексей Б.
12 сообщений
#9 лет назад
floppox,  спасибо!
Андрей Андреев
798 сообщений
#9 лет назад

На долго завис.

bulatovweb, Небольшая стилистическая рекомендация, везде используйте классы, ID нужно только для JS и то не везде, почитайте про методологию БЭМ.
Алексей Б.
12 сообщений
#9 лет назад
b0nn1e,  спасибо!
Владимир Р.
3315 сообщений
#9 лет назад
Картинки без альтов и размеров
form без атрибутов
не вешайте стили на айдишники, id - только для скриптов
неправильное с точки зрения семантики использование тегов html5, в частности article
группируйте элементы в классы и им присваивайте стили, а не цепочке разных айдишников
дальше писать лень))
Алексей Б.
12 сообщений
#9 лет назад
vovka-morkovka,  
все картинки с альтами с заданными для них размерами. 
какой атрибут в form Вы хотели увидеть?
отказ от id только потому что он может использоваться только один раз на странице по сравнению с class. Так что это больше предпочтение верстальщика. 
не соглашусь относительно article. на что ссылаетесь? 
дальше писать лень. 
Владимир Р.
3315 сообщений
#9 лет назад
1. 
2. action, mehod, id
3. это предпочтение неопытного верстальщика
4. w3c
Цитата:
The <article> element specifies independent, self-contained content.
An article should make sense on its own, and it should be possible to read it independently from the rest of the web site.
Examples of where an <article> element can be used:

  • Forum post
  • Blog post
  • Newspaper article
5. инлайновые стили.
6. Сайт должен корректно смотреться при отключенном js.


7. JS-валидация форм
8. Используйте правильные input type (email, tel, url).
9. Подключайте js перед закрывающим </body>.
10. 
Забыл важное: видел h2,h3,h5, но нет h1.

Это то, то видишь сразу. Ньюансы позиционирования и прочее не смотрел - вот уж, извините, действительно лень)
Сергей З.
114 сообщений
#9 лет назад
vovka-morkovka, Уже давно js это часть браузера, нету требования по отключению js.
JS давным давно уже научились грамотно парсить браузеры и все равно где вы скрипты подключаете.
Алексей Б.
12 сообщений
#9 лет назад
vovka-morkovka,
1. Альты прописаны)
2. скрипт для отправки почты не прикручивал, поэтому action и method пропустил, тоже самое и с валидацией форм. При интеграции темы в cms все можно заменить плагином. 
4. <arcticle> - используется для обособленной части содержимого в том числе и  для статьи. Чем Вам левый блок не статья? ))
5. покажите если не сложно. 
6. да, согласен)
7. см. пункт 2. 
8. да. есть оплошность с одним type ) 
9. почему?)))
10. да, нужно было обернуть в див. )
11. нет h1 да)

 
Антон Б.
1132 сообщения
#9 лет назад
Цитата (sergik_zver):
JS давным давно уже научились грамотно парсить браузеры и все равно где вы скрипты подключаете.
браузеры вообще могут многое, в т.ч. и ошибки исправлять, это же не значит, что надо верстать как попало.
Кроме того, скрипты, подключенные внизу не мешают загрузке страницы, повышают оценку в google
Сергей З.
114 сообщений
#9 лет назад
parkgraphic, Еще раз повторяю браузеры сами вниз ваши скрипты вставляют, им вообще на многое пофигу.
Владимир Р.
3315 сообщений
#9 лет назад
Цитата (sergik_zver):
JS давным давно уже научились грамотно парсить браузеры и все равно где вы скрипты подключаете.
Тем не менее в рекомендациях Google на апрель 2014 это правило все еще присутствует.
Цитата (sergik_zver):
Уже давно js это часть браузера
Как этот факт влияет на отключенный (незагруженный) js?

bulatovweb, про инлайновые стили - в спешке не заметил, что их генерит js )
Алексей Б.
12 сообщений
#9 лет назад
vovka-morkovka, спасибо за критику верстки))) в целом что скажите как для начинающего верстальщика?)
Сергей З.
114 сообщений
#9 лет назад
vovka-morkovka, А зачем js отключать ? В вебе полно стереотипов, например ие8-9-10,
пиксел перфект, отключеный джиэс да и еще куча всякой хрени, так не делают современные
приложения, по методу сингл-пэйдж вообще ничего не будет работать без js, так что
привыкать пора, а то до сих пор народ ничего не знает про флетбоксы и эсвиджи.
Алексей Б.
12 сообщений
#9 лет назад
parkgraphic,  Вы предлагаете подключать скрипты внизу страницы?))
Антон Б.
1132 сообщения
#9 лет назад
sergik_zver, да ладно... Пруфлинк?
Тут как бы зависит от того, как решит верстальщик.
Отдать страницу заблокированной до загрузки скриптов, дольше грузит,  но выдает готовую со всей анимацией и поведением.
Либо отдает быстрее, но поведения, анимация добавляются позже.
Я верстаю так, что отключенные скрипты не ломают страницу и поэтому могу позволить себе отдавать страницу ДО загрузки скриптов.
Что считаю более правильным и гугл со мной согласен.
А еще собранные скрипты в одном месте - это порядок и опрятность.
ИМХО.

bulatovweb, по-возможности да.