Дмитрий Кислый
84 повідомлення
#16 років тому
Главная:

Внутренняя:


Большое спасибо.

P.S. валидатор ругаеться на слой с кнопками в форме коментариев, но пока не знаю как правильно сделать. Вот если бы кто подсказал
Максим П.
165 повідомлень
#16 років тому

Мандри - список картинок. Справа пустое место.

Зачем тут список? Можно было просто обойтись "dl"



Нафига такие сложности? ul li {display:inline;} и можно убрать все обертки.

<dl>
<dd class="photo"><a href="#"><img alt="Безлади в Ірані тривають" src="design/video-a.jpg"/> <span/></a></dd>
<dd class="length">2:04</dd>
<dd class="search">Проглянуто:<span>1245</span></dd>
<dd class="rating"><div class="stars"><div style="width: 18px;" class="rating_stars"/></div></dd>
<dd class="date">21.06.08</dd>
</dl>

Да вы батенька маньяк
---
Это профессиональные придирки. С точки зрения заказчика же: "Добро. Верстку принял".
Дмитрий Кислый
84 повідомлення
#16 років тому
2 adrianis
Цитата ("adrianis"):
Да вы батенька маньяк

ага, проще надо быть

спасибо за проф. придирки
Литтманн Саймон
13 повідомлень
#16 років тому
adrianis, подскажите, что за программа в двух последних ссылках?
Идет Р.
11 повідомлень
#16 років тому
Очень похоже на
Максим П.
165 повідомлень
#16 років тому
Firebug
Дмитрий Кислый
84 повідомлення
#16 років тому
И попутный вопрос, помогите с выбором CMS для такого сайта.
Максим П.
165 повідомлень
#16 років тому
Цитата ("ieroglyph"):
И попутный вопрос, помогите с выбором CMS для такого сайта.

Ту которую лучше знаешь.
Евгений Б.
5 повідомлень
#16 років тому
Поля "логин" , "пароль" , "RSS" .
При входе в поля "логин" , "пароль" или "RSS" , остаются надписи внутри.
Дмитрий Кислый
84 повідомлення
#16 років тому
Цитата ("Yugene"):
Поля "логин" , "пароль" , "RSS" .
При входе в поля "логин" , "пароль" или "RSS" , остаются надписи внутри.

это знаю, не дописал js. Все равно спасибо
Александр К.
30 повідомлень
#16 років тому
Очень и очень профессионально, даже более чем!

Что понравилось:
1. Полный image-replacement для всех картинок в хедере (я бы так не парился - вы молодец!)
2. Всегда выровнянное по центру меню независимо от его ширины
3. Все повесил на float, соответственно, верстка чуть сложнее, но универсальнее, да и .clear не нужен
4. Хорошая кроссбраузерность, включая min-width для ИЕ6 (жаль накрылся VirtualBox - не могу посмотреть, как сделал)

Что смутило:
1. bloquote в хедере для IR, имхо, несементично
2. маниакальная мания использования dl, dt, dd и списков для абсолютно всех блоков с текстом и картинкой, имхо, опять же не очень семантично. Можно просто дивом, в который положить заголовок, абзац и картинку, что будет явно попроще, а там уже по надобности отрицательне маргины и т.п.
3. Для логина и пароля надо бы добавить js, чтобы текст убирался при фокусе, и снова возвращался, если поле осталось пустым, а фокус снят
4. Верхнее меню, при наведении не конкретно на ссылку, а на сам блок ссылки реагирует неадекватно, т.е. подменю уже появилось, а ховер пункта меню еще нет или же с него очень быстро спадает фокус (ни желтого бэка сверху, ни подчеркивание не снимается) - нужно подправить
5. Слегка напрягает эта карусель, слишком быстро она "реагирует" на фокус - но это наверное не к вам)

Что можно улучшить:
1. Семантика: кое-где в заголовках возможно стоит ссылки обернуть в <h>, но это уже выборочно.
2. Уменьшить кол-во верстки и вложенность соответственно за счет избавления от dl, dt, dd и т.п. вещей ))
3. Обязательно поправить менюшку
4. Без iepngfix наверное в данном случае можно было бы и обойтись, и просто резать для ИЕ6 гифки с фоном. Но это уже каприз. )

