Альтернативный Форум

apol · Регистрация: **29.12.2006**

Зравствуйте господа форумчане, есть "сырой" сайтик и мне бы хотелось услышать критику по сайту (что не хватает, что не доделанно и просто советы)

Mirage · Регистрация: **31.10.2005**

1. Голосовать скоко раз можно (одного ему мало - предлагает еще)?!
2. Абзацы что все разные (форматирование)?!
3. Заголовок "Все про аквариум" - через чур примитивный, не серьезно (хоть бы флеш анимацию добавил)!

apol · Регистрация: **29.12.2006**

Цитата:

1. Голосовать скоко раз можно (одного ему мало - предлагает еще)?!

Не понял вопроса, что имеешь ввиду?

Цитата:

2. Абзацы что все разные (форматирование)?!

Это поправлю

Цитата:

3. Заголовок "Все про аквариум" - через чур примитивный, не серьезно (хоть бы флеш анимацию добавил)!

Флеш не умею делать, придется начинать изучать Macromedia Flash MX или посоветуйте другую прогу(может у кого есть учебник по Macromedia Flash MX в электронном вариканте для чайника?).

Mirage · Регистрация: **31.10.2005**

Цитата:

Сообщение от apol

Не понял вопроса, что имеешь ввиду?

То что проголосовав, не статистика видна, а очередное предложение проголосовать. Сайт только при нажатии кнопки говорит - типа вы уже, а изначально этого не видать!

Цитата:

Сообщение от apol

Флеш не умею делать, придется начинать изучать Macromedia Flash MX или посоветуйте другую прогу(может у кого есть учебник по Macromedia Flash MX в электронном вариканте для чайника?).

Для такого дела я использую Swish (пока лучшего не попадалось).

Mike · Регистрация: **08.06.2006**

Не сильно полазил по разделам, увидел только что сделан на joomla, шаблон сам рисовал?
Про флэш в заголовке - я бы не советовал - мне флэши в заголовках (а особенно сделанные не профессионально) ну очень не нравиться. Очень редко когда грамотно сделанный флеш порадует.
И может я немного бы поджал ширину сайта или перерисовал бы топ чтобы его можно было масштабировать, т.е. чтобы можно было бы немного поджимать размер страницы и смотреть не развёрнутой на весь экран без линий прокрутки, но всё что я сказал выше - это мои привычки и имхо, т.к. проект частный и насколько я видел не коммерческий - то нормально.

P.Ы. И ещё можно модуль ЧПУ сторонний поставить, чтоб пользователям немного ясней было что там и как, но это тоже на вкус и цвет...

apol · Регистрация: **29.12.2006**

Цитата:

Не сильно полазил по разделам, увидел только что сделан на joomla, шаблон сам рисовал?

Рисовал сам, писал по примерам

metallphilin · Регистрация: **05.10.2005**

ну критика так критика... поехали :baby:

Сайт судя по всему делался в торопях и на коленке - первое что бросилось в глаза - это в коде строки:

PHP код:


			    <META HTTP-EQUIV="Page-Exit" CONTENT="BlendTrans(Duration=0.3)">
    <META HTTP-EQUIV="Page-Enter" CONTENT="BlendTrans(Duration=0.3)">

Это я так понимаю для тех кто один раз не понимает? =) К тому же "ценность" этих строк равна нулю.

Так же в <meta> не прописаны строки для роботов:

PHP код:


			<meta name="robots" content="all">
<meta name="revisit-after" content="1 days">
<meta name="distribution" content="global">

хотя на вкус и цвет...

Далее.

Зачем абсолютные пути? типа:

PHP код:


			<img src="http://www.aqua.izmuroma.ru/images/stories/picmenu.gif" border="0" alt="Новости"/>

что img что a href что css, icon и та аблютный. Кешно это не недочет, но случись перенос на другой домен.. эх весело тебе придеццо. :spiteful: Это больше не рациально, чем ошибочно.

Вот это вообще не понятно:

PHP код:


			<td class="pathway" valign="center" align="left"  bgcolor="#000000" width="75%">

