Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Оценить сайт. (https://javascript.ru/forum/project/27946-ocenit-sajjt.html)

andrey888 30.04.2012 19:27

Оценить сайт.
 
Прошу дать оценки.. Что хорошо, что плохо, что изменить на ваш взгляд.
***Не карить за отсутствие поиска по городам. Сайт еще будет дорабатываться. В чем возможно и помогут ваши взгляды и мнения)

Заранее спасибо.

9xakep 30.04.2012 21:05

andrey888,
отсутствие фона малость напрягает..

andrey888 30.04.2012 22:46

Ок.
Спасибо за ответ.

Gvozd 30.04.2012 23:03

девушка на банере - контуры рванные

Gvozd 30.04.2012 23:04

масштабирование загруженных изображений - лучше обрезать до квадрата, чем искажать пропорции

andrey888 30.04.2012 23:29

Gvozd
Насчет баннера - сорри не понял тебя.. поясни.
Масштабирование изображений - дело в том что квадрат тоже исказит пропорции если фото вытянуто по вертикале или по горизонтале. Или я что то не учел ?

Gvozd 30.04.2012 23:43

Вложений: 1
Вот как у меня выглядит ваш баннер, и например другое изображение девушки с вашего сайта
Вложение 1246
разницу замечаете?
на банере качество картинки весьма ужасное

Цитата:

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

судя по всему вы выбрали третий вариант, и это не самый удачный вариант

andrey888 30.04.2012 23:54

По баннеру - сорри не вижу. При открытии вашей ссылки выдает - "Изображение «http://javascript.ru/forum/attachmen...t-banner-png» не может быть показано, так как содержит ошибки.".
По масштабированию - ясненько, буду думать.

Спасибо за содействие.

andrey888 05.05.2012 00:07

Gvozd можешь еще раз выложить изображение ???
Спасибо.

devote 05.05.2012 00:33

Gvozd,
у форума давно проблемы с прикреплением файлов. портит файлы, одному Богу известно в чем тут трабла ( ну и Илье тоже наверно, вот тока делать с этим ничего не хочет ).

Gvozd 05.05.2012 01:02

Цитата:

Сообщение от andrey888
Gvozd можешь еще раз выложить изображение ???

http://clck.ru/d/TP4o6vX31299G
заодно протестриуем пабличные ссылки от Яндекс-диска

andrey888 05.05.2012 11:50

Просмотрел Флеш баннер в 10 кратном увеличении - все отлично.
Тоже самое с этим же Jpg баннером - увеличил, просмотрел - все ок.
Вопрос - вы какой из баннеров просматривали и через что ?
Спасибо.

Gvozd 05.05.2012 14:34

andrey888,
На моем скриншоте вам нравится как выглядит верхнее изображение?
Если да, то и говорить тут особо не о чем - это мое субъективное мнение, и можете не парится тогда
Если же не нравится, то это был скриншот верхнего банера(который в хедере), как он у меня выглядит

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

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

Tim 05.05.2012 15:54

<span style="color: red; font-size: 14px; padding-left: 10px; cursor: help; " onclick="javascript:document.getElementById('showtel').innerHTML='+79022542092';">Показать тел.</span>

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

andrey888 05.05.2012 20:17

Gvozd Спасибо. Бум наводить красоту.
Tim хорошо, но если честно не очень понятно зачем такое делают на подобных же ресурсах.

Раед 05.05.2012 20:45

Цитата:

Сообщение от Tim
он должен загружаться с сервера при клике

Зачем при клике? Достаточно просто сразу загрузить и скрыть картинку, а при клике сделать её видимой

Tim 05.05.2012 21:16

Цитата:

Сообщение от andrey888 (Сообщение 172872)
если честно не очень понятно зачем такое делают на подобных же ресурсах.

чтобы пользователи не получали SMS c текстом вроде "до розыгрыша Porsche Cayenne осталось 10 дней".

Цитата:

Сообщение от Раед (Сообщение 172875)
Зачем при клике? Достаточно просто сразу загрузить и скрыть картинку, а при клике сделать её видимой

тогда и смысла скрывать её нет. загрузка по требованию будет оптимальнее. далеко не каждый из тех кто смотрит подробное описание объявления захочет узнать номер. генерация картинки 100х15 не очень большая нагрузка, но если так рассуждать то можно и js c css не сжимать и ещё много чего не делать.

Раед 05.05.2012 21:33

Цитата:

