Сохраняет табуляторы в теге code и не даёт им превращаться в пробелы
[code=php]), но лично для меня это не столь важно, потому что при работе с кодами достаточно хотя бы просто иметь возможность скопировать их в первоначальном виде, это уже хорошо. Возможно, автор исправит эту деталь в следующих версиях своего мода.
Лично я доволен результатом. Хотя это пока бета, для subsilver2 тоже пока не делал.
типа того.
В FF 26 всё ok. Ща поставлю 27... поставил, всё OK.Shredder писал(а):в FF 27 - не пашет...
И правда... Однако, только в случае "выделить всё" и затем копировать. Будем копать.Shredder писал(а): Если скопировать код, то при его вставке добавляет паразитные пробелы перед каждой строчкой. На твоём сайте то же самое.
Что есть, то есть... Однако, поправимо.Shredder писал(а):файл highlight.pack.js продуцирует ошибки javascript
На всякий случай: я не имел в виду, что совсем не работает. Речь шла именно об этих пробелахc61 писал(а):поставил, всё OK.
Не только. Если просто выделить мышью от начала до конца, не нажимая "выделить всё", то будет то же самое.c61 писал(а): только в случае "выделить всё" и затем копировать
[code=css], то при нажатии "выделить всё" просто кидает на главную без каких-либо объяснений... И баг с пробелами именно в этом случае (css) никуда не ушёл. И опять "отличился" FF. На твоём же сайте, в 3-м, 9, 10 и 12-м пунктах вместо выделения кидает наверх страницы, причём в п. 12 - только для первого куска кода. Именно если полностью выделить мышью в FF?c61 писал(а):Баг с пробелами для code=css не наблюдаю почему-то.
Обнаружил баг в FF. Исправление в forum_fn.jsShredder писал(а):Именно если полностью выделить мышью в FF?
Код: Выделить всё
// FireFox bug
e.innerHTML = e.innerHTML + ' ';
r.setStart(e.firstChild, 0);
r.setEnd(e.lastChild, e.lastChild.textContent.length-1);
Код: Выделить всё
function selectCode(a)
{
// Get ID of code block
var e = a.parentNode.parentNode.getElementsByTagName('CODE')[0];
// Not IE and IE9+
if (window.getSelection)
{
var s = window.getSelection();
// Safari
if (s.setBaseAndExtent)
{
s.setBaseAndExtent(e, 0, e, e.innerText.length - 1);
}
// Firefox and Opera
else
{
// workaround for bug # 42885
if (window.opera && e.innerHTML.substring(e.innerHTML.length - 4) == '<BR>')
{
e.innerHTML = e.innerHTML + ' ';
}
var r = document.createRange();
var ua = navigator.userAgent;
if (window.opera || (ua.search(/MSIE/) > -1))
{
r.selectNodeContents(e);
}
else
{
// FireFox bug
e.innerHTML = e.innerHTML + ' ';
r.setStart(e.firstChild, 0);
r.setEnd(e.lastChild, e.lastChild.textContent.length-1);
}
s.removeAllRanges();
s.addRange(r);
}
}
// Some older browsers
else if (document.getSelection)
{
var s = document.getSelection();
var r = document.createRange();
r.selectNodeContents(e);
//r.setStart(e.firstChild, 0);
//r.setEnd(e.lastChild, e.lastChild.textContent.length);
s.removeAllRanges();
s.addRange(r);
}
// IE
else if (document.selection)
{
var r = document.body.createTextRange();
r.moveToElementText(e);
r.select();
}
}
Вроде xml и html одно и то же, для html надо указывать xml.Shredder писал(а):Не совсем понял - html это для мода подсветки то же самое, что и xml? Когда пишешь code=html, то просто обрезает часть =html и всё, а на сайте, где скачивается скрипт подсветки, HTML и XML поставлены около одного и того же чекбокса.
Завтра-послезавтра проверю. Возможно, и не работала)Shredder писал(а):в IE8 подсветка вообще не работает
Похоже это из-за бага в FF. Сейчас не добавляются ?Shredder писал(а):в самый хвост кода в некоторых случаях добавлялись какие-то паразитные пробелы
Код: Выделить всё
function selectCode(a)
{
// Get ID of code block
var e = a.parentNode.parentNode.getElementsByTagName('CODE')[0];
// Not IE and IE9+
if (window.getSelection)
{
var s = window.getSelection();
// Safari
if (s.setBaseAndExtent)
{
s.setBaseAndExtent(e, 0, e, e.innerText.length - 1);
}
// Firefox and Opera
else
{
if(e.innerHTML.length > 6)
{
if(e.innerHTML.substring(e.innerHTML.length - 6) == ' ')
{
e.innerHTML = e.innerHTML.slice(0, -6);
}
}
// workaround for bug # 42885
if (window.opera && e.innerHTML.substring(e.innerHTML.length - 4) == '<BR>')
{
e.innerHTML = e.innerHTML + ' ';
}
var r = document.createRange();
var ua = navigator.userAgent;
if (window.opera || (ua.search(/MSIE/) > -1))
{
r.selectNodeContents(e);
}
else
{
// FireFox bug
e.innerHTML = e.innerHTML + ' ';
r.setStart(e.firstChild, 0);
r.setEnd(e.lastChild, e.lastChild.textContent.length-1);
}
s.removeAllRanges();
s.addRange(r);
}
}
// Some older browsers
else if (document.getSelection)
{
var s = document.getSelection();
var r = document.createRange();
r.selectNodeContents(e);
//r.setStart(e.firstChild, 0);
//r.setEnd(e.lastChild, e.lastChild.textContent.length);
s.removeAllRanges();
s.addRange(r);
}
// IE
else if (document.selection)
{
var r = document.body.createTextRange();
r.moveToElementText(e);
r.select();
}
}Посмотрел - в стандартном phpbb и на phpbbguru (где тоже установлена подсветка) багов с пробелами нет.c61 писал(а):Эта фича, кстати, присутствует в исходном коде фунции, то есть фича самого phpbb.