Jump to content

оцените верстку


rediskavet
 Share

Recommended Posts

Влидатор просит сделать:

<html xmlns="http://www.w3.org/1999/xhtml">

а в остальном всё замечательно, код чёткий чистый читабельный

P.s. CSS не смотрел

В шестом Эксплоере лучше это не смотреть :) ну я думаю и не было цели.

в Седьмом эксплоере немного кое-где подсъехало не критично но вид портит, так что ковыряйте

В восьмом дома посмотрю :)

Ваш враг IE - боритесь с ним

Link to comment
Share on other sites

не под шестой не просили подгонять. А в седьмом где съехало боди и подвал да? Я просто в иеТестере посмотрел, тоже съехало, перезагрузил страницу все на место встало, думал что тестер глючит.

Link to comment
Share on other sites

Да, подвал и шапка, когда делаешь окно не фулскрин, её сносит. + посмотри там где логин и пароль облачко, тоже по-моему что-то куда-то ушло

Вот тут проблемные места видны

Link to comment
Share on other sites

С облаком решил, просто все время забываю, что седьмой ие не правильно понимает padding-и, тогда нужно margin-ы использовать, или наоборот, ну вот уже забыл, хотя только что разбирался :)

А с шапкой и контентом только частично разрешил, сейчас ничего не съезжает, но при сужении окна браузера(только в седьмом експлорере), сайт остается на месте как-бы застыв посередине. Пока не нашел решение.

Link to comment
Share on other sites

Извиняюсь за нубский вопрос. А где вообще вы его смотрите, я немогу понять? Как до него добраться, это же тсс.

Ну для удобства есть куча прибомбасин, но даже за отсутствием чего либа кроме браузера можнож путь к КСС посмотреть в коде и загрузить его

<link rel="stylesheet" type="text/css" href="DD1.css" />

соответственно:

http://red.dax.ru/DD/DD1.css

Link to comment
Share on other sites

Ну для удобства есть куча прибомбасин, но даже за отсутствием чего либа кроме браузера можнож путь к КСС посмотреть в коде и загрузить его

<link rel="stylesheet" type="text/css" href="DD1.css" />

соответственно:

http://red.dax.ru/DD/DD1.css

О, спасибо! Самый оптимальный вариант.

Link to comment
Share on other sites

Буду рад услышать критику по верстке.

- Явных косяков тут нет, но может кто найдет недочеты на которые я не обратил внимания.

http://red.dax.ru/DD/DD1.html

1. это конечно жесть полная

body {

margin:0 auto;

width:980px;

}

2. все блоки зафлочены, интересно что будет в более сложном макете?

Link to comment
Share on other sites

1. это конечно жесть полная

body {

margin:0 auto;

width:980px;

}

2. все блоки зафлочены, интересно что будет в более сложном макете?

1.Не понял в чем тут жесть. Поставил сайт по центру.

2.И к флоатам недоверие мне тоже не понятно.

Может поподробнее опишите в чем я ошибся по вашему.

Link to comment
Share on other sites

rediskavet

По боди и флоату все нормально, считаю. На выбор исполнителя.

А вот эти части:

<p class="Stih"> 
<span>
Как ты красива,<br />
как ты нежна,<br />
как я тебя увидить хочу.<br />
Милая мама, <br />
приди поскорей!<br />
Ты очень нужна<br />
в этой жизни моей!<br />
</span>
</p>

Наверное правильней списком,

И здесь:

<p> 
<span>24.05.2010</span>
<span>Маленькие звёздочки</span>
В Пермском крае прошёл конкурс художественного творчества
воспитанников детских домов.
<a href="#">подробнее</a>
</p>

Даты убрать из <p>, а в заголовках, спан соответственно заменить, на заголовок, какой-нибудь.

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

Ну и вот это как-то хрен его знает...:

<div class="BDRight"> 
<p> </p>
<p> </p>
<p> </p>
</div>

Там обязательно прям так резать было?) В один пустой тег нельзя?)

Edited by Clayton
Link to comment
Share on other sites

1.Не понял в чем тут жесть. Поставил сайт по центру.

2.И к флоатам недоверие мне тоже не понятно.

Может поподробнее опишите в чем я ошибся по вашему.

1. а почему Вы центрируете сайт с помощью тега body?