Что разве в css нельзя было в классе "pathway" задать эти парметры. Ибо случись менять стиль сайта - тоже весело будет =) ТОже не ошибка, но в целом см. выше.

Вот это зачем:

PHP код:


			                  <div class="module">
            <div>
                <div>
                    <div>

4 тега <div> я так понимаю для тоже непонятливых.... тоже в топку не нужно.

Вот это тоже на помойку:

PHP код:


			<ul class="mostread"> [что-то там...] </ul>

Ибо из-за этого все сдвинуто по кой то хрен вправо (см. скрин ниже). Если нужно задать таблице отступ то лучше прибегнуть к css, путем использования старых добрых:

PHP код:


			PADDING-RIGHT: 10px; PADDING-LEFT: 10px; PADDING-BOTTOM: 4px; PADDING-TOP: 4px;

Дальше....

На фига картинка:

PHP код:


			<img border="0" src="http://www.aqua.izmuroma.ru/templates/aqua/images/bar_down.gif">

Из-за нее глюк в опере и лисе. Кстати ссылки лучше ставить с корня:

PHP код:


			src="[B]/[/B]images/M_images/arrow.png"

Так как если на сайте будет какая нить папка - отутда картинку не выведешь таким путем. Лучше так же не использовать png, ибо не каждый браузер коректно с ним работает и странный у картинки alt я должен отметить

В class="map" CSS стиля строго поправить строку с цветом ссылки, ибо синий ну тут вообще никак не прет :ph34r:

В менюшку "Кто на сайте" подгрузочный код воткни - ибо пустота

PHP код:


			<td valign="top" width="180" height="370">

убрать нифиг высоту столбца - у тебя динамическая страница, а не статика. нафига высота? страница будет тянуться несмотря на этот параметр, ибо растягиваеццо она вызиваем текством движка.

тут я тоже чего то не понял:

PHP код:


			</em><br /></h6></h4>

Почему тег 4-го заголовка надписи зеленым закрывается после 6-го? в заголовке, задаешь еще один заголовок? а разве так можно :huh:

PHP код:


			<a name="top"></a>

безусловно нужная ссылка что бы выделить ее и нажать delete

Это были небольшие замечания по коду....смотрел одним глазом и поверхностно - многое вероятно прописутил.

Едем по дизайну:

Во первых что просаеццо в глаза эта черная полоса. Зачем она? Жутко привлекает внимание.. взгляд медленно ползает где то рядом, но не найдя ничего интересного.....

Далее как уже сказали лого сделан совково. Тени текста вообще не втыкают в общий градиент фона. Флеш бы пизать не советовал, ибо см. выше там Майк помоему писал...

Ну а дальше просто смотри вложения (так же винды некоторые баги, а которых писал выше):

про первый скрин - подумай ка нужна черная полоса эта или нет?

Так же не понятно почему поиск на сайте свдинут влево и очень мало места сверху (скрин уж делать не буду).

Ну до кучи, что бы уж совсем закритиковать - могу сказать что жумла далеко не лучшее решение в сайтостроении, как и мамба, лимба, джимбо, phpx, phpnuke, e107 и все им подобные. Ангину попробуй заюзать - путевый движек. я его тут на расчленение взял, версия сырая - мрак, но на overdoze.ru ее доводят до ума. Движка хоть и фриверная, но она стоит того.

Если будут вопросы - постараюсь помочь, если время будет. (только ася - в жабу по тех. вопросам не отвечаю)

Всем спасибо за внимание. :alcoholic

apol · Регистрация: **29.12.2006**

Спасибо за критику очень полезна!!!
Давайте разберемся по порядку:
1.

Цитата:

Сайт судя по всему делался в торопях и на коленке - первое что бросилось в глаза - это в коде строки:

Код:
<META HTTP-EQUIV="Page-Exit" CONTENT="BlendTrans(Duration=0.3)">
<META HTTP-EQUIV="Page-Enter" CONTENT="BlendTrans(Duration=0.3)">
Это я так понимаю для тех кто один раз не понимает? =) К тому же "ценность" этих строк равна нулю.

