Перейти к публикации

Рекомендованные сообщения

Сайт: 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

Изменено пользователем www.zeos.com.ua
Ссылка на сообщение
Поделиться на других сайтах

А также в файле 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=\"\"

Ссылка на сообщение
Поделиться на других сайтах

Будут ли такие оптимизации для шаблона Default ?

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

Ссылка на сообщение
Поделиться на других сайтах

А валидность самого DLE скрипта будет исправление в новой версии? В первом и втором сообщении я много их привел :-)

Заранее спасибо за ответ!

Ссылка на сообщение
Поделиться на других сайтах

А валидность самого DLE скрипта будет исправление в новой версии? В первом и втором сообщении я много их привел :-)

все остальное конечно будет проверено

Ссылка на сообщение
Поделиться на других сайтах

celsoft, можно дальше тему продолжать? Точнее писать если, что-то новое люди найдут из валидности? :rolleyes:

Ссылка на сообщение
Поделиться на других сайтах

celsoft, можно дальше тему продолжать? Точнее писать если, что-то новое люди найдут из валидности?

конечно

Ссылка на сообщение
Поделиться на других сайтах
  • 3 месяца спустя...

Занялся проверкой валидности сайта и нашел еще несколько ошибок, которые хорошо бы подправить :

1. выравнивание картинок типа “absmiddle” отменено и надо бы его везде заменить на “middle”.

2. в некоторых тэгах "<img " вместо обязательного параметра "alt=" написан "title=" - это есть во многих файлах в папках engine (show.* и т.д.).

3. В файлах админки есть много "onClick" вместо "onclick".

Ссылка на сообщение
Поделиться на других сайтах

cannot generate system identifier for general entity "user"

4x HTML errors (login.tpl)

файл engine/init.php

subaction=userinfo&user
-->
subaction=userinfo&amp;user

Изменено пользователем gosu
Ссылка на сообщение
Поделиться на других сайтах

Изначально, если я не ошибаюсь, весь двигатель 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!

Изменено пользователем Nektov
Ссылка на сообщение
Поделиться на других сайтах
Да, чуть не забыл, кроме того валидатор ругается на <li> и </li> в файле show.full.php. Я это решил заменой <li> на <br />
я бы не стал так делать)
Ссылка на сообщение
Поделиться на других сайтах

Да, чуть не забыл, кроме того валидатор ругается на <li> и </li> в файле show.full.php. Я это решил заменой <li> на <br />
я бы не стал так делать)

Почему? И если это возможно подробно.

Ссылка на сообщение
Поделиться на других сайтах

кстати, аналогичная ситуация на dle-news.ru вот ссылка Result: 25 Errors

Ошибка из-за отмененного тэга <nobr>, вместо него рекомендуют использовать стили

<span class=”nobr”>

...

.nobr { white-space: nowrap; }

Этот тэг (<nobr>) присутствует в 3-х файлах в админке и в нескольких файлах шаблонов.

Ссылка на сообщение
Поделиться на других сайтах

кстати, аналогичная ситуация на dle-news.ru вот ссылка Result: 25 Errors

Ошибка из-за отмененного тэга <nobr>, вместо него рекомендуют использовать стили

<span class="nobr">

...

.nobr { white-space: nowrap; }

Этот тэг (<nobr>) присутствует в 3-х файлах в админке и в нескольких файлах шаблонов.

По поводу lostpassword.php. Это не решение, у моего скина и скина офф сайта, при проверке валидности матюгается совсем на другое.

Посмотрите офф и мой сайт

Ссылка на сообщение
Поделиться на других сайтах

Это не решение

Я писал про проверку на 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" />

Ссылка на сообщение
Поделиться на других сайтах

Спасибо 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 ошибки.

Ссылка на сообщение
Поделиться на других сайтах

Почему? И если это возможно подробно.
а потому, что когда ссылки оформлены в виде списка (<li>)

- мне очень удобно с ними работать и в CSS и если надо в JS.

Я вообще не заморачиваюсь над валидностью, а тот же NOBR использует гугль)

и все-таки, в lostpassword.php осталось 3 ошибки.
какие именно?
Ссылка на сообщение
Поделиться на других сайтах

Я вообще не заморачиваюсь над валидностью, а тот же NOBR использует гугль)

Ну, это дело вкуса.

какие именно?

Вот

Ссылка на сообщение
Поделиться на других сайтах

Как обычно, Гугл нам поможет :) :

во всех файлах, где есть "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

Ссылка на сообщение
Поделиться на других сайтах

Пожалуйста :) .

Надеюсь, уважаемый celsoft включит эти маленькие исправления в следующую версию.

А также все "onMouse" и "onClick" в "onmouse" и "onclick" соответственно.

Изменено пользователем Akela
Ссылка на сообщение
Поделиться на других сайтах

Продолжаем валидацию :)

Вставка "<span id='dle-info'>" в index.php вроде бы не актуальна уже, да и стандарт не допускает наличие блоков внутри span. Стоит убрать.

Много мест, где <br> надо поменять на <br />.

Изменено пользователем Akela
Ссылка на сообщение
Поделиться на других сайтах
Вставка "<span id='dle-info'>" в index.php вроде бы не актуальна уже, да и стандарт не допускает наличие блоков внутри span. Стоит убрать.
Согласен, где-то читал про это :(
Ссылка на сообщение
Поделиться на других сайтах

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>

.....

Ссылка на сообщение
Поделиться на других сайтах

Вставка "<span id='dle-info'>" в index.php вроде бы не актуальна уже, да и стандарт не допускает наличие блоков внутри span.

Где это написано? Дайте ссылку на источник..

Зависит от ДОКТАЙПА документа..

Transitional допускает и всё валидно! <_<

Ссылка на сообщение
Поделиться на других сайтах

Присоединяйтесь к обсуждению

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

Гость
Ответить в тему...

×   Вставлено в виде отформатированного текста.   Вставить в виде обычного текста

  Разрешено не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отобразить как ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставить изображения напрямую. Загрузите или вставьте изображения по ссылке.

×
×
  • Создать...