Jump to content

Оцените, пожалуйста, первую верстку


Songbird
 Share

Recommended Posts

Самая первая работа, шаблон нашёл на просторах сети.
Представления о HTML и CSS получил из курсов Попова, хотя его здесь, как я понял, не любят. 
Прошу оценить качество работы, очень интересно ваше мнение, жду критики)
Ссылка на сайт  http://firstvirst.e3w.ru/

  • Like 1
Link to comment
Share on other sites

  On 6/3/2014 at 1:10 PM, WEB-best said:

Для первой верстки отлично, но что то очень сильно сомнневаюсь что это первая верстка.

До этого только курс по CSS того же Попова. А что именно вызвало сомнения?

Link to comment
Share on other sites

  On 6/3/2014 at 3:58 PM, CroaToa said:

 

  On 6/3/2014 at 1:18 PM, Songbird said:
А что именно вызвало сомнения?
Хотя бы то, что у Попова везде сплошные id вместо class :) 

 

Да, в практике по СSS у него везде idю Но, Попов дал только представление, ещё немного читал форум и смотрел Sorax пару уроков, в частности-слайдер на CSS, его наработка. 

Link to comment
Share on other sites

  On 6/3/2014 at 6:41 PM, Ironimus said:

И все же, подозрительно хорошо для первой верстки. Кстати, я, походу, один сам изобретал велосипед и слайдер на чистом css(знал я лишь о псевдокласе checked).

на основе велосипедов рождаются новые идеи 

Link to comment
Share on other sites

  On 6/3/2014 at 1:06 PM, Songbird said:

Самая первая работа, шаблон нашёл на просторах сети.

Представления о HTML и CSS получил из курсов Попова, хотя его здесь, как я понял, не любят. 

Прошу оценить качество работы, очень интересно ваше мнение, жду критики)

Ссылка на сайт  http://firstvirst.e3w.ru/

Почему head_line, вместо margin-top: 20px у wrapper'а?

Зачем логотип и иконки в шапке сайта абсолютом выровнены?

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

Link to comment
Share on other sites

При добавлении контента в подписи к картинкам(хотя-бы 2 строчки) Featured Work вся верстка просто улетает. Так как высота для блока featured-works-elements задана фиксировано. Так-же происходит и в других блоках. Фиксированная высота динамических блоков это очень плохо.

 

Про абсолютное позиционирование в шапке сказали, а вот в навигации не сказали. Там вообще что-то намудрено.

Link to comment
Share on other sites

  On 6/4/2014 at 1:28 AM, ALEEX SWN said:

 

  On 6/3/2014 at 1:06 PM, Songbird said:

Самая первая работа, шаблон нашёл на просторах сети.

Представления о HTML и CSS получил из курсов Попова, хотя его здесь, как я понял, не любят. 

Прошу оценить качество работы, очень интересно ваше мнение, жду критики)

Ссылка на сайт  http://firstvirst.e3w.ru/

Почему head_line, вместо margin-top: 20px у wrapper'а?

Зачем логотип и иконки в шапке сайта абсолютом выровнены?

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

 

Спасибо) А чем лучше позиционировать их? Внешними отступами?

  On 6/4/2014 at 8:41 AM, MichaelD said:

При добавлении контента в подписи к картинкам(хотя-бы 2 строчки) Featured Work вся верстка просто улетает. Так как высота для блока featured-works-elements задана фиксировано. Так-же происходит и в других блоках. Фиксированная высота динамических блоков это очень плохо.

 

Про абсолютное позиционирование в шапке сказали, а вот в навигации не сказали. Там вообще что-то намудрено.

Понял, спасибо. А насчёт навигации-в чём конкретно беда?

Edited by Songbird
Link to comment
Share on other sites

  On 6/4/2014 at 10:00 AM, MichaelD said:

В данном случае можно обойтись и без абсолютного позиционирования) логотипу float:left, а контейнеру иконок float:right. Ну и соответствующие отступы. А если иконок будет не 4, а 6?

Хорошо, понял. А что насчёт менюшки?)

  On 6/4/2014 at 10:18 AM, CroaToa said:

читайте http://ru.bem.info/method/ .

Спасибо большое)

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