Jump to content

Прошу критики первого макета


verbkinm
 Share

Recommended Posts

Прошу критики и побольше, мой первый макет, а это с чего делал! Целый день за ним сидел! Вчера, правда, вообще ток узнал что такое эти макеты :blush: ! Оцените в плане, на сколько отличается от оригинала!

Link to comment
Share on other sites

Что такое target="b" Может быть ты имел ввиду target="_blank"?

Ссылки при наведении прыгают и дергают находящиеся рядом элементы - это не хорошо.

BigRed redText - не стоит использовать верхний регистр в названиях классов лучше big-red или red_text

И что это за классы такие babah1 и babah. Лучше сразу привыкать давать классам осмысленные имена.

color:wheat - не стоит использовать названия цветов, уж лучше #F5DEB3.

Хотя на макете этот текст вообще скрыт под другой картинкой, да и спозиционирован он у тебя не на том месте.

Со шрифтами, кстати тоже явная промашка. Нужно быть внимательнее.

Лупа на макете не прозрачная, а у тебя в ней почему то фон проглядывает.

<img src="images/ElementsOfSite/Raira.png" alt="Raira" width="800" border="1px">

А что это за бордер такой?) Он тут не нужен. А был бы нужен его стоило бы описать в css.

Да и вообще зачем здесь табличная верстка? Привыкай верстать блоками. В данном случае таблицы тут совсем не нужны.

Edited by Duddits
Link to comment
Share on other sites

Очень хорошие и точные замечания, спасибо большое, буду стараться. По поводу шрифтов у меня все хорошо отображает, я просто URW Chancery L использовал, а такой конечно далеко не у всех. target="b" это ж вроде тоже самое что и blank, работает же? С классами ты прав, надо быть серьезней :angry: !

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

Link to comment
Share on other sites

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

на счет таблиц тоже самое. Их используют сейчас только когда есть необходимость разместить именно таблицу. Инлайновые стили тоже выкинуть, только через css.

br'ов тоже по-меньше. Лучше их вообще не использовать.

еще бы не писать русские слова латиницей. Использовать англ.

Для background'a можно все в одну строчку написать. т. е. background: url(../images/Babah.png) no-repeat;

Link to comment
Share on other sites

Каюсь, не заглядывал в css.

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

Для этого нужно использовать @font-face или, например, google fonts если такой шрифт там есть. Ну и в свойстве font-family после шрифта лучше на всякий случай указывать семейство шрифта: sans-serif, serif, fantasy и т.д.

Link to comment
Share on other sites

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

на счет таблиц тоже самое. Их используют сейчас только когда есть необходимость разместить именно таблицу. Инлайновые стили тоже выкинуть, только через css.

br'ов тоже по-меньше. Лучше их вообще не использовать.

еще бы не писать русские слова латиницей. Использовать англ.

Для background'a можно все в одну строчку написать. т. е. background: url(../images/Babah.png) no-repeat;

Спасибо большое за внимание, будем стараться, а вот сокращения в css я пока не знаю :(

Каюсь, не заглядывал в css.

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

Для этого нужно использовать @font-face или, например, google fonts если такой шрифт там есть. Ну и в свойстве font-family после шрифта лучше на всякий случай указывать семейство шрифта: sans-serif, serif, fantasy и т.д.

Приму во внимание, большое спасибо!

Link to comment
Share on other sites

Спасибо большое за внимание, будем стараться, а вот сокращения в css я пока не знаю

а ты лазий по htmlbook в справочнике всех свойств и тегов, и само собой в голове останется

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