Нужна критика по сайту

apol · 07.01.2007

Зравствуйте господа форумчане, есть "сырой" Link hidden, please Sign in or Sing up и мне бы хотелось услышать критику по сайту (что не хватает, что не доделанно и просто советы):help:

Mirage · 07.01.2007

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

apol · 07.01.2007

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

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

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

Это поправлю

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

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

Mirage · 07.01.2007

apol сказал(а):
Не понял вопроса, что имеешь ввиду?

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

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

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

Mike · 07.01.2007

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

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

apol · 08.01.2007

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

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

metallphilin · 08.01.2007

ну критика так критика... поехали :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 стиля строго поправить строку с цветом ссылки, ибо синий ну тут вообще никак не прет

h34r:

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

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 · 08.01.2007

Спасибо за критику очень полезна!!!
Давайте разберемся по порядку:
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 · 08.01.2007

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

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

apol сказал(а):
Объсни подробнее, в чем беда не пойму

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

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

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

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

apol · 08.01.2007

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

<?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 · 08.01.2007

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

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

Link hidden, please Sign in or Sing up

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

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

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

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

apol · 08.01.2007

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

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

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

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

metallphilin · 08.01.2007

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 там есть такая фигня как открытие и закрытие тегов.... и теги, как оно обычно бывает - не всегда закрыты. Слествие, я лично, стараюсь как можно сильнее урезать код самой страницы.

Например на сайте смальты (Link hidden, please Sign in or Sing up) я изощрялся даже до такой степени, что задавал в 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 · 08.01.2007

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

Код:
/images/mod_bot.gif

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

metallphilin · 08.01.2007

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

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

apol · 08.01.2007

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

Mirage · 08.01.2007

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 · 08.01.2007

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

metallphilin · 13.01.2007

Mirage сказал(а):
А это как?

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

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

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

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

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

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

Нужна критика по сайту

apol

Писатель

Mirage

Живу я тут ...

apol

Писатель

Mirage

Живу я тут ...

Mike

Форумчанин

apol

Писатель

metallphilin

apol

Писатель

metallphilin

apol

Писатель

metallphilin

apol

Писатель

metallphilin

apol

Писатель

metallphilin

apol

Писатель

Mirage

Живу я тут ...

apol

Писатель

metallphilin