2.ну это Ваше дело, я просто имею ввиду что можно было обойтися и без них в некоторых случаях..

Link to comment
Share on other sites

body {

margin:0 auto;

width:980px;

}

Сразу же, с самого начала избавляйтесь от привычки центрировать сайт тегом <body>.

Сделайте общий блок (например #wrapper) и его центрируйте.

На боди оставь

body{
margin:0;
font: 14px/16px Verdana,Tahoma,sans-serif;
}

В пикселях или в емах размер шрифта, это на ваш вкус. Сама суть что там нужно обнулить маргин, и задать параметры шрифта.

p,h1 {
margin: 0;
padding: 0;
}

На параграфах и заголовках паддингов и небыло никогда. И нехорошо такое обнуление делать.

a{
outline: none;
}

Жесть... Не делайте так. Это фича браузера. И иногда очень удобно табом перебирать линки, а вы напрочь лишаете юзеров такой возможности.

float: left;
background-image: url(img2/BgHDRTop1.png);
width: 960px;
margin: 0 auto;

И что оно тут отцентрируется? Ведь оно же флотнуто.

Ваш враг - флоат.

.leftForm p input {
background-image:url("img2/FormButton.png");
background-repeat:no-repeat;
border:0 none;
clear:both;
height:22px;
margin-left:50px;
width:63px;
}
.leftForm p input:hover {
background-image:url("img2/FormButtonHover.png");
cursor:pointer;
}

Старайтесь делать спрайтами такие бг. Чтобы при первом ховере небыло мерцания.

Link to comment
Share on other sites

А ты попробуй теперь сюда лайтбокс применить.

Макс, ну я же не заставляю. Я же ток совет даю. Хочешь так центрировать - да пожалуйста. Но потом при косяках начинаються вопросы, а почему у меня не так да не эдак как хотелось бы.

Edited by mishka2
Link to comment
Share on other sites

Clayton-у:

<div class="BDRight">
<p> </p>
<p> </p>
<p> </p>
</div>

Там обязательно прям так резать было?) В один пустой тег нельзя?)

- В один нельзя(т.к. первый всегда вверху, а третий всегда внизу) но вот в два в принципе да можно было засунуть.
<p>
<span>24.05.2010</span>
<span>Маленькие звёздочки</span>
В Пермском крае прошёл конкурс художественного творчества
воспитанников детских домов.
<a href="#">подробнее</a>
</p>

Даты убрать из <p>, а в заголовках, спан соответственно заменить, на заголовок, какой-нибудь.

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

- я в динамике пока не силен но, если даты убрать из <p> тогда нужно будет еще в один див обернуть, чтобы даты подгружать вместе с новостью прально?

Мишке:

И что оно тут отцентрируется? Ведь оно же флотнуто.
- тут просо убрать забыл.
На параграфах и заголовках паддингов и небыло никогда. И нехорошо такое обнуление делать.
- ок. спасибо буду знать, просто обычно через "*" обнуляю.

body

- всегда воспринимал как обычный тег, иногда фон ему ставлю, ну и центрирую через него, чем грозит?

Старайтесь делать спрайтами такие бг. Чтобы при первом ховере небыло мерцания.
- можно пример, стыдно признаться но я не знаю что такое спрайты.

Всем спасибо за критику.

Link to comment
Share on other sites

Насчет спрайтов - погугли, на блогах расписанно достаточно подробно(набери "css спрайты"). Это когда изображения режут не отдельно, а обьединяют.

Насчет body - когданить порежешь страничку, и попытаешся прикрутить лайбокс (это первое что вспомнил, так как тут на форуме уже обращались с подобной проблемой) тогда и увидишь что нехорошая идея центрировать сайт тегом body.

Edited by mishka2
Link to comment
Share on other sites

Насчет спрайтов - погугли, на блогах расписанно достаточно подробно(набери "css спрайты"). Это когда изображения режут не отдельно, а обьединяют.

Насчет body - когданить порежешь страничку, и попытаешся прикрутить лайбокс (это первое что вспомнил, так как тут на форуме уже обращались с подобной проблемой) тогда и увидишь что нехорошая идея центрировать сайт тегом body.

Центрирование боди может идея и не очень хорошая, но вешать на боди бг считаю нормальным.

Link to comment
Share on other sites

Guest
This topic is now closed to further replies.
 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