Николай Галкин
3 сообщения
#17 лет назад


спасибо заранее за критику.
(Верстка в основном таблицами - нужен быстрый и качественный результат )
Игорь Грицишин
519 сообщений
#17 лет назад
Цитата ("nickgal"):
https://www.weblancer.net/users/nickgal/portfolio/

спасибо заранее за критику.
(Верстка в основном таблицами - нужен быстрый и качественный результат )

Начнём с хорошего. Свёрстанно кроссбраузерно.
Однако код твой мягко говоря не самый чистый.
Даже на таблицах можно сверстать лучше.
Вот твоя сама частая помарка:
"<table cellspacing="0" cellpadding="0" border="0" width="100%" bgcolor="#E60000" background="/sites/millana/versions/ru/images/top_menu_right.jpg" style="background-repeat:no-repeat; background-position:right">
<tr>" - всё это надо было нагрузить в CSS.Если такие куски кода переносить в цсс, то:
1. Получается более чистый код.
2. Уеличивается соотношения "полезного" текста к html, и как следствие твой сайт лучше ранжируется посковиком.
3. Сайт грузится быстрее, поскольку файл css кешируется браузером и не подгружаются постоянно куски кода типа
"cellspacing="0" cellpadding="0" border="0""
И вообще, разве не проще вот так написать: "table{border: none;border-collapse: collapse;width: 100%;}"
И тогда нет необходимости постоянно писать " <table cellspacing="0" cellpadding="0" border="0" width="100%" bgcolor="#E60000"
Ну а если у таблицы другие параметры, то можно просто присвоить ей идентификатор.
Ещё бросиллся в глаза тег <script>, ему в html индекса вообще не место.
Разве тяжело его в файд вынести?
Николай Галкин
3 сообщения
#17 лет назад
Спасиба за коммент..

Канеша не сложно ))

Но обломно, времени то мало (пользуюсь HomeSite 5.5 по комбинации Control + J вызываю готовый темплейт и усе дела)
чистый код смотреть здеся (смотрите быстрее - скоро переедет)
патамуша было время

З.Ы. Я исправлюсь ))
Кстати, чота кодеров талантливых расплодилось - пора наверно переучиваться ))
Может в прожект менеджеры податься? Шютка ))
Алексей Попов
1344 сообщения
#17 лет назад
Я непойму Вас

table cellspacing="0" cellpadding="0" это то же самое что border-collapse:collapse;
То есть уже на 10% код облегчаем так как это у Вас в каждой таблице стоит
Николай Галкин
3 сообщения
#17 лет назад
Понял-понял Не бейте..