Jump to content

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


aleksandr-n
 Share

Recommended Posts

Всем большой привет!

Мне очень хотелось бы чтоб провели независимую, здоровую, критику со стороны других грамотных верстальщиков.

Я сам уже тоже не первый год занимаюсь вёрсткой но не когда не выставлял свои работы на критику вот и решил попробовать что скажите ваше мнение будет очень интересным.

И да вот что ещё хотел узнать во сколько вы оцениваете эту работу если бы это делали вы.

Заранее говорю всем большое спасибо за ваше потраченное время и здоровую критику.

Вот ссылки.

http://ph155070.www12.pagehost.ru/ITSF/main.html

http://ph155070.www12.pagehost.ru/ITSF/contacts.html

http://ph155070.www12.pagehost.ru/ITSF/download.html

http://ph155070.www12.pagehost.ru/ITSF/materials.html

http://ph155070.www12.pagehost.ru/ITSF/meropriyatie.html

http://ph155070.www12.pagehost.ru/ITSF/news_all.html

http://ph155070.www12.pagehost.ru/ITSF/news.html

http://ph155070.www12.pagehost.ru/ITSF/raspisanie.html

Link to comment
Share on other sites

1. избавится от <div class="clear"></div> -- научится писать код без него. для начала почитать тут: http://www.xiper.net/collect/html-and-css-tricks/css-tricks/clearfix.html

2. .menu_top -- ну это же список а не просто набор ссылок. и кстати меню едет в Chrome Mac: http://screencast.com/t/dz8TLdfLX

3. .conteyner -- взять за привычку использовать англоязычные термины а не это -- нечитабельное.

4. .con_carusel --наверно имелось ввиду .container-carousel? почему бы так и не написать?

5. <!-- Colonka 3 right --> -- никогда не использовать транслит.

6. .staff .item-staff .staff-info .date -- зачем вешать на <span> display:block; ведь можно использовать <div>

7. .staff .item-staff .staff-info .avatar-info -- а тут вместо <strong> достаточно было бы <b>. И скорее всего это не .avatar-info а .staff-info

Link to comment
Share on other sites

1. избавится от <div class="clear"></div> -- научится писать код без него. для начала почитать тут: http://www.xiper.net...s/clearfix.html

2. .menu_top -- ну это же список а не просто набор ссылок. и кстати меню едет в Chrome Mac: http://screencast.com/t/dz8TLdfLX

3. .conteyner -- взять за привычку использовать англоязычные термины а не это -- нечитабельное.

4. .con_carusel --наверно имелось ввиду .container-carousel? почему бы так и не написать?

5. <!-- Colonka 3 right --> -- никогда не использовать транслит.

6. .staff .item-staff .staff-info .date -- зачем вешать на <span> display:block; ведь можно использовать <div>

7. .staff .item-staff .staff-info .avatar-info -- а тут вместо <strong> достаточно было бы <b>. И скорее всего это не .avatar-info а .staff-info

Большое спасибо за вашу критику обязательно обращу внимание на ваше замечание, а по поводу меню почему сделал просто ссылками есть разумное объяснение тут описано

http://www.xiper.net/learn/tegofenshuj/accessibility/html5-razmetka-menu.html

еще раз спасибо.

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