Оцените вёрстку

[black40]

Оцените, плиз, вёрстку макета. Не дизайн ( шаблон я скачал где то в сети, кажись psd-мания), а именно вёрстку. http://sample.besaba.com/  Спасибо

[Struggle]

Ссылки почему блочные? там должны быть поинтером только сами ссылки, новости текстом? они должны быть ссылками, картинки которые относяться к дизайну принято внедрять через css, а не картинками прямыми, кнопка read more скачет, а именно текст в ней, задай line-height или padding, под подвалом не понятый отступ, плюсом он не тянется вниз, а здесь он типа должен тянуться, юзай шаблонизаторы чтобы понять как он устроен, логотип не кликабельный, поправь — а его делаем картинкой и ссылкой, так принято типа, ну тебе хватит пока

Edited May 18, 2014 by Struggle

[npofopr]

Ссылки почему блочные?

В меню? Не блочные, а табличные, чтобы выравнивание сделать.

 

 

там должны быть поинтером только сами ссылки, новости текстом?

Поддерживаю, у вас кругом не расставлены ссылки, видимо особо не заморачивались, undeline всегда легче поставить)

 

 

картинки которые относяться к дизайну принято внедрять через css

Кто такое принял? И что вы в данном макете считаете дизайном? Росток из земли? А если пользователь захочет вместо этого ростка поставить большое дерево?

 

 

кнопка read more

+ это и не кнопка вовсе.

 

 

а его делаем картинкой и ссылкой, так принято типа

м? сейчас все нормально, только обернуть надо в ссылку.

 

 

плюсом он не тянется вниз, а здесь он типа должен тянуться

кому должен?) да и шаблонизаторы тут не причем. Есть много способов. Например.

 

А так нормально. Не знаю только, правильно вы ли html5 теги наставили. Но наверно да. Вот, недавно статья по этому поводу была.

Edited May 18, 2014 by npofopr

[Struggle]

 

Ссылки почему блочные?

В меню? Не блочные, а табличные, чтобы выравнивание сделать.

 

 

там должны быть поинтером только сами ссылки, новости текстом?

Поддерживаю, у вас кругом не расставлены ссылки, видимо особо не заморачивались, undeline всегда легче поставить)

 

 

картинки которые относяться к дизайну принято внедрять через css

Кто такое принял? И что вы в данном макете считаете дизайном? Росток из земли? А если пользователь захочет вместо этого ростка поставить большое дерево?

 

 

кнопка read more

+ это и не кнопка вовсе.

 

 

а его делаем картинкой и ссылкой, так принято типа

м? сейчас все нормально, только обернуть надо в ссылку.

 

 

плюсом он не тянется вниз, а здесь он типа должен тянуться

кому должен?) да и шаблонизаторы тут не причем. Есть много способов. Например.

 

А так нормально. Не знаю только, правильно вы ли html5 теги наставили. Но наверно да. Вот, недавно статья по этому поводу была.

 

умник много? парень не силён в верстке, пусть смотрит код генератора шаболна, ссылки иммелись введу что выделяются они как блочные, пустое то место не должно кликаться

[npofopr]

парень не силён в верстке

почему вы так считаете? А кто силен?) Он даже вполне адекватные имена классам дает, а не как бывает: class="block1" и т.п.

 

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

Как раз таки в большинстве своем ошибочное суждение. Не надо заставлять пользователя целиться в пиксель пиксель.

 

[Struggle]

 

парень не силён в верстке

почему вы так считаете? А кто силен?) Он даже вполне адекватные имена классам дает, а не как бывает: class="block1" и т.п.

 

 

 

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

Как раз таки в большинстве своем ошибочное суждение. Не надо заставлять пользователя целиться в пиксель пиксель.

 

 

Моё мнение не может быть ошибочным, там пункт никак не обозначен дизайном, это значит там выделяться должна  только сама ссылка,так нарисовал дизайнер потому что, если бы был какой про hover эффект типа изменение цвета li, то надо верстать как щас есть, а так - просто изменение цвета + underline

Слов о  пиксельной верстки вообще не было

 

upd: каждая новость - новая ссылка, а не одной строкой

Edited May 18, 2014 by Struggle

[CroaToa]

это значит там выделяться должна  только сама ссылка,так нарисовал дизайнер потому что

 Поддерживаю, когда то я тоже верстал этот макет, тоже была проблема точно такая же с ссылками блоками. В данном случаи, мне кажется, нужно что бы выделялся сам текст, так как ничто не требует блочных (табличных, не суть) ссылок (в плане дизайна)

Edited May 18, 2014 by CroaToa

[xzarxzes]

Думаю что article вокруг параграфа тут лишний. Для дат и времени есть теги time

Edited May 18, 2014 by xzarxzes

[black40]

Сейчас это дело выглядит вот так : http://sample.besaba.com/

Не всё , конечно, исправил. Ссылки оставил как есть, добавил только background при hover (ТЗ нету к макету, кто ж его знает. что там дизайнер задумал)

article убрал, сразу их проставил для того чтоб выделить, что это отдельная статья(так мне показалось , что правильно будет)

 

 

upd: каждая новость - новая ссылка, а не одной строкой

- так там один текст, как его отдельными ссылками проставлять?

20px ниже footera ставил для того чтоб видеть где макет заканчивается, забыл убрать(это "минус" конечно), исправил.

 

 

Поддерживаю, у вас кругом не расставлены ссылки, видимо особо не заморачивались, undeline всегда легче поставить)

А вот этого я так и не понял, что Вы имели ввиду((

Логотип(рисунок) сделал ссылкой. но текст. мне кажется всё-таки лучше оставить текстом, зачем его картинкой делать? Вдруг поменять придётся...

кажись всё... А, да

 

парень не силён в верстке

ребята, я в вёрстке пока вообще никакой Учусь.

Авось что то да получится... Вы как считаете? 

Edited May 18, 2014 by black40