Jump to content

Верстка макета BusinessCo


MadMax
 Share

Recommended Posts

Сверстал макет BusinessCo из этой этой темы.

Верстка проверялась в ie7+ и последних версиях нормальных браузеров. Прошу оценить и покритиковать (пожестче). Заранее благодарю всех откликнувшихся.

Link to comment
Share on other sites

Извините, что не по теме критика: звездное небо сразу в глаза бросается, что дублировано.

Несколько раз пробовал фон перезать - красиво не получается :dash:

Link to comment
Share on other sites

  • 2 weeks later...

1.

<div class="logo"> 
<h1>Business Co - Reach stability with us!</h1>
<a href="#" title="Business Co - Reach stability with us!"><img src="img/logo.png" alt="Business Co - Reach stability with us!"></a>
</div>

ссылку можно было бы и в заголовок положить

2.

<ul class="main_menu">
<li><a href="#"><span>Contacts</span></a></li>
<li><a href="#"><span>About</span></a></li>
<li><a href="#"><span>Products</span></a></li>
<li class="active_link"><a href="#"><span>Solutions</span></a></li>
<li><a href="#"><span>Home</span></a></li>
</ul>

а тут лучше бы использовать фишки CSS3 с поддержкой любой ширины (и что бы в ИЕ 7+ тоже было гуд), вместо того что бы лепить одинаковые стили.

а вот вдруг завтра заказчик решит свой сайт перевести на другой язык и попадетс я длинное слово, или наоборот короткое?

3.

div.conten
- div.sidebar
- div.primary_content

а положить основной контент перед сайдбаром слабо?

[offtop]

Смотрю сегодня уже 3й(или 4й) макет и везде тупо одно и тоже. В начале дополнительная информация а потом уже основная.

[/offtop]

4.

<div class="block_1">

угу, а если проект пишется пол года, то мы увидим нечто типа:


<div class="block_128">
<div class="block_129">
<div class="block_130">

5.

<ul>
<li>
<h4>Phasellus porta.</h4>
<p>Fusce suscipit varius mi. Cum soc natoque penatibus et magnis. Parturient montes, nascetur.</p>
</li>
<li>
<h4>Ridiculus mus nulla dui.</h4>
<p>Fusce feugiat malesuada odio. Morbi nunc odio, gravida at, cursus nec, luctus.</p>
</li>
<li>
<h4>Maecenas tristique orci.</h4>
<p>Duis ultricies pharetra magna. Donec accumsan malesuada orci. Donec sit amet eros. </p>
</li>
</ul>

что-то мне подсказывает что использование Н4 тут нецелесообразно.

6.

<div class="block_4_1">
<h4>Praesent vestibulum molestie lacus.</h4>
<p>Aenean nonummy hendrerit mauris. Phasellus porta. Fusce suscipit varius mi.
Cum sociis natoque penatibus et magnis dis parturient montes, nascetur ridiculus mus.</p>
</div>

картинка должна лежать <img> а не фон. а еще ето список.

7.

<ul class="copyright">

<li>Business Co. © 2008</li>

<li>Privacy Policy</li>

</ul>

© - пишем © в разметке.

8.

<div id="slider">

зачем тут ID?

PS: незачёт.

Link to comment
Share on other sites

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

2. думаю мне на надо объяснять что такое списки )

3. правило хорошего тона.

Link to comment
Share on other sites

1) Резонно, хотя это скорее вопрос к разработчику CMS (в хорошей CMS на вкладке "Add/edit solution" в админке будет поле "Аватарка", которое движок сможет хоть на лету в спрайт вклеить).

2) Лучше объяснить, потому что почему решения обязательно должны представляться списком, а не, например, набором цельных подразделов (article) — как минимум неочевидно. В данном конкретном случае я скорее соглашусь, что здесь список в тему, но вообще такое нуждается в доказательствах.

3) А палитру цветов, случаем, не надо ограничивать "безопасной"? Тоже ведь правило хорошего тона... было когда-то :)

Link to comment
Share on other sites

а тут лучше бы использовать фишки CSS3 с поддержкой любой ширины (и что бы в ИЕ 7+ тоже было гуд), вместо того что бы лепить одинаковые стили.

Не пойму, про что речь. Ткните носом ?

что-то мне подсказывает что использование Н4 тут нецелесообразно.

Почему нет ?

зачем тут ID?

ID - JS

Link to comment
Share on other sites

а тут лучше бы использовать фишки CSS3 с поддержкой любой ширины (и что бы в ИЕ 7+ тоже было гуд), вместо того что бы лепить одинаковые стили.

Не пойму, про что речь. Ткните носом ?

что-то мне подсказывает что использование Н4 тут нецелесообразно.

Почему нет ?

зачем тут ID?

ID - JS

1. про ето: https://skitch.com/klierik/g6khn/business

2. потому что это блок скорее похож на список определений. хотя тут следует всё же консультироватся в СЕОшником )

Link to comment
Share on other sites

похож на список определений

Имхо, очень мало. Где тут отношение "имя — значение"? Просто неупорядоченный список (перечисление) однотипных сущностей с названием и пояснением — куда ни шло. И названия сущностей вполне подходят под определение "заголовки"...

Впрочем, по лорем-ипсуму и вправду трудно понять, что это. Может, в реальном сайте оно и действительно будет похоже на термины с расшифровкой... :)

Link to comment
Share on other sites

оффтоп: klierik, ну ты же образованный, умный парень. Пиши Э вместо Е. Это простое правило и его легко запомнить. Русские итак смеются над нами.

мне, по сути, пофиг "это" или "ето". никому ничего не обязан!

Link to comment
Share on other sites

а тут лучше бы использовать фишки CSS3 с поддержкой любой ширины (и что бы в ИЕ 7+ тоже было гуд), вместо того что бы лепить одинаковые стили.

а вот вдруг завтра заказчик решит свой сайт перевести на другой язык и попадетс я длинное слово, или наоборот короткое?

Долго думал... Разве для того что-бы ширина закладки растягивалась в зависимости от содержания не достаточно поменять width на min-width ?

Link to comment
Share on other sites

сразу в глаза бросилось то, что у блока Recent solutions нет градиента у рамки. у блока Ideal solutions тоже непорядок с рамкой

так же к Learn more прилипают стрелочки. по крайней мере у меня в опере

Edited by cyklop77
Link to comment
Share on other sites

сразу в глаза бросилось то, что у блока Recent solutions нет градиента у рамки. у блока Ideal solutions тоже непорядок с рамкой

есть такое, схалтурил немного

к Learn more прилипают стрелочки. по крайней мере у меня в опере

у меня нормально. опера 11.61

Link to comment
Share on other sites

Долго думал... Разве для того что-бы ширина закладки растягивалась в зависимости от содержания не достаточно поменять width на min-width ?

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

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