Сообщение от Tim
чтобы пользователи не получали SMS c текстом вроде "до розыгрыша Porsche Cayenne осталось 10 дней".

Про Porsche Cayenne, так это по-моему МТС сами сливают номер

Цитата:

Сообщение от Tim
можно и js c css не сжимать

можно

Цитата:

Сообщение от Tim
загрузка по требованию будет оптимальнее. далеко не каждый из тех кто смотрит подробное описание объявления захочет узнать номер

Далеко не каждый, кто захочет узнать номер, захочет ждать, пока он загрузится ))

Gvozd 05.05.2012 22:10

Цитата:

Сообщение от Раед
Далеко не каждый, кто захочет узнать номер, захочет ждать, пока он загрузится ))

Думаешь уйдет менее чем через секунду, если номер не загрузится?

Tim 05.05.2012 22:15

Цитата:

Сообщение от Раед (Сообщение 172883)
Про Porsche Cayenne, так это по-моему МТС сами сливают номер

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

Цитата:

Сообщение от Раед (Сообщение 172883)
Далеко не каждый, кто захочет узнать номер, захочет ждать, пока он загрузится ))

учитывая размер картинки, понять что происходит загрузка и на неё требуется какое то время смогут понять разве что счастливые обладатели модема 56к

andrey888 23.05.2012 13:32

никто не подскажет самый простой способ сделать RSS ленту валидной для всех браузеров .. а то в Хроме например - все просто ужасно..

И еще вопросик по верстке..
Перейдите на любое объявление . Там есть "Смотреть фото полностью" с изображением слева .. Во всех браузерах все ок.. Только в IE этого изображения нет.. не пойму прикола . ЧТо ему не нравится..

Спасибо за советы.

andrey888 24.05.2012 12:39

Никто не делал RSS ??

B~Vladi 24.05.2012 12:51

Цитата:

Сообщение от andrey888
Никто не делал RSS ??

Я делал. Открываешь википедию, учишь структуру xml и делаешь по аналогии. В чем конкретно проблемы?

andrey888 24.05.2012 21:45

Проблема в том что в хроме этот документ отображается никак.. я так понимаю чтоб это пофиксить надо добавить несколько строк специально для хрома в начале документа.. Если у вас был опыт с хромом))) - буду рад если вы его озвучите ..
Другой трабл в следующем .. пытаясь добавить RSS в виджет Яндекса.. он (Яндекс) ругается на неверный формат одного из элементов .. при этом открывая RSS в Мозилле - все ок .. Все так как и ожидалось..

За википедию примного благодарен ))

B~Vladi 24.05.2012 22:27

Ну скинь хотя бы кусок твоего xml, я не телепат.

andrey888 25.05.2012 00:00

Скидываю полностью.
<?php
header("content-type: application/rss+xml");
include 'connect.php';
echo "<?xml version=\"1.0\" encoding=\"UTF-8\"?>";?>
<rss version="2.0">
<channel>
<title>RSS</title>
<link>http:/..</link>
<description>Последние Объявления на ...</description>
<language>en-us</language>
<?php
$sql="SELECT *,unix_timestamp(add) AS `Ad` FROM `list` WHERE `sale_of`='1' AND `on_of`='1' AND `Mn`<30 ORDER BY `rate` DESC, `add` DESC LIMIT 0,30;";
$result=mysql_query($sql,$con) or die ('errors found');
$rows=mysql_num_rows($result);
if($rows > 0){
while($row = mysql_fetch_assoc($result)){
$row["i_path"]!='' ? $ia=explode(',',$row["i_path"]) : $ia[0]='nf.gif';
?>
 <item>
  <title><?php echo $row["s_title"];?></title>
  <link>http:/..</link>
  <description></description>
<?php
$upd = date('d m Y',$row["Add"]);
echo "<pubDate>$upd</pubDate>";
echo '<guid>http:/..</guid>';
?>
 </item>
<?php	
 }
}
?>
</channel>
</rss>

B~Vladi 25.05.2012 00:01

Не надо код, скинь то что получается в итоге.

andrey888 25.05.2012 00:11

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

andrey888 01.06.2012 15:53

Спасибо за ответ))))

B~Vladi 01.06.2012 16:08

Цитата:

Сообщение от andrey888
Всмысле скинуть код на выходе всего этого, конечную разметку ?

Да.

andrey888 01.06.2012 17:41

vot

andrey888 03.06.2012 23:05

Есть догадки ? мнения ?

andrey888 05.06.2012 22:02

...


Часовой пояс GMT +3, время: 02:18.