Вердикт: твердая 5! Плюс снят за семантику, большую вложенность тегов и реакцию меню Вы уж извините

Что интересует: сколько времени ушло на верстку и во сколько вы сами оцениваете это по деньгам?
Дмитрий Кислый
84 повідомлення
#16 років тому
Цитата ("Nikolas_Sharp"):
4. Верхнее меню, при наведении не конкретно на ссылку, а на сам блок ссылки реагирует неадекватно, т.е. подменю уже появилось, а ховер пункта меню еще нет или же с него очень быстро спадает фокус (ни желтого бэка сверху, ни подчеркивание не снимается) - нужно подправить

не могли бы скриншот преложить, что-то не понял до конца. и в каком браузере?
Цитата ("Nikolas_Sharp"):
1. bloquote в хедера для IR, имхо, несементично

- блоки с типа с цитатами? что посоветуете?
про логины и паролю помню, пока не прицепил js.
про списки согласен - немного перебор.

времени - 2-3 дня, точно сказать не могу, да и не спешил особо.
по деньгам - за обе станицы 40у.е.

Огромное спасибо за критику.
Александр К.
30 повідомлень
#16 років тому


40 у.е. за 2 страницы???! Да вы меценат!

1. "Інформаційна агенція" определенно в <h>, ибо, имхо, Сео-важный факт. The Ukrainian Point of view либо совсем без blockquotе, либо в дивку просто. Blockquote в идеале использовать в комментах, как здесь же на веблансере сделано.

4. Смотрел в ФФ3. Скриншоты прилагаются. На скриншоте мой курсор находится на пункте "Статті", при этом сам пункт почему-то не в фокусе, как к примеру соседний пункт "Новости", который является на данный момент активным разделом. Если по ссылке "Статті" поводить, или навести конкретно на само слово, то ситуация исправляется, но в любом случае по каким-то причинам, когда курсор над ссылкой, ссылка периодически теряет фокус. МБ js всему виной. В ИЕ тоже такое есть.

Да, точно js! Такое наблюдается только у пунктов, у которых есть подменю!
Дмитрий Кислый
84 повідомлення
#16 років тому
Про меню, действительно есть такое. исправлю.....
Цитата ("Nikolas_Sharp"):
40 у.е. за 2 страницы???! Да вы меценат!

не не меценат, я только учусь, и тем более есть еще ошибки. поэтому пока так. дальше будет видно.
Дмитрий Кислый
84 повідомлення
#16 років тому


Там такое получаеться, потому что подменю (ul) при наведении оказываеться выше основого меню.
исправить пытался, но что-то не очень выходило.
если выравнять, то в FF потом при Ctr++ сильно уходит подменю.
Александр К.
30 повідомлень
#16 років тому
Вы определенно знаете, где и у кого учиться! ) Отличные успехи! Честно скажу, что оценил бы такую верстку в 90 - 120, а то все 150 у.е. за обе страницы, если js тоже ваш. Можете ориентироваться на расценки вот этого вот человека: - у него неплохие оценочные критерии.
Дмитрий Кислый
84 повідомлення
#16 років тому
Цитата ("Nikolas_Sharp"):
Вы определенно знаете, где и у кого учиться! )

тут точно есть у кого учиться, стоит пробежаться по работам ТОП-10 по верстке.
js нагугленный ))
Александр К.
30 повідомлень
#16 років тому
Цитата ("ieroglyph"):
Там такое получаеться, потому что подменю (ul) при наведении оказываеться выше основого меню.
исправить пытался, но что-то не очень выходило.
если выравнять, то в FF потом при Ctr++ сильно уходит подменю.


Да, у вас там кашка со стилями. Вообщем, вкратце нужно поставить высоту и лайн-хейт на ссылку подменю, поубирать все ненужные паддинги со ссылки подменю и с самого подменю, и задать адекватный отступ сверху для подменю. И все это в em естественно. Тогда будет нормально.
Дмитрий Кислый
84 повідомлення
#16 років тому
Понял, спасибо.