Разное

Есть сайт. Если не трудно, оцените его, пожалуйста.
Хочу узнать Ваше мнение по дизайну, юзабилити, защите, наличие ошибок. + советы.

Заранее большое спасибо.

Из минусов:
1. Подсказки, постоянно бегающие за курсором, раздражают. Я бы их оставил только у рейтинга. Предназначение элементов и без них сразу понятно )
2. Голосование лучше реализовать без перезагрузки страницы.
3. Я бы организовал просмотр оригинала с помощью одного из плагинов фотогалерей, например вот такого, вместо открытия новой страницы.
4. На странице загрузки фотографии целых 5 полей для выбора файла и его описания - зачем так много? Достаточно одного.

Из плюсов:
1. Приятные цвета.

где там "цвета"? я только один увидела - цвет застиранных треников. ничего приятного. и вообще это не сайт.

не хватает флеша??? 8-)

>где там "цвета"? я только один увидела - цвет застиранных треников. ничего приятного. и вообще это не сайт.
Согл. Уж пусть автор темы не обижается... Скажем так, если чуть пополиткорректнее - ещё не сайт. И тестировать его ещё очень рано. Там ещё пахать и пахать.

Можно побольше текста? :)
Я так понял пахать не в плане наполнения контентом (графикой)? А в техническом плане + дизайн?

Действительно, подсказки стоит убрать. А все остальное - нормально

Три десятка ошибок в коде.

И в HTML, и в CSS.

примерно сутки сервер был недоступен.
А когда, наконец, отдался, я зашел и побродил по нему.
И довольно быстро заметил повторяющийся оборот в текстте
глагол_в_повелительном_наклонении прямо сейчас

Восстановите их.

Зарегистрируйтесь прямо сейчас

Войдите прямо сейчас

И прямо тут же нажал на крестик справа вверху.

Что я сделал не так?

Большое спасибо за критику, есть над чем работать.

По-поводу текста:
Ирбис
1. Что касается "много подсказок", согласен, но для ПС ведь важен не только ALT, но и TITLE изображения. Или я не прав?
2. По-поводу отдельной галереи категорически не согласен. Считаю удобней смотреть в новом окне полноэкранную картинку.
3. На странице загрузки фотографии целых 5 полей для выбора файла и его описания - зачем так много? Достаточно одного.
это не 5 независимых поля, это возможность добавить сразу 5 фотографий. Или я вас не понял?

elenaki
4. Почему это "не сайт"? Это сайт! И сейчас вы находитесь на сайте.

Владимир55
5. Например?

Trianon
Спасибо.

Commander
Спасибо :)

Еще раз всем спасибо за советы.

5. =====>>>>

и это только на главной

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

Да, согласен, код не далеко не идеален. Но это не ошибки, а предупреждения.
Фото = Photo = Foto

Photo != foto.
Foto - это таки транслит слова "фото".
И, кстати, почему "you"? "Ты фото"? Может быть, your?

А почему YouTube? - Ты телек?

Этот вопрос стоит задавать авторам.
Вы то тут при чем?

Такой же вопрос вы задаете мне, как автору.
Поэтому я сделал ссылку на аналогичный пример, который, кстати, не страдает из-за "не понятно, что означает".

То есть вы решили собрать все говно ошибки и непонятки в одном месте?

Вы вообще о чем?
Какие ошибки, какие непонятки?? (Вы считаете название ЮТуб - ошибкой?) В коком месте?

5. Например?

