Jump to content

cris7iano

User
  • Posts

    39
  • Joined

  • Last visited

Everything posted by cris7iano

  1. как по мне резиновая верстка это уже не так актуально, сейчас эра responsive-лично моё мнение. center contener—-весь блок хорошо сделан у новичков не часто такое встретишь(с точки зрения html), только вот еще поработать с названиями классов(прийдет с опытом), contener-не так пишется по поводу заголовка не нужно в h1 вот сегодня писал об этом: А Зачем логотипу h1? а так как логотип скорее всего будет повторяться по всему сайту, то и заголовок будет одинаков везде это не правильно! и после того как этот сайт натянут на cms, то когда юзер будет кликать на новость, его будет беребрасывать на страницу одного поста, где h1 будет как раз заголовок новости, причем тут логотип?? Лучше обернуть для более конструированного html <div class="logo"><img src="......"</div> по поводy header, footer-лучше добавлять классы. а так приличная работа(то что просмотрел) просмотрел только первую стран. но не оч. внимательно))
  2. А Зачем логотипу h1? а так как логотип скорее всего будет повторяться по всему сайту, то и заголовок будет одинаков везде это не правильно! и после того как этот сайт натянут на cms, то когда юзер будет кликать на новость, его будет беребрасывать на страницу одного поста, где h1 будет как раз заголовок новости, причем тут логотип?? Лучше обернуть для более конструированного html <div class="logo"><img src="......"</div>
  3. Почему? Рекомендации к верстке почитай, еще есть хорошие книги по html5 and css 2 and 3 сам читал много чего усвоил "CSS - Каскадные таблицы стилей" - O'Reilly, Эрик А.Мейер ——————————————————————————————————————————————————————————————————-- HTML5. Рецепты программирования Кристофер Шмитт, Кайл Симпсон ————————————————————————————————————————————————————————————————————--CSS3. Руководство разработчикаАвтор: Питер Гастон
  4. можно вместо дисплей тейбл в артикл использывать display: inline-block vertical-align: top
  5. То что айдишник используется для чего то глобального или уникального, а не сунуть его в каждый блок
  6. <section class="block-content"> <article class="content">............<article> <article class="content">............<article> <article class="content">............<article> </section> вот как было б правильно, по поводу классов, что Артиклы без классов, это плохо
  7. http://prntscr.com/2ipaqy ———————————————————————————————————————————————————————————————— Картинку логотипа вставляй в html http://prntscr.com/2ipbkw Навигация тоже относится к хедеру много используешь айдишников, не хорошо, что-то у всех предыдущих кого просматривал, такое же ошибки, вы не родственники случайно???))))
  8. Делается спрайтом и вставляется в css, а не в html. http://prntscr.com/2io89t http://prntscr.com/2io8yu ———————————————————————————————————————————————————————————— на сайте может быть тысячи дивов и больше, лучше классы давать http://prntscr.com/2io9zk ———————————————————————————————————————————————————————————— Много айдишников - это плохо (почитай о них) ———————————————————————————————————————————————————————————— http://prntscr.com/2iobqw ———————————————————————————————————————————————————————————— http://prntscr.com/2ioce2 ———————————————————————————————————————————————————————————— http://prntscr.com/2iodav ———————————————————————————————————————————————————————————— http://prntscr.com/2iodxp еще много доработак, все и не перечислю! ДЕрзай! Удачи))
  9. тут тоже нужно править http://prntscr.com/2i9rec ———————————————————————————————————————————— <p class="sidebar-name">Lorem ipsum sed amet</p>ну название блока никак не должно быть в P.sidebar-name { color: #FFFFFF; font-weight: bold; left: 30px;——удалить position: relative;;——удалить top: 17px;;——удалить padding: 15px 0 0 30px;———————-Добавить}———————————————————————————————————————————————————— как бы я верстал этот блок http://prntscr.com/2i9lvx <article class="post"> <h1><span>Veroeros</span> sed dolor </h1> <figure class="post-img"> <img src="img/pic1.jpg" alt=""> </figure> <div class="entry-text"> <p> Fusce nec eros augue hendrerit erat. Nam dapibus lorem ipsum <a href="#"> dolor et suscipit semper.</a> Lorem amet sed consequat. </p> <p> Sed fermentum dui in bibendum tempus, est orci iaculis dolor vel gravida varius, dui est <a href="#"> tincidunt augue vel</a> facilisis mauris turpis ac mauris. Etiam elementum odio ac erat convallis sit <a href="#"> amet porttitor libero</a> sed vestibulum. </p> </div></article>Еще много работы! Дерзай! удачи! ;) не согласен, если бы вы работали с cms'ками то заметили, что cms'ка добавляет классы или айди тому же футеру или хедеру, к примеру wordpress добавляет айдишники к хедеру и футеру (header id="header" )
  10. в навигации глупо задавать каждой лишке класс, если здесь такого не нужно, привязывайся к классу родителя лишки по поводу ашки тоже самое, иными словами: main-nav__list li {.........................}main-nav__list li a{........................}при ховере должны хотя бы цвет ссылки менятся. article-content__list————не нужно так писатьarticle-content-list—--в рекомендации к cms можно найти по поводу классов и вообще посмотри на сайтах названия классов, так как у тебя не очень название(понимания приходят со временем, особенно если начнешь работать с cms'кой) article-header———--> block-titleтут тоже нужно править http://prntscr.com/2i9rec как бы я верстал этот блок http://prntscr.com/2i9lvx <article class="post"> <h2><span>Veroeros</span> sed dolor </h2> <figure class="post-img"> <img src="img/fig1.jpg" alt="сюда пишут что-то внятное, чтоб если человек зашел с откл.картинками ему было понятно что здесь есть,а не твоё (fig1-что это????)"> </figure> <div class="entry-text"> <p> Fusce nec eros augue hendrerit erat. Nam dapibus lorem ipsum <a href="#"> dolor et suscipit semper.</a> Lorem amet sed consequat. </p> <p> Sed fermentum dui in bibendum tempus, est orci iaculis dolor vel gravida varius, dui est <a href="#"> tincidunt augue vel</a> facilisis mauris turpis ac mauris. Etiam elementum odio ac erat convallis sit <a href="#"> amet porttitor libero</a> sed vestibulum. </p> </div> </article>Еще много работы! Дерзай! удачи! ;)
  11. Высота wrapper не нужна, непонятные названия классов пустые блоки это плохо-для cms'ки. картинки в content-bottom clearfix вставляются в html, это не часть дизайна, а контент соц.сети делать нужно спрайтом 100 раз об этом писал. FootText1-FootText2-FootText3-FootText4-почему разные классы???ведь все 4 блоки одинаковы.
  12. Тебе говорил уже, айди это уникальный элемент, а он у тебя на уровне класса
  13. бредовая тема, напиши в гугле psd шаблоны и будет тебе радость
  14. чем больше ты умеешь и знаешь, тем лучше для тебя и твоего заработка=)
  15. Разве это задание? Найди и сделай копи-паст, делов то ) Это будет полезней, чем просто вырезать и вставить картинку
  16. для второй верстке убери слеши в меню и сделай вот так .main-nav li { float: left; font-size: 0.825em; letter-spacing: 0.4px; line-height: 62px; text-align: center; text-transform: uppercase; } li:after { content: "/"; padding: 0 22px 0 5px;(+ - сам подгонишь) }
  17. http://vk.com/dev/sites вот подсказка!_)
  18. Ограничение ставят в cms, это описание фильма, его не будет столько много что испоганит блок.
  19. Зачем блоку leftBlock и rightBlock установлена фиксир. высота:?а если добавится контент:? лучше убери высоту и чтоб были нижние отступы пропиши нижний паддинг блокам. может тебе уже сказали по поводу этого, не читал все сообщения
  20. http://www.ozon.ru/context/detail/id/3881079/ http://www.ozon.ru/context/detail/id/8465610/ http://www.ozon.ru/context/detail/id/17513568/
  21. Логотип должен быть кликабельным прочитай про id and class, у тебя все построенно на айди,используй классы, id-это уникальный элемент его не везде нужно использывать. Убери outline, и не смотри больше Попова OUR PORTFOLIO зачем фото запихнул в css???а ашке дал id? Вот так лучше <a href="#"><img src="images/image.jpg" alt="image"></a> social картинки делай спрайтом не правильно <p>info@CreativeStudio.com</p> вот как нужно <a href="mailto:info@CreativeStudio.com">info@CreativeStudio.com</a> второй способ читабельней. не создай пустых дивов <div id="topLine"></div> <div id="top-line"></div> а еще лучше <div class="top-line"></div> по поводу полоски чёрной, или создай общий див и задай ему border-top, или к body добавь border-top: 10px solid #000; Еще одно и очень важное, у тебя все блоки с фикс. высотой,это не правильно!!!
  22. Зачем в разметке прописывать стили??? и не прописуйте в названиях лефт,райт.если после вас захотят поменять меню с лево на право??потом будет брррр)
  23. главное чтоб блоки были пиксель в пиксель,а текст может быть с погрешностью. два футера это бред. и две навигации тоже не то.
×
×
  • 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