А почему «раскрыть комментарий» есть, а «скрыть» нет?

[nini]

мда, у меня собственно когда то была такая же идея, но я не стал это публиковать, как раз из-за того, что топик заминусуют =)

[ttim]

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

[nini]

"Несогласные", так сказать.
Нет чтоб какой-ниб полезный комментарий написать, просто минусуют... Им же вкуснятинку подавай, шоб интересно было, а еще желательно в картинках.
Зачем собсвтвенный блог существует - вот вопрос.

[nini]

Мне кажется, нужно разделить "Новые" на "Новые из общих блогов" и "Новые из персональных блогов". Чтоб кому интересно было, в персональные заглядывали. А то когда все в общей куче подобный топик потенциально понятное дело не так интересен, как топик из /top/ списка.

Поэтому и минусуют.

[GogA]

Наибольшая актуальность «скрыть комментарий» был когда спамили блоги длинными комментариями по несколько мегабайт…

А чтобы это исправить нужно всего пару строк изменить:

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, который заключает комментарий, чтобы при нажатии на ссылку новая появлялась в конце скрытого комментария :)

[SynteZZZ]

некрасиво. Лучше проверять поле $('hiddenl' + commentID).style.display == "none" ? .. : .. ;

[GogA]

Всё равно :)
Не пойму в чём разница, пример рабочий, протестил

[SynteZZZ]

Потому, что если потребуется сменить текст фразы - его придется менять в двух местах. Отделяйте контент от функционала.

[SynteZZZ]

Гм. Хотя на самом деле я плохо вчитался в скрипт. Тогда уж лучше передавать текст параметром.

[GogA]

Параметром тем более плохо, представь что к каждому отрицательному комментарию — нужно передавать будет и дополнительный параметр… Не считаю это такой большой проблемой, так как смена этого названия будет раз в вечность, или реже… Конечно можно вбить константами в начала файла, но до этого было сделано по другому

[MTonly]

да!