Мой первый сайт, хочу критики

[Dmitrij]

Вот сайт

http://dima.chpmodul.com.ua/

сделан по макету http://absolvo.ru/tmp/uta.png

на вид все вроде правильно, только текст пиксель в пиксель никак не могу подогнать, расстоянием между строк между словами и буквами угадать не получается.

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

Спасибо заранее 

Edited August 4, 2015 by Dmitrij

[afdw]

Кнопка "Искать" сдвинута вверх.

[stan07]

логотип и пункты меню должны быть ссылками 

[Dmitrij]

Кнопка "Искать" сдвинута вверх.

Не заметил сразу, это только на хроме так, уже исправил

логотип и пункты меню должны быть ссылками 

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

[stan07]

если по верстке то :

 

Элемент <main> предназначен для основного содержимого документа. Содержимое должно быть уникальным и не включать типовые блоки вроде шапки сайта, подвала, навигации, боковой панели, формы поиска и т. п.

 

источник:  http://htmlbook.ru/html/main

[antonKar]

логотип ... ссылками

 

Это если на внутряке, если на главной странице, то не надо.

 

1) В поле ввода поиска текст не видно при вводе. Плюс отступов по бокам нет в инпуте.

2) http://li.tl/v/BCH/imgэто ещё что такое?

3) Список ссылок с датами не таблица, а список.

Edited August 4, 2015 by antonKar

[Dmitrij]

если по верстке то :

 

Элемент <main> предназначен для основного содержимого документа. Содержимое должно быть уникальным и не включать типовые блоки вроде шапки сайта, подвала, навигации, боковой панели, формы поиска и т. п.

 

источник:  http://htmlbook.ru/html/main

 

Да все верно, исправил и это... А в целом верстка нормальная?

 

логотип ... ссылками

 

Это если на внутряке, если на главной странице, то не надо.

 

1) В поле ввода поиска текст не видно при вводе. Плюс отступов по бокам нет в инпуте.

2) http://li.tl/v/BCH/imgэто ещё что такое?

3) Список ссылок с датами не таблица, а список.

 

 

я вообще ничего не понял

1) все тут видно, или ты имел ввиду что там лучше белым там написать? тогда ты прав тут я исправил и отступы добавил

2) "http://li.tl/v/BCH/imgэто ещё что такое?" действительно что это такое?) откуда это

3) а почему там список а не таблица?

[antonKar]

все тут видно

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

 

действительно что это такое?) откуда это

А я откуда знаю, вы, видимо, так фон нарезали криво http://dima.chpmodul.com.ua/images/background.jpg

 

а почему там список

Потому что там перечень статей. 

Edited August 5, 2015 by antonKar

[stan07]

 

если по верстке то :

 

Элемент <main> предназначен для основного содержимого документа. Содержимое должно быть уникальным и не включать типовые блоки вроде шапки сайта, подвала, навигации, боковой панели, формы поиска и т. п.

 

источник:  http://htmlbook.ru/html/main

 

Да все верно, исправил и это... А в целом верстка нормальная?

АХ да еще ссылки и картинки имеют обязательный атрибут title

<img src="" alt="" title=""/> <a href="" title=""></a>

Я не сеошник но горят что поисковики придают этому атрибуту большое значение  

[antonKar]

 обязательный атрибут title

 

Обязательный? Серьёзно?

 

 

 

Я не сеошник

СЕО-атрибуты пусть сеошники прописывают, они лучше в этом понимают.

Edited August 5, 2015 by antonKar

[Dmitrij]

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

Понятно, в общем тут я исправил.

так фон нарезали криво

 

Да кстати, я когда резал фон, сомневался что правильно режу, он был слоями, и я просто объединил их, а как правильно сделать?

 

Потому что там перечень статей.

Так записать?

ol{

width:400px;

}

li{

width: 200px;

float: left

}

[antonKar]

ul>li>div+div>a

[Dmitrij]

Понятно, всем спасибо за критику)

[cyklop77]

над ссылками и кнопочками курсор лучше бы делать типа pointer

Edited August 27, 2015 by cyklop77

[angangell]

если по верстке то :

Элемент <main> предназначен для основного содержимого документа. Содержимое должно быть уникальным и не включать типовые блоки вроде шапки сайта, подвала, навигации, боковой панели, формы поиска и т. п.

источник: http://htmlbook.ru/html/main

Да все верно, исправил и это... А в целом верстка нормальная?

АХ да еще ссылки и картинки имеют обязательный атрибут title

<img src="" alt="" title=""/> <a href="" title=""></a>

Я не сеошник но горят что поисковики придают этому атрибуту большое значение

Не title ,а alt

[raptor-dm]

ну лично я когда начал верстать, я не допустил большого количества ошибок что тут есть, не знаю как объяснить, то ли естественно как то понимал это.
ИМХО

Edited August 27, 2015 by raptor-dm

[o5Tolik]

Не плохо для начинающего. Продолжай в том же духе и опыта наберешься очень быстро