Это строчки для плавного перехода страниц для ослика и только
2.

Цитата:

Так же в <meta> не прописаны строки для роботов:
Код:
<meta name="robots" content="all">
<meta name="revisit-after" content="1 days">
<meta name="distribution" content="global">
хотя на вкус и цвет...

В админке есть управление роботами для каждого контента

3.

Цитата:

Зачем абсолютные пути? типа:

Код:
<img src="http://www.aqua.izmuroma.ru/images/stories/picmenu.gif" border="0" alt="Новости"/>
что img что a href что css, icon и та аблютный. Кешно это не недочет, но случись перенос на другой домен.. эх весело тебе придеццо. Это больше не рациально, чем ошибочно.

metallphilin откуда такой путь выдрал? По крайней мере я его так нигде не прописывал и должен идти относительно корня сайта.

4.

Цитата:

Вот это зачем:

Код:
<div class="module">
<div>
<div>
<div>
4 тега <div> я так понимаю для тоже непонятливых.... тоже в топку не нужно.

Объсни подробнее, в чем беда не пойму

Дальше пока разбераюсь

metallphilin · Регистрация: **05.10.2005**

Цитата:

Сообщение от apol

metallphilin откуда такой путь выдрал? По крайней мере я его так нигде не прописывал и должен идти относительно корня сайта.

В код страницы глянь - они все там.

Цитата:

Сообщение от apol

Объсни подробнее, в чем беда не пойму

Беды то нет, но зачем лишние теги в коде - они все равно бесполезны.

Добавлено через 10 минут
Пы.Сы.:

Цитата:

Сообщение от apol

Спасибо за критику очень полезна!!!

нажал бы просто "Спасибо" и тебе не сложно и мне приятно =)

apol · Регистрация: **29.12.2006**

Код страницы:

Цитата:

<?php
defined( '_VALID_MOS' ) or die( 'Direct Access to this location is not allowed.' );
$iso = split( '=', _ISO );
echo '<?xml version="1.0" encoding="'. $iso[1] .'"?' .'>';
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//">

<head>
<META HTTP-EQUIV="Page-Exit" CONTENT="BlendTrans(Duration=0.3)">
<META HTTP-EQUIV="Page-Enter" CONTENT="BlendTrans(Duration=0.3)">
<?php if ( $my->id ) initEditor(); ?>
<meta http-equiv="Content-Type" content="text/html; <?php echo _ISO; ?>" />
<?php mosShowHead(); ?>
<link rel="shortcut icon" href="<?php echo $mosConfig_live_site;?>/images/favicon.ico" />
<link rel="stylesheet" type="text/css" href="<?php echo $mosConfig_live_site; ?>/templates/aqua/css/template_css.css" />
</head>

<body class="body" leftmargin="0" topmargin="0" marginwidth="0" marginheight="0">
<a name="top"></a>
<table width="100%" border="0" cellpadding="0" cellspacing="0" background="<?php echo $mosConfig_live_site;?>/templates/aqua/images/logo.jpg" >
<tr height="120">
<td align="left" valign="top"><img src="<?php echo $mosConfig_live_site;?>/templates/aqua/images/logo_left.jpg" /></td>
<td align="center" valign="top"><img src="<?php echo $mosConfig_live_site;?>/templates/aqua/images/logo_center.jpg" /></td>
<td align="right" valign="top"><img src="<?php echo $mosConfig_live_site;?>/templates/aqua/images/logo_right.jpg" border="0" align="right" /> </td>
</tr>
</table>
<table border="0" cellpadding="0" cellspacing="0" width="100%">
<tr>
<td class="pathway" valign="center" align="left" bgcolor="#000000" width="75%"><img hspace="5" src="images/M_images/arrow.png" border="0" alt="arrow" /><?php include $GLOBALS['mosConfig_absolute_path'] . '/pathway.php'; ?></td>
<td class="pathway" valign="right" align="right" bgcolor="#000000"><a href="http://www.aqua.izmuroma.ru/component/option,com_rd_sitemap/Itemid,108/" class="map">Карта сайта</a></td>
<td class="pathway" valign="center" align="right" bgcolor="#000000"><?php echo date(_DATE_FORMAT); ?>  </td>
</tr>
</table>
<table border="0" cellpadding="0" cellspacing="0" width="100%">
<tr>
<td background="<?php echo $mosConfig_live_site;?>/templates/aqua/images/bar_down.gif"><img border="0" src="<?php echo $mosConfig_live_site;?>/templates/aqua/images/bar_down.gif"></td>
</tr>
</table>
<table width="100%" border="0" cellspacing="0" cellpadding="0">
<tr>
<td> </td>
</tr>
</table>
<table border="0" cellpadding="5" cellspacing="0" width="100%">
<tr>

