www.zeos.com.ua 6 Опубликовано: 9 марта 2009 Рассказать Опубликовано: 9 марта 2009 (изменено) Сайт: www.zeos.in 35. Проведены многочисленные оптимизации HTML кода, используемого в файлах скрипта, для обеспечения валидности XHTML стандартам. Для примера сайт dle-news.ru: http://validator.w3.org/check?uri=http://dle-news.ru Будут ли такие оптимизации для шаблона Default ? У меня установлен шаблон "Default" и если пропустить сайт через: http://validator.w3.org/check?uri=http://zeos.in, то результат такой: Result: 156 Errors, 9 warning(s) Заранее спасибо за ответ! ******************************************************** И еще: Например тут: engine/modules/search.php есть такой код: $searchtable = <<<HTML <form name="fullsearch" id="fullsearch" action="{$config['http_home_url']}index.php?do=search" method="post"> <input type="hidden" name="do" id="do" value="search"> <input type="hidden" name="subaction" id="subaction" value="search"> <input type="hidden" name="search_start" id="search_start" value="$search_start"> <input type="hidden" name="full_search" id="full_search" value="$full_search"> HTML; Вот тут не хватает в <input ... /> (отсутствует пробел и слеш перед закрытием тега) А также тут: $searchtable .= <<<HTML <input type="hidden" name="result_from" id="result_from" value="$result_from"> <input type="hidden" name="result_num" id="result_num" value="$result_num"> Вот тут не хватает в <input ... /> (отсутствует пробел и слеш перед закрытием тега) И тут: <input type="reset" class="bbcodes" style="margin:0px 20px 0 20px;" name="doreset" id="doreset" value="{$lang['s_freset']}"> Вот тут не хватает в <input ... /> (отсутствует пробел и слеш перед закрытием тега) ******************************************************** Также тут: engine/modules/fullsearch.php есть такой код: $searchtable = <<<HTML <form name="fullsearch" id="fullsearch" action="{$config['http_home_url']}index.php?do=search" method="post"> <input type="hidden" name="do" id="do" value="search"> <input type="hidden" name="subaction" id="subaction" value="search"> <input type="hidden" name="search_start" id="search_start" value="$search_start"> <input type="hidden" name="full_search" id="full_search" value="$full_search"> HTML; Вот тут не хватает в <input ... /> (отсутствует пробел и слеш перед закрытием тега) А также тут: $searchtable .= <<<HTML <input type="hidden" name="result_from" id="result_from" value="$result_from"> <input type="hidden" name="result_num" id="result_num" value="$result_num"> Вот тут не хватает в <input ... /> (отсутствует пробел и слеш перед закрытием тега) и тут: <input type="reset" class="bbcodes" style="margin:0px 20px 0 20px;" name="doreset" id="doreset" value="{$lang['s_freset']}"> Вот тут не хватает в <input ... /> (отсутствует пробел и слеш перед закрытием тега) ******************************************************** И еще: $catselall = "selected"; Должно быть: $catselall = "selected=\"selected\""; По стандарту xHTML должно быть имя="значение" И еще: <label for="lbexactname"> Должно быть: <label for="exactname"> ******************************************************** Все onClick заменить на onclick (в соответствии со стандартами xHTML) На http://www.dle-news.ru верно сделано, но вот в дистрибутиве - нет (в обоих файлах) Там их примерно по 14-16 штук. Celsoft, хотелось бы исправить в новой версии http://validator.w3.org/ http://validator.w3.org/check?uri=http%3A%...lidator%2F1.606 http://validator.w3.org/check?uri=http%3A%...lidator%2F1.606 Изменено 9 марта 2009 пользователем www.zeos.com.ua Цитата Ссылка на сообщение Поделиться на других сайтах
www.zeos.com.ua 6 Опубликовано: 9 марта 2009 Рассказать Опубликовано: 9 марта 2009 Автор А также в файле feedback.php $tpl->set( '{code}', "<span id=\"dle-captcha\"><img src=\"" . $path['path'] . "engine/modules/antibot.php\" alt=\"{$lang['sec_image']}\" border=\"0\" /><br /><a onclick=\"reload(); return false;\" href=\"#\">{$lang['reload_code']}</a></span>" ); заменить на: $tpl->set( '{code}', "<span><img id=\"dle-captcha\" src=\"" . $path['path'] . "engine/modules/antibot.php\" alt=\"{$lang['sec_image']}\" border=\"0\" /><br /><a onclick=\"reload(); return false;\" href=\"#\">{$lang['reload_code']}</a></span>" ); document.getElementById('dle-captcha').innerHTML = '<img src="{$path['path']}engine/modules/antibot.php?rndval=' + rndval + '" border="0" width="120" height="50" alt="" /><br /><a onclick="reload(); return false;" href="#">{$lang['reload_code']}</a>'; заменить на: document.getElementById('dle-captcha').src = '{$path['path']}engine/modules/antibot.php?rndval=' + rndval + ''; Тоже самое касается и show.full.php (добавление комментариев) Дело в том, что валидатор ругается на .innerHTML, а он есть только тогда, когда на сайте можно добавлять комментарии как гость или пользователь для которого установлена капча (на www.dle-news.ru такого нету) ******************************************************** В файле feedback.php есть строчка: g['comm_req_f']}');return false}\"action=\"\" Не хватает пробела из-за чего валидатор и ругается, нада сделать так: g['comm_req_f']}');return false}\" action=\"\" Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 5 998 Опубликовано: 9 марта 2009 Рассказать Опубликовано: 9 марта 2009 Будут ли такие оптимизации для шаблона Default ? Нет. Стандартный шаблон предназначен для демонстрации возможностей скрипта, а не для использования на своих проектах и с этой функцией он справляется на отлично. Цитата Ссылка на сообщение Поделиться на других сайтах
www.zeos.com.ua 6 Опубликовано: 9 марта 2009 Рассказать Опубликовано: 9 марта 2009 Автор А валидность самого DLE скрипта будет исправление в новой версии? В первом и втором сообщении я много их привел :-) Заранее спасибо за ответ! Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 5 998 Опубликовано: 9 марта 2009 Рассказать Опубликовано: 9 марта 2009 А валидность самого DLE скрипта будет исправление в новой версии? В первом и втором сообщении я много их привел :-) все остальное конечно будет проверено Цитата Ссылка на сообщение Поделиться на других сайтах
www.zeos.com.ua 6 Опубликовано: 9 марта 2009 Рассказать Опубликовано: 9 марта 2009 Автор celsoft, можно дальше тему продолжать? Точнее писать если, что-то новое люди найдут из валидности? Цитата Ссылка на сообщение Поделиться на других сайтах
celsoft 5 998 Опубликовано: 9 марта 2009 Рассказать Опубликовано: 9 марта 2009 celsoft, можно дальше тему продолжать? Точнее писать если, что-то новое люди найдут из валидности? конечно Цитата Ссылка на сообщение Поделиться на других сайтах
Akela 9 Опубликовано: 30 июня 2009 Рассказать Опубликовано: 30 июня 2009 Занялся проверкой валидности сайта и нашел еще несколько ошибок, которые хорошо бы подправить : 1. выравнивание картинок типа “absmiddle” отменено и надо бы его везде заменить на “middle”. 2. в некоторых тэгах "<img " вместо обязательного параметра "alt=" написан "title=" - это есть во многих файлах в папках engine (show.* и т.д.). 3. В файлах админки есть много "onClick" вместо "onclick". Цитата Ссылка на сообщение Поделиться на других сайтах
gosu 1 Опубликовано: 30 июня 2009 Рассказать Опубликовано: 30 июня 2009 (изменено) cannot generate system identifier for general entity "user" 4x HTML errors (login.tpl) файл engine/init.php subaction=userinfo&user --> subaction=userinfo&user Изменено 30 июня 2009 пользователем gosu Цитата Ссылка на сообщение Поделиться на других сайтах
Nektov 59 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 (изменено) Изначально, если я не ошибаюсь, весь двигатель DLE ориентирован на XHTML 1.0 Transitional Только вот подогнал свой шаблон под стандарты w3.org. Дело геморройное, но возможное. Из описанного выше ничего незамечено. Но вот код, выводимый lostpassword.php упорно не проходит валидность. И кстати, аналогичная ситуация на dle-news.ru вот ссылка Result: 25 Errors Так вот, большая просьба, поправьте код lostpassword.php Если кто решил и поделится, буду очень благодарен. Да, чуть не забыл, кроме того валидатор ругается на <li> и </li> в файле show.full.php. Я это решил заменой <li> на <br /> А прочекить сайт на валидность мобильных браузеров кто-то пробовал? Вот ссылка http://www.w3.org/Mobile/ у меня This page is not mobile-friendly! Изменено 1 июля 2009 пользователем Nektov Цитата Ссылка на сообщение Поделиться на других сайтах
lifestar 18 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 Да, чуть не забыл, кроме того валидатор ругается на <li> и </li> в файле show.full.php. Я это решил заменой <li> на <br />я бы не стал так делать) Цитата Ссылка на сообщение Поделиться на других сайтах
Nektov 59 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 Да, чуть не забыл, кроме того валидатор ругается на <li> и </li> в файле show.full.php. Я это решил заменой <li> на <br />я бы не стал так делать) Почему? И если это возможно подробно. Цитата Ссылка на сообщение Поделиться на других сайтах
Akela 9 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 кстати, аналогичная ситуация на dle-news.ru вот ссылка Result: 25 Errors Ошибка из-за отмененного тэга <nobr>, вместо него рекомендуют использовать стили <span class=”nobr”> ... .nobr { white-space: nowrap; } Этот тэг (<nobr>) присутствует в 3-х файлах в админке и в нескольких файлах шаблонов. Цитата Ссылка на сообщение Поделиться на других сайтах
Nektov 59 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 кстати, аналогичная ситуация на dle-news.ru вот ссылка Result: 25 Errors Ошибка из-за отмененного тэга <nobr>, вместо него рекомендуют использовать стили <span class="nobr"> ... .nobr { white-space: nowrap; } Этот тэг (<nobr>) присутствует в 3-х файлах в админке и в нескольких файлах шаблонов. По поводу lostpassword.php. Это не решение, у моего скина и скина офф сайта, при проверке валидности матюгается совсем на другое. Посмотрите офф и мой сайт Цитата Ссылка на сообщение Поделиться на других сайтах
Akela 9 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 Это не решение Я писал про проверку на dle-news.ru у моего скина ... при проверке валидности матюгается совсем на другое Ну так надо исправить одну лишь строчку <input name="submit_lost" type="hidden" id="submit_lost" value="submit_lost"> на <input name="submit_lost" type="hidden" id="submit_lost" value="submit_lost" /> Цитата Ссылка на сообщение Поделиться на других сайтах
Nektov 59 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 Спасибо Akela помогло. Но вот опять ест пару «НО»: 1. в моём файле lostpassword.php (DLE 8.0) нет строки <input name="submit_lost" type="hidden" id="submit_lost" value="submit_lost"> , а есть строка <input name=\"submit_lost\" type=\"hidden\" id=\"submit_lost\" value=\"submit_lost\"> , которую нужно дополнить «/», а именно нужно писать <input name=\"submit_lost\" type=\"hidden\" id=\"submit_lost\" value=\"submit_lost\"/> 2. и все-таки, в lostpassword.php осталось 3 ошибки. Цитата Ссылка на сообщение Поделиться на других сайтах
lifestar 18 Опубликовано: 1 июля 2009 Рассказать Опубликовано: 1 июля 2009 Почему? И если это возможно подробно.а потому, что когда ссылки оформлены в виде списка (<li>) - мне очень удобно с ними работать и в CSS и если надо в JS. Я вообще не заморачиваюсь над валидностью, а тот же NOBR использует гугль) и все-таки, в lostpassword.php осталось 3 ошибки.какие именно? Цитата Ссылка на сообщение Поделиться на других сайтах
Nektov 59 Опубликовано: 2 июля 2009 Рассказать Опубликовано: 2 июля 2009 Я вообще не заморачиваюсь над валидностью, а тот же NOBR использует гугль) Ну, это дело вкуса. какие именно? Вот Цитата Ссылка на сообщение Поделиться на других сайтах
Akela 9 Опубликовано: 2 июля 2009 Рассказать Опубликовано: 2 июля 2009 Как обычно, Гугл нам поможет : во всех файлах, где есть "document.getElementById('dle-captcha').innerHTML = ...", нужно поменять (на примере lostpassword.php ): <script language='JavaScript' type="text/javascript"> function reload () { var rndval = new Date().getTime(); document.getElementById('dle-captcha').innerHTML = '<img src="{$path['path']}engine/modules/antibot.php?rndval=' + rndval + '" border="0" width="120" height="50" alt="" /><br /><a onclick="reload(); return false;" href="#">{$lang['reload_code']}</a>'; }; </script> на <script language='JavaScript' type="text/javascript"> //<![CDATA[ function reload () { var rndval = new Date().getTime(); document.getElementById('dle-captcha').innerHTML = '<img src="{$path['path']}engine/modules/antibot.php?rndval=' + rndval + '" border="0" width="120" height="50" alt="" /><br /><a onclick="reload(); return false;" href="#">{$lang['reload_code']}</a>'; }; //]]> </script> Данные изменения нужны для файлов - ajax: addcomments.php; modules : addnews.php, feedback.php, lostpassword.php, pm.php, register.php, show.full.php Цитата Ссылка на сообщение Поделиться на других сайтах
Nektov 59 Опубликовано: 2 июля 2009 Рассказать Опубликовано: 2 июля 2009 Снова спасибо Цитата Ссылка на сообщение Поделиться на других сайтах
Akela 9 Опубликовано: 2 июля 2009 Рассказать Опубликовано: 2 июля 2009 (изменено) Пожалуйста . Надеюсь, уважаемый celsoft включит эти маленькие исправления в следующую версию. А также все "onMouse" и "onClick" в "onmouse" и "onclick" соответственно. Изменено 2 июля 2009 пользователем Akela Цитата Ссылка на сообщение Поделиться на других сайтах
Akela 9 Опубликовано: 2 июля 2009 Рассказать Опубликовано: 2 июля 2009 (изменено) Продолжаем валидацию Вставка "<span id='dle-info'>" в index.php вроде бы не актуальна уже, да и стандарт не допускает наличие блоков внутри span. Стоит убрать. Много мест, где <br> надо поменять на <br />. Изменено 2 июля 2009 пользователем Akela Цитата Ссылка на сообщение Поделиться на других сайтах
WWW.ZEOS.IN 1 161 Опубликовано: 2 июля 2009 Рассказать Опубликовано: 2 июля 2009 Вставка "<span id='dle-info'>" в index.php вроде бы не актуальна уже, да и стандарт не допускает наличие блоков внутри span. Стоит убрать.Согласен, где-то читал про это Цитата Ссылка на сообщение Поделиться на других сайтах
gosu 1 Опубликовано: 5 июля 2009 Рассказать Опубликовано: 5 июля 2009 engine/editor/comments.php <textarea id="comments" name="comments" rows=10 cols=70>{$text}</textarea> -> <textarea id="comments" name="comments" rows="10" cols="70">{$text}</textarea> engine/editor/shortnews.php <textarea id="short_story" name="short_story" rows=10 cols=100>{$row['short_story']}</textarea> -> <textarea id="short_story" name="short_story" rows="10" cols="100">{$row['short_story']}</textarea> ..... Цитата Ссылка на сообщение Поделиться на других сайтах
Hydrargyrum 2 Опубликовано: 5 июля 2009 Рассказать Опубликовано: 5 июля 2009 Вставка "<span id='dle-info'>" в index.php вроде бы не актуальна уже, да и стандарт не допускает наличие блоков внутри span. Где это написано? Дайте ссылку на источник.. Зависит от ДОКТАЙПА документа.. Transitional допускает и всё валидно! Цитата Ссылка на сообщение Поделиться на других сайтах
Рекомендованные сообщения
Присоединяйтесь к обсуждению
Вы можете опубликовать сообщение сейчас, а зарегистрироваться позже. Если у вас есть аккаунт, войдите в него для написания от своего имени.