35 .tooltip Свойство border-radius не существует в CSS2, но существует в : 0.3em 0.3em
36 .tooltip Свойство -moz-border-radius не существует : 0.3em 0.3em
52 #top Свойство -webkit-border-bottom-right-radius не существует : 20px 20px
53 #top Свойство -webkit-border-bottom-left-radius не существует : 20px 20px
54 #top Свойство -moz-border-radius-bottomright не существует : 20px 20px
55 #top Свойство -moz-border-radius-bottomleft не существует : 20px 20px
56 #top Свойство border-bottom-right-radius не существует в CSS2, но существует в : 20px 20px
57 #top Свойство border-bottom-left-radius не существует в CSS2, но существует в : 20px 20px
130 #center td Свойство -webkit-border-top-left-radius не существует : 20px 20px
131 #center td Свойство -webkit-border-bottom-right-radius не существует : 20px 20px
132 #center td Свойство -moz-border-radius-topleft не существует : 20px 20px
133 #center td Свойство -moz-border-radius-bottomright не существует : 20px 20px
134 #center td Свойство border-top-left-radius не существует в CSS2, но существует в : 20px 20px
135 #center td Свойство border-bottom-right-radius не существует в CSS2, но существует в : 20px 20px
145 #center td a Ошибка значения : display inline-block не является значением display : inline-block inline-block
154 #center h1 Ошибка значения : display inline-block не является значением display : inline-block inline-block
173 #center a h2 Ошибка значения : display inline-block не является значением display : inline-block inline-block
185 #footer Свойство -webkit-border-top-left-radius не существует : 20px 20px
186 #footer Свойство -webkit-border-top-right-radius не существует : 20px 20px
187 #footer Свойство -moz-border-radius-topleft не существует : 20px 20px
188 #footer Свойство -moz-border-radius-topright не существует : 20px 20px
189 #footer Свойство border-top-left-radius не существует в CSS2, но существует в : 20px 20px
190 #footer Свойство border-top-right-radius не существует в CSS2, но существует в : 20px 20px

А как вы это получили?

Много букв :)
Просто ссылка на плагин к лисе http://users.skynet.be/mgueury/mozilla/

Такие вещи ,для нормальной регистрации в поиске делаются через ЧПУ
но
/page/1/cat/7/subcat/70/foto/22/ ---Финики : "баклажаны" порадовало:)))
/download/22/ ---Скачать!

а надо :
/еда_напитки/фрукты_ягоды/финики.html (конечный документ)
/download/еда_напитки/фрукты_ягоды/финики.jpg ( действие :скачать -с путём до файла(картинки))

лучше разбить еду_напитки на
категория ЕДА
в ней : НАПИТКИ
но глубже 3 уровня вложенности не желательно ( Ведь это не интернет магазин :)))) )

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

Кстати, кто знает, почему, в файле sitemap, пишу:

<?php // header ... echo " <url>   <loc>http://youfoto.org/view.php=foto".$foto['id']."</loc>     <changefreq>daily</changefreq>     <priority>1.00</priority>   <image:image>     <image:loc>".$foto_original."</image:loc>     <image:title>Абстракции</image:title>   </image:image> </url>"; // ..

Работает, а пишу:

<?php // header ... // $foto["title"] [0] = "Абстракции", [1] = "prem" echo " <url>   <loc>http://youfoto.org/view.php=foto".$foto['id']."</loc>     <changefreq>daily</changefreq>     <priority>1.00</priority>   <image:image>     <image:loc>".$foto_original."</image:loc>     <image:title>".$foto["title"]."</image:title>   </image:image> </url>"; // ..

Не работает! Пишет ("This page contains the following errors: error on line 315 at column 22: Encoding error Below is a rendering of the page up to the first error.")

> Полность согласен. но эти задачки из разряда "на потом", т.к. щас резона нет ими заниматься.
Зато резон будет, когда ссылки уже будут в Яндексе и Гугле, разданы друзьям в аську, добавлены в закладку и т.д.
Убить хочется тех, кто берет и меняет адреса страниц.
Делайте нормально сразу. Это нифига не случай преждевременной оптимизации.

С одной стороны (русские УРЛы не рассматриваю) есть смысл делать ссылки под красивый латинский мотив. Но это несет в себе некоторые минусы, которые, лично я думаю, покроет его плюсы. Я думаю понятно какие минусы. Да, избавиться от слов Page, cat, subcat (фото оставить, т.к. потом будет не только фото, а и file...) стоит. Вопрос - как? Вернее как построить читаемый урл.
Либо "youfoto.org/photogalery/flora/cats/mycat-in-home", либо еще каким-то макаром.

Показали ж.
Или
/еда-напитки/фрукты-ягоды/финики.хтмл.

Разве тут только 2 варианта? О_о
И хватит меня говном считать!

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

Обгадить? Ой ли? Мы же не в гостевой пишем.
Человек сам попросил - мы и высказываемся.
Но критику он, очевидно, воспринимать не умеет.
Или вам лично, lightning.say, тоже нужны только восторженные отзыви писающихся кипятком девочек? И вы правда надеетесь услышать такие от веб-разработчиков, которые в любом проекте найдут недочеты или ошибки (да, даже в своем)?

я не против критики, я Вам, neadekvat, о этических аспектах критики говорю в своем предыдущем высказывании... не стоит путать...
->...Мы же не в гостевой пишем

> Но критику он, очевидно, воспринимать не умеет.
Вы высказываете критику не в адрес сайта, а в личный адрес, чего я делать не просил.

За критику по сайту, несомненно, большое спасибо, но не путайте личное и творческое.

Где тут лично про вас? оО

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

UPD: Кстати, мы от темы отошли :).

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

Возможно причины и другие, но факт фактом.
Думаю, на этом можно закончить, т.к. это пустой разговор.

Да ничего сложного, просто грамотно составленная таблица
Пишите правило

RewriteRule ^([a-z]{3,15})/([A-Za-z0-9\-_]{3,20})/([A-Za-z0-9\-_]{2,})(/|\.html)$ index.php?mod=$1&cat=$2&doc=$3 [L]

в параметрах получите:

mod --> food  cat -->drinks doc-->beer

или все запросы гоните на index.php и делайте разбор url

RewriteCond %{REQUEST_FILENAME} !-f RewriteCond %{REQUEST_FILENAME} !-d RewriteCond %{REQUEST_URI} !^/favicon.ico RewriteRule ^(.*)$ index.php?route=$1 [L,QSA]

сама таблица

id      parent_id          mame              linc  1             0            ЕДА               food 2             1            НАПИТКИ           drinks 3             2            ПИВО              beer

Ну и выбирайте из базы :)))

RewriteCond %{REQUEST_URI} !^/favicon.ico
по сути лишнее, так как вы уже написали
RewriteCond %{REQUEST_FILENAME} !-f
Разве что лишний раз не придется проверять, существует ли файл.

А вообще - я подписываюсь под вторым вариантом, т.к. первый превращает htaccess файл в нечто страшное. Да и работать так, прописывая каждое правило - это ж убиться можно.

RewriteCond %{REQUEST_URI} !^/favicon.ico
это для ореры ,в случае отсутствия оного, чтобы не делала 2 запроса на сервер

> это для ореры
о.О

Спасибо. Будем делать.

Стоп! Зачем в таблице parent_id и name? Можно же и без них.
SELECT id FROM table WHERE link = '$url'

угу...а саму ссылку как строить будете?

В смысле ее текст?
Так же как я описал, только на оборот. Т.е. берем название (латинское (либо на ходу переводим)) по ИД фото и вставляем в ссылку.

с выбранной категорией понятно. а откуда будут браться имена родительских категорий? например если выбрана категория "beer", то как вы узнаете что она относится к "drinks" не зная parent_id?

Да, точно, тупонул. Естественно такое есть.

xml очень строгий формат ,любая ошибка в выводе и он падает

<image:title>".$foto["title"]."</image:title>

проблема в

$foto["title"]

или из базы чушь лезет или обработайте htmlspecialchars( )- ом

Я как бы уже понял, что в массиве проблема :). Но не понятно почему. В массиве находится русские текст, который, если не пользоваться массивом, а просто вписать в строку, нормально работает.
К сожалению...

Ошибка, которую вы показали - это ведь браузер выдает, а не скрипт?
Посмотрите исходник по координатам, которые даны в ошибке - и вы увидите, какой символ вызывает фэйл.

Что еще странно - когда пишу $foto["id"] - работает, когда пишу $foto["title"] - не работает.. Дамп массива нечего противоестественного не показал.

Вот, а потом удивляетесь..
Вы вообще прочитали то, что я вам написал? А сделали?

Насколько я понял, вы пишите о ЧПУ в ветке про sitemap (image)? Логика?

Вот-вот. Где логика?
Если я пишу в ветке о карте сайта - то и спрашиваю о ней.
Выше вы привели ошибку. Я спрашиваю - это ошибка браузера, а не скрипта? Если так - то откройте в редакторе свою карту сайта и найдите эту строку, посмотрите, какой там символ не подходит под описание "вполне обычного" (квадратик, значочек и др.)

И я конечно же забыл сделать такую проверку :). Нет, это шутка!
http://clip2net.com/s/JWb9
Кстати, в том файле всего 50 строк.

что это за дерьмо по ссылке лезет?

Ошибка, которая выдается, когда ставлю "<image:title>".$foto['title']."</image:title>" в sitemap.

Вернее, вот ссылка