<?php
if (mosCountModules('left')>0) {
?>
<td valign="top" width="180" height="370">
<?php mosLoadModules ( "left",-3 ); ?>
</td>
<?php
}
?>
<td valign="top">
<?php if (mosCountModules( "top" )) { ?>

<?php mosLoadModules ( "top" ); ?>

<?php } ?>
<?php mosMainBody(); ?>
</td>

<?php
if (mosCountModules('right')>0) {
?>
<td valign="top" width="180">
<?php mosLoadModules ( "right",-3 ); ?>
</td>
<?php
}
?>
</tr>
</table>

<br />

<table align="center" border="0" cellpadding="3" cellspacing="0" width="100%">
<tr height="1 px">
<td bgcolor="#000000"</td>
</tr></table>
<table width="100%" border="0" align="center" cellpadding="3" cellspacing="0" background="<?php echo $mosConfig_live_site;?>/templates/aqua/images/fon_adres.jpg">
<tr>
<td align="center">
<address> © 2006 Все про аквариум</address>
</tr>
</table>
</body>
</html>

Все ссылки относително корня

Цитата:

div.module {
width: 185px;
background: url(../images/mod_mid.gif) repeat-y;
margin-bottom: 8px;
}

div.module div {
background: url(../images/mod_bot.gif) bottom left no-repeat;
}

div.module div div {
background: url(../images/mod_top.gif) top left no-repeat;
padding: 10px 10px 15px 10px;

}
div.module div div div {
background: none;
padding: 0;
width: auto !important;
width: 100%;
}

div.module div div div h3 {
display: block;
padding : 0px 0px 10px 0px;
margin : 0px;
font-size : 10px;
color : #990000;
text-align : center;
letter-spacing : 1px;

}

Можно поподробнее, чего здесь лишнее? Заранее спасибо

metallphilin · Регистрация: **05.10.2005**

исчо пакритикую, если пазволити =)

Результаты голосования целесобразно сделать в pop-up окне.

http://www.aqua.izmuroma.ru/componen...E0%ED%E8%E8%21

Hапример что у меня выходит (Опера 9,10) см. вложение.

Т.е. много места по бокам - лучше сделать pop-up*ом соб-но заданных размеров. Не будет отвлекать + бага на той же странице строка местонахождения испарилась :mellow:

Поиск на сайте лучше бы сделать по центре и с отспутом пиксела в 4 вниз. как соб-но и другие модули на главной... можно забомбить в css это.

Кстати на днях тут тоже правил сайт - turizm.izmuroma.ru так тоже такая шняга была с отступами. Движек там мамбо...такая же шняга. пришлось резать сами модули на предмет вызова оформления. :baby:

apol · Регистрация: **29.12.2006**

Цитата:

Результаты голосования целесобразно сделать в pop-up окне.

Что это такое и с чем его едят я не знаю -помогите движок joomla

Цитата:

Поиск на сайте лучше бы сделать по центре и с отспутом пиксела в 4 вниз. как соб-но и другие модули на главной... можно забомбить в css это.

пробовал не выходит в ослике норм а в опере не реагирует. (может в сам модуль впихнуть таблицу с отступами?)

metallphilin · Регистрация: **05.10.2005**

Цитата:

Сообщение от apol

Все ссылки относително корня

