Jump to content

psywalker

Moderator
  • Posts

    13707
  • Joined

  • Last visited

  • Days Won

    58

Everything posted by psywalker

  1. Это холиварный вопрос. Не стоит его поднимать тут.
  2. Я ещё не придирался ни грамма. Если бы захотел придраться, то боюсь, что человек подумал бы в итоге, что он выбрал неправильную профессию
  3. А, да с этим нет проблем, даже согласен, и, да, так даже будет правильно
  4. Современному миру нужны современные решения и не надо бояться. Волка бояться - в лес не ходить)

  5. Ругали хорошие люди. И если уж делать хлебные крошки, то списком, а эту штуку засовывать в лишки.
  6. Я не навязываю Медведь. Человек просил меня написать своё мнение, я это сделал, а прислушаться к нему или нет, это уже его личное дело. Если уж делать список, то вот эта хрень сильно загрязняет код, в своё время меня за такое ругали. Дружище, ты слышал уже, на сколько баллов из 10-и я оцениваю свою (по моему мнению) самую лучшую вёрстку?
  7. Не за что. Главное, чтобы ты сделал правильные выводы из прочитанного.
  8. Вот такой вот совет от меня лично http://forum.htmlbook.ru/index.php?showtopic=25566&view=findpost&p=191420
  9. Это ничтожно мало для начала. Запомни дружище, я никогда ни к кому не придираюсь, а всегда говорю по факту. И мало того, я ещё не тестил, как следует, а только бегло пробежался по коду.
  10. Просили оценку, ну что ж, получайте. 1. Наверху есть отдельные стили для ИЕ6. Почему нет стилей для других ИЕ, которые ещё к тому же надо и объединить? 2. Логотип обычно - это ссылка, в которой есть ещё и титл. 3. Серый фон у шапки можно было повесить на body смело, а правую картинку как фон на шапку. Лишние необоснованные элементы, это очень хреново. 4. <div class="left"> - что за left, я не знаю, кто это. Классы нужно задавать осмысленные. 5. <ul class="menu"> - то же самое, что за menu, непонятно мне вообще. 6. <h5>31.12.2002</h5> - что за ужас? Зачем тут заголовок. Сделай <span class="date">31.12.2002</span>, и будет намного лучше и понятнее. 7. Сама новость <h5>31.12.2002</h5> <p class="news_bottom_line">To use flash and navi gate, fully customizab i syst add discussing andby topic oro alliscussing andy.</p> Что за бред? Почему каждая новость не в своём блоке? Почему всё замешано в одной кастрюле? Каждая новость - это отдельный самодостаточный раздел. 8. <p class="news_bottom_line"> - вот это очень и очень хреново. В пункте 7 я уже говорил, что для каждой новости нужен свой блок и уже ему ты можешь делать хоть 10 границ. 9. <div class="right"> - не знаю такого. Намного лучше писать <div class="aside2"> 10. <form class="search" action=""> - Почему атрибут action пустой? Ставь хотя бы решётку. 11. <div class="first_caption"> <h1>Welcome to company name</h1> </div> Что за бред вообще? Зачем лишняя обёртка? Там легко можно было обыграть ситуацию по другому, и обойтись всего одним элементом. 12. <h1>Welcome to company name</h1> - какого чёрта на странице 10 заголовков первого уровня? 13. <div class="nav"> <a href="#">Main page</a>/<a href="#">Products</a>/<span>Downloads</span> </div> Это ужасно. Во-первых это навигация (хлебные крошки), а значит нужно делать списком, а во-вторых вот за такое между ссылками / обычно канделябром бьют по голове. 14. <div class="about about1"> - Что за класс about1? Вообще не вижу смысла в нём в этом месте. Кстати, а что он вообще делает) 15. <form class="search" action=""> - Всё же наверное нужно было её прибить абсолютом, а не зафлоачивать. Но это имхо возможно. 16. <div class="bm_bg1"> - вот эта хрень явно лишняя тоже, спокойно можно было без неё обойтись. И лучше бы класс на список повесил. 17. Кстати нижняя относится к футеру, так что тут так же грубая ошибка. 18. - Вот это что за хрень? Какой нафиг баг? Тут виден не баг, а дикая нехватка знаний и от этого безнадёга в выборе вёрстки этого элемента.19. В ИЕ6 вёрстка прыгает иногда, когда резко дёргаешь экран туда-сюда. 20.<div class="article_wrap_top"> - Вот этот блок сделан совершенно неправильно. Картинка <img alt="friends" src="img/article_img_friends.png" class="img_friends">, которая засоряет структуру, должна была быть фоном какого нибудь пустого span-a, а текст внутри блока <div class="article article1"> должен просто обтекать картинку слева, а не иметь левый отступ. Убираем картинку и ппц, текст остаётся на месте. Грубая ошибка. 21. В стилях нет совершенно организации. Очень плохо. 22. Много лишних элементов, заголовков и прочего. Необдуманно свёрстаны элементы, понатыкано ненужных обёрток и т.д. В целом вёрстка по качеству вполне сойдёт для фраланса. Но если есть желание двигаться выше, то придётся выкинуть эту работу в урну, всё перечитать заново (даже больше в разы), учесть свои ошибки и переверстать, только уже обдумывая каждый свой шаг! Оценка по 10-ой шкале: 1 (максимум). В общем Плохо! swetlana В будущем оценивайте пожалуйста работы объективно. Открою вам секрет, кроме самых высоких баллов, которые вы ставите всем подряд, есть ещё и низкие Совершенно не увидел там никакого качества. Там его нет.
  11. rows="20" cols="90" браузеры воспринимают по разному, сделай лучше так: 1. Все размеры задавай через CSS 2. background: inherit; - вместо этого пиши цвет нормальный.
  12. Тогда сделай ресет, вместо * { margin: 0px; padding: 0px; }, где не включай в обнуление таблицы.
  13. 1. Убери старые атрибуты из таблицы. 2. Повесь на таблицу класс и рули стилями для неё через этот класс.
  14. А для чего она?
  15. Да, за сутки можно управиться с каждым макетом, если другими делами НЕ заниматься.
  16. А вот так ещё можно <tr> <td class="add" colspan="2"></td> </tr>
  17. Т.е. если я правильно понял, то: 1. Слайдшоу, это блок с меню слева и с меняющимися картинками справа? 2. Жмёшь на пункт меню слева, а справа просто меняется картинка, так? 3. Если да, то плавно или нет?
  18. Ребят, а я смотрю у вас много общего. Нет желания тряхнуть стариной, устроить конкурс на самую древнюю вёрстку
  19. Зря. Плохой вывод.
  20. Ну добавь ещё else if ( Кстати Ява != ЯваСкрипт
  21. 1. Сделай всё нормально, через список UL-LI 2. Убери старый атрибут align="left" и используй float: left
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy