Comments 11
мда, у меня собственно когда то была такая же идея, но я не стал это публиковать, как раз из-за того, что топик заминусуют =)
Обидно ведь. Всего одна использованная строчка, а помогает.
Не понимаю за что минусуют...
Не понимаю за что минусуют...
"Несогласные", так сказать.
Нет чтоб какой-ниб полезный комментарий написать, просто минусуют... Им же вкуснятинку подавай, шоб интересно было, а еще желательно в картинках.
Зачем собсвтвенный блог существует - вот вопрос.
Нет чтоб какой-ниб полезный комментарий написать, просто минусуют... Им же вкуснятинку подавай, шоб интересно было, а еще желательно в картинках.
Зачем собсвтвенный блог существует - вот вопрос.
Мне кажется, нужно разделить "Новые" на "Новые из общих блогов" и "Новые из персональных блогов". Чтоб кому интересно было, в персональные заглядывали. А то когда все в общей куче подобный топик потенциально понятное дело не так интересен, как топик из /top/ списка.
Поэтому и минусуют.
Поэтому и минусуют.
Наибольшая актуальность «скрыть комментарий» был когда спамили блоги длинными комментариями по несколько мегабайт…
А чтобы это исправить нужно всего пару строк изменить:
http://www.habrahabr.ru/js/habrahabr.js
function OpenComment(commentID)
{
$('hiddenl' + commentID).innerHTML = $('hiddenl' + commentID).innerHTML == 'раскрыть комментарий' ? 'скрыть комментарий' : 'раскрыть комментарий';
$('hiddenc' + commentID).style.display = $('hiddenc' + commentID).style.display == 'block' ? 'none' : 'block';
return false;
}
Ещё можно поменять месторасположение ссылки «скрыть комментарий» и span, который заключает комментарий, чтобы при нажатии на ссылку новая появлялась в конце скрытого комментария :)
А чтобы это исправить нужно всего пару строк изменить:
http://www.habrahabr.ru/js/habrahabr.js
function OpenComment(commentID)
{
$('hiddenl' + commentID).innerHTML = $('hiddenl' + commentID).innerHTML == 'раскрыть комментарий' ? 'скрыть комментарий' : 'раскрыть комментарий';
$('hiddenc' + commentID).style.display = $('hiddenc' + commentID).style.display == 'block' ? 'none' : 'block';
return false;
}
Ещё можно поменять месторасположение ссылки «скрыть комментарий» и span, который заключает комментарий, чтобы при нажатии на ссылку новая появлялась в конце скрытого комментария :)
некрасиво. Лучше проверять поле $('hiddenl' + commentID).style.display == "none" ? .. : .. ;
Всё равно :)
Не пойму в чём разница, пример рабочий, протестил
Не пойму в чём разница, пример рабочий, протестил
Потому, что если потребуется сменить текст фразы - его придется менять в двух местах. Отделяйте контент от функционала.
Гм. Хотя на самом деле я плохо вчитался в скрипт. Тогда уж лучше передавать текст параметром.
Параметром тем более плохо, представь что к каждому отрицательному комментарию — нужно передавать будет и дополнительный параметр… Не считаю это такой большой проблемой, так как смена этого названия будет раз в вечность, или реже… Конечно можно вбить константами в начала файла, но до этого было сделано по другому
да!
Sign up to leave a comment.
А почему «раскрыть комментарий» есть, а «скрыть» нет?