Jump to content

Покритикуйте верстку


JIyHa
 Share

Recommended Posts

1) Вместо


<div class="logo">
<h1>Finance company official site</h1>
<p>company slogan here</p>

можно было использовать hgroup, а меню вынести за пределы этого дива. А иконки вынести в background, зачем они картинками сделаны?

2) Здесь, имхо, подошел бы лучше aside вместо section.

<section class="dop_article">
<article class="article_left">

3) Кружочки в меню лучше бы сделали спрайтом, чтобы не дергались они на ховере.

4) Здесь

<p><img src="img/foto.jpg" height="119" width="286" alt="">
<a href="#" class="comment">Praesent erat elit, tincidunt eget, vehicula id, molestie convallid, massa.
Pellenteque sit amet augue quis massa feugiat</a></p>

не очень понятно - это относится напрямую к изображению или нет. Можно было figure>figcaption сделать.

Стили не смотрел.

  • Like 1
Link to comment
Share on other sites

http://screencast.com/t/A9Ovtr9gAGLv

И еще какие то странное построение сайта у тебя, в class="logo" явно не должны быть кнопки "домой" и тд. И в class="banner" главное меню ни к чему.

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

Кстати вот она

Edited by fragilewelldone
Link to comment
Share on other sites

А какой? Буду очень благодарна, если подскажете. Этот шрифт я действительно не смогла правильно определить, искала и через myfonts тоже. Опыта верстки очень мало, сложно пока, но я научусь.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • 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