Jump to content

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

Lisjann

By Lisjann,
in Discussion of works

 Share

Recommended Posts

Lisjann Explorer

Lisjann

Posted
Posted (edited)

не давно закончил сайт для магазина мебели, заказчик попросил сделать версткук резиновой, тк изначально верстал под статику не стал менять css а добавил JS скрипты для растягивания, какие косяки видите? и вообще как сама верстка? Скажу сразу что под Осла не верстал! собственно сама верстка

Edited by Lisjann
Link to comment
Share on other sites
Great Rash Explorer

Great Rash

Posted
Posted

Оно не резиновое.

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">

Почему 2 раза?

  • Like 1
Link to comment
Share on other sites
Helis Explorer

Helis

Posted
Posted

Ни в одном меню пункты не реагируют на наведение.

На разрешении 1024 горизонтальный скролл.

"В

корзине:"

Зачем предлог на отдельной строке?

Нельзя добавить несколько экземпляров товара за раз. Табличка после каждого добавления в корзину — зачем?!

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

Удобство пользования стремится к нулю. Про дизайн в целом молчу тактично.

Логотип пахнет "интелом".

PS Код не смотрел.

Link to comment
Share on other sites
Lisjann Explorer

Lisjann

Posted
  • Author
Posted (edited)

Оно не резиновое.

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">

Почему 2 раза?

Спасибо убрал!!!!

"В

корзине" Спасибо тоже исправил!!!

Пожалуйста оценивайте токо верстку, дизайн и юзабилити делал не я

Edited by Lisjann
Link to comment
Share on other sites
  • 3 weeks later...
amelice Explorer

amelice

Posted
Posted (edited)

Пожалуйста оценивайте токо верстку, дизайн и юзабилити делал не я

1. кодировку сайта указывайте до title.

2. Можно было ромашку в меню сделать бекграундом li.

3. <div class="menu_footer" align="center"> , атрибут align актуальна только элементов таблицы, а для всех остальных блоков свойство -> text-align.

Визуальное представление документа в css.

4. в макете были такие несовпадения?

_http://file.qip.ru/photo/qpeNrPUr/Screen_Shot_2012-09-29_at_1131.html?

Edited by amelice
  • Like 1
Link to comment
Share on other sites
Lisjann Explorer

Lisjann

Posted
  • Author
Posted

Пожалуйста оценивайте токо верстку, дизайн и юзабилити делал не я

1. кодировку сайта указывайте до title.

2. Можно было ромашку в меню сделать бекграундом li.

3. <div class="menu_footer" align="center"> , атрибут align актуальна только элементов таблицы, а для всех остальных блоков свойство -> text-align.

Визуальное представление документа в css.

4. в макете были такие несовпадения?

_http://file.qip.ru/photo/qpeNrPUr/Screen_Shot_2012-09-29_at_1131.html?

Спасибо за коммент!!!!!!!!

1. исправил

2. я может чего то не знаю но вроде бы более одного background-image тег не может иметь а у этого li свойство background-image это зеленый фон кнопки. Или как варант фон можно заменить свойствами градиента, но опять же ромашка слегка выходит за пределы тега li, и тут я тоже не знаю обрежется ли ромашка или нет.

3. у <div class="menu_footer" align="center"> убрал align="center поставил text-align не выровнелось по центру, забил сделал как было, но если скажите по чему так буду признателен

4. исправлю

ЕЩЕ РАЗ СПАСИБО

Link to comment
Share on other sites
cyklop77 Explorer

cyklop77

Posted
Posted (edited)

вёрстка из серии "на отвали"

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

Edited by cyklop77
Link to comment
Share on other sites
Lisjann Explorer

Lisjann

Posted
  • Author
Posted

вёрстка из серии "на отвали"

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

да буду доработывать

Link to comment
Share on other sites
amelice Explorer

amelice

Posted
Posted

Спасибо за коммент!!!!!!!!

1. исправил

2. я может чего то не знаю но вроде бы более одного background-image тег не может иметь а у этого li свойство background-image это зеленый фон кнопки. Или как варант фон можно заменить свойствами градиента, но опять же ромашка слегка выходит за пределы тега li, и тут я тоже не знаю обрежется ли ромашка или нет.

3. у <div class="menu_footer" align="center"> убрал align="center поставил text-align не выровнелось по центру, забил сделал как было, но если скажите по чему так буду признателен

4. исправлю

ЕЩЕ РАЗ СПАСИБО

Пожалуйста. :)

2. Да, я не заметила, что у li есть бекграунд.

Но, даже в этом случае, можно обойтись без дополнительного блока:

  1. можно сделать бекграундом тега а, и правильно организовать стили.
  2. можно использовать псевдоэлемент :before к li или к a.

Почему вы задаете флоат к тегу а в li? Ведь он единственный элемент в теге li, представление элемента меняется - display: block.

3. Так, понятно почему у вас не выровнивается список, потому что ul не текст. Свойство text-align выравнивает текст внутри блока.

В вашем случае, нужно указать margin:auto для списка.

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
Go to topic listing
×
×
  • 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

  I accept