тут гвоздь в переменной <?php echo $mosConfig_live_site;?> судя по всему движек берет с кинфига адрес сайта и все вбиваеццо абсолютными путями. править или нет - смотри сам. но по сути что еть они что нет - все равно по сути =)

PHP код:


			<table width="100%" border="0" align="center" cellpadding="3" cellspacing="0" background="<?php echo $mosConfig_live_site;?>/templates/aqua/images/fon_adres.jpg">

и

PHP код:


			<table width="100%" border="0" align="center" cellpadding="3" cellspacing="0" background="/templates/aqua/images/fon_adres.jpg">

разницы никакой

Цитата:

Сообщение от apol

Можно поподробнее, чего здесь лишнее? Заранее спасибо

PHP код:


			../images/mod_bot.gif

можно смело менять на

PHP код:


			/images/mod_bot.gif

Дальше....

PHP код:


			div.module div div {
background: url(../images/mod_top.gif) top left no-repeat;
padding: 10px 10px 15px 10px;

}
div.module div div div {
background: none;
padding: 0;
width: auto !important;
width: 100%; 
}

div.module div div div h3 { 
display: block; 
padding : 0px 0px 10px 0px; 
margin : 0px;
font-size : 10px;
color : #990000;
text-align : center;
letter-spacing : 1px;

кажеться я понял зачем в коде по три\четыре одинаковых тега <div> судя по всему каждый из них задаеццо отдельным css стилем. хотя мне, лично, не понятно почему нельзя было сделать div.module, div.module1, div.module2, div.module3 например. Ибо когда ковыраешься в кодах, тем более html там есть такая фигня как открытие и закрытие тегов.... и теги, как оно обычно бывает - не всегда закрыты. Слествие, я лично, стараюсь как можно сильнее урезать код самой страницы.

Например на сайте смальты (http://smalta.net/) я изощрялся даже до такой степени, что задавал в css бекграунд ячейки\таблицы и соб-но задавал стиль этой ячийки.

Бля если попроще, то вместо:

PHP код:


			    <td class="pathway" valign="center" align="left"  bgcolor="#000000" width="75%"><img hspace="5" src="images/M_images/arrow.png" border="0" alt="arrow" /><span class="pathway">Новости </span></td>

у меня выходило что-то типа того:

<td class="m">>Новости</td>

apol · Регистрация: **29.12.2006**

Цитата:

Код:
../images/mod_bot.gif
можно смело менять на

Код:
/images/mod_bot.gif

Так не получится css лежит в отдельной папке по отношению images

metallphilin · Регистрация: **05.10.2005**

Цитата:

Сообщение от apol

Так не получится css лежит в отдельной папке по отношению images

это на столько мелочно что даже как то самому обломно считать это за недочет =)

apol · Регистрация: **29.12.2006**

теперь необходимо время для исправления всех багов, т.к. я разбираюсь во всем этом на уровне "начинающего"

Mirage · Регистрация: **31.10.2005**

Цитата:

Сообщение от apol

Цитата:

<td class="pathway" valign="right" align="right" bgcolor="#000000"><a href="http://www.aqua.izmuroma.ru/component/option,com_rd_sitemap/Itemid,108/" class="map">Карта сайта</a></td>

Все ссылки относително корня

А это как?

apol · Регистрация: **29.12.2006**

Это единственная ссылка, которую я добавлял сам т.к. вновь установленный компонент и сослаться на него по другому я не знаю ( в принципе можно так <?php echo $mosConfig_live_site;?>/component/option,com_rd_sitemap/Itemid,108/)

metallphilin · Регистрация: **05.10.2005**

Цитата:

Сообщение от Mirage

А это как?

Цитата:

Сообщение от apol

Это единственная ссылка, которую я добавлял сам т.к. вновь установленный компонент и сослаться на него по другому я не знаю ( в принципе можно так <?php echo $mosConfig_live_site;?>/component/option,com_rd_sitemap/Itemid,108/)

да он тебе не про это.

Он намекает на то что бы в классе class="pathway"

задать вот это:

valign="right" align="right" bgcolor="#000000"

Скачай учебник по css - полезная вещь то =)