Javascript-форум (https://javascript.ru/forum/)
-   (X)HTML/CSS (https://javascript.ru/forum/xhtml-html-css/)
-   -   Оценить верстку ! (https://javascript.ru/forum/xhtml-html-css/48203-ocenit-verstku.html)

domius 24.06.2014 16:33

Оценить верстку !
 
Критика приветствуется
http://domius.cc.ua/

BETEPAH 25.06.2014 00:05

- меню рекомендуется вынести в список ul > li
- для картинок с подписями есть figure и figcaption
- для логических блоков есть section
- font-size лучше указывать в относительных единицах (я предпочитаю rem + фаллбэк с пикселями)
- респонсивность так и просится под этот дизайн, а ее нет

depp 26.06.2014 06:18

не понимаю я таких дизайнов. кто-то ввел в моду. но они же бестолковые. артемия лебедева на них не хватает.
верстку не оценивал. но при наличии фразы Responsive Web "респонсивность" все таки должна быть.

MallSerg 28.06.2014 13:25

document.all = HTMLAllCollection[159]
document.styleSheets.length = 1
document.styleSheets[0].rules.length = 84
19.000 ms Loading
7.000 ms Scripting
95.999 ms Rendering
840.822 ms Painting
85.178 ms Other
426.304 ms Idle

Круто че =) долго вылизывалось наверно xD
Понравилось что мало элементов и мало стилей а это сильно влияет на скорость отображения страницы.

Немного непонятно почему ?
margin-bottom: 0px;
margin-left: 45px;
margin-right: 0px;
margin-top: 75px;

имхо margin должны быть симметричными иначе потом понадобятся костыли вроде .service-block>:first-child { margin: 0 0 0 1px!important;} и чем дальше тем больше костылей.

bes 02.07.2014 22:01

Цитата:

Сообщение от domius
http://domius.cc.ua/

Цитата:

Сообщение от BETEPAH
- респонсивность так и просится под этот дизайн, а ее нет

это не стандарт ещё: если нет - в топку?
+ всегда был против горизонтальной полосы прокрутки, хотя иногда она действительно нужна (например, в деревьях, тут всплывающие подсказки в тему)

Цитата:

Сообщение от depp
но при наличии фразы Responsive Web "респонсивность" все таки должна быть.

не обязательно
из этой фразы вообще мало понятно, о чём речь
Цитата:

Proin iaculis purus consequat sem cure digni ssim. Donec porttitora entum suscipit aenean rhoncus posuere odio in tincidunt.
https://translate.google.com/#auto/r...%20ti ncidunt.
Цитата:

Это целевые снадобье рака с большим количеством достойным высокого. До всего сто победы eneus комплекты ненавижу клавиатуры.
:)

domius 03.07.2014 18:50

да я учусь ))) просто скачал psd формат и так сверстал )))

tenshi 04.07.2014 12:29

<h6>Welcome To Our Studio!</h6>
это правда заголовок?

tenshi 04.07.2014 12:37

<div class="services">

  <section class="content-width">
    <h2 class="name-page">SERVICES</h2>
    <p>Proin iaculis purus consequat sem cure.</p>

  </section>
</div>
section и div нужно поменять местами. И раз уж берешься использовать фичи html5 - не плохо было бы почитать о них. https://developer.mozilla.org/en-US/...HTML5_document

tenshi 04.07.2014 12:40

<article>
        <a href="#"><img src="img/basket.png" alt="basket"></a>
        <h4>E-Commerce</h4>
        <p>Proin iaculis purus consequat sem cure 
         digni ssim. Donec porttitora entum suscipit 
		 aenean rhoncus posuere odio in tincidunt.</p>
      </article>


Про figure уже сказали. Ссылка в данном случае должна быть на всю фигуру, а не только на картинку. И раз уж это активный элемент - должна быть визуализация для ховера.

tenshi 04.07.2014 12:45

<a href="#">
  	      <img src="img/item1.jpg" alt="">
  	      <p>Ebony &amp; Ivory</p>
  	      <span>Branding</span>
  	    </a>

Не стоит смешивать блочные и строковые элементы на одном уровне иерархии.
Для картинок, кстати, должен быть задан размер. Лучше если в css.


Часовой пояс GMT +3, время: 04:52.