rumkin 6 июл 2008 в 21:29

Чем вам поможет Хабракомментатор или новый способ навигации по комментариям

2 мин

522

Habr

+25

Комментарии 49

posthuman 6 июл 2008 в 21:49

жаль, что в ff2 не работает.

rumkin 6 июл 2008 в 21:55

Я сейчас протестировать не могу. А вы смотрели что какое сообщение выдаёт консоль ошибок?

rumkin 6 июл 2008 в 22:09

Попробуйте ещё раз...

posthuman 6 июл 2008 в 22:21

сейчас тож не могу, вот до дома доеду и обдумаем.

posthuman 6 июл 2008 в 22:33

$("commentsdiv").getElementsByClassName is not a function
это firebug.

может это один из экстеншинов балуется?

в фаерфокс 3 все прикольно работает, пасиб и на этом)

rumkin 6 июл 2008 в 22:47

Хм… странно, это скорее всего с prototype проблемы, хотя…

rumkin 6 июл 2008 в 22:49

Мда… Станное поведение… Вроде prototype кроссбраузерный, а у вас Jquery не подключён ко всем страницам?

posthuman 6 июл 2008 в 22:55

нет. но ff 2 у меня перегружен расширениями.. а у вас во втором фаерфоксе работает?

rumkin 6 июл 2008 в 22:56

У меня его сейчас нет под рукой.

avenu 6 июл 2008 в 22:50

Я знаменит :)

rumkin 6 июл 2008 в 22:52

С этого места поподробнее ;)

unst 6 июл 2008 в 22:54

думаю он имеет ввиду, что засветился на скрине

avenu 6 июл 2008 в 22:55

Я есть на картинке поста :)

rumkin 6 июл 2008 в 22:55

Ах да! Поздравляю! ;)

avenu 6 июл 2008 в 22:58

Когда вечеринка? :)

rumkin 6 июл 2008 в 23:00

Сразу после церемонии вречения Хабраоскара ;)

agata 7 июл 2008 в 00:04

data: URL

agata 7 июл 2008 в 00:06

Может, стоит изображения через data:URL добавлять.
Прошу прощения, недописалось выше.

Power 7 июл 2008 в 00:28

Можно покритиковать?
1. Если уж пользоваться prototype.js, то на всю катушку, я так думаю. Кода будет меньше, а кроссбраузерность повысится :)
2. Зачем "cursor:pointer;cursor:hand;" ? Достаточно "cursor:pointer;".
3. У вас это не проявится, но для parseInt() лучше явно указывать вторым аргументом базу системы счисления, потому что parseInt("09") != parseInt("09", 10).
4. У вас код в текстовичке не совсем соответствует коду в посте...

rumkin 7 июл 2008 в 00:50

Честно говоря, я с prototype не работал, мне ближе jQuery, поэтому я не стал углубляться в изучение его возможностей, да и версия 1.4 менее навороченная чем 1.6, а с некоторыми вещами я честно говоря до конца не разобрался.
Поэтому с удовольствием послушал советы по более обширному использованию prototype, что бы вы улучши?

Power 7 июл 2008 в 02:29

Постараюсь вечером обработанную версию вашего скрипта выложить...

Power 7 июл 2008 в 22:07

Вот что получилось:
1. Bookmarklet (проверен в FF 2).
2. Исходный код букмарклета.
3. Версия для Greasemonkey (FF 2) — нужно просто вставить этот код в свой юзерскрипт (отличается от [2] только первой и последней строчками).

А вообще, конечно, с нетерпением ждём Суперхабра.

posthuman 7 июл 2008 в 23:33

Спасибо за FF 2!

rumkin 8 июл 2008 в 01:13

Большое спасибо, кое-что забыл уже!

hlomzik 7 июл 2008 в 01:23

имхо, к навигации это имеет малое отношение ;)
у меня вот все руки не доходят сделать вторую версию ХабраЛеса, ~~с блекджеком и шлюхами~~ с уменьшением отступов в том числе...

rumkin 7 июл 2008 в 01:38

Можете использовать мои функции я не против, код есть, вам осталось только допилить.
Просто мой код кросс-браузерный или GreaseMonkey есть под все браузеры?

hlomzik 7 июл 2008 в 02:11

Спасибо!
Сам код будет последним этапом, и про него сказать не могу ничего) Кстати, волнует один технический момент — как заставить код выполняться до полной загрузки страницы. Я попытался это сделать, однако неуспешно...

GreaseMonkey — инструмент для работы userscripts в ff. В опере userscripts нативны; и точно знаю, что они прикручиваются к сафари. Для ие в голову приходят только букмарклеты...

rumkin 7 июл 2008 в 02:40

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

hlomzik 7 июл 2008 в 02:51

Меня как раз интересует время до загрузки страницы ;) Это особенно актуально для Хабра с тяжелым кодом на длинных страницах.
Насколько я помню, доступ к DOM может быть и до полной его загрузки... Но вот вряд ли это нормально кроссбраузерно... Я привык работать через инлайн-скрипты с запусками инициализации (
В HabraForest я пытался сделать подсмотренную где-то вещь — просто запускать цикл, пока DOM полностью не загрузится, в котором проходиться по нужным элементам и на обработанные ставить маркер, чтобы пропускать их на дальнейших итерациях.

rossomachin 7 июл 2008 в 01:43

На Суперхабре комментарии реализованы по-новому. Готовьте ваши острые умы ;)

hlomzik 7 июл 2008 в 02:03

А вы инвайты на него не раздаете? ;)
Я забросил ХабраФорест в ожидании СуперХабра и собирался либо поучаствовать в обсуждении новой системы комментирования, либо сделать ХабраФорест уже под СуперХабр. А оказался облом в виде лимита кармы...

rossomachin 7 июл 2008 в 02:11

Как всё устаканится — откроем его для всех на привычном домене. Дело идёт к тому.

rumkin 7 июл 2008 в 02:09

Деревья сворачивать стало проще в 10 раз, а расширение комментов, можно сделать из учёта вложенности.

rossomachin 7 июл 2008 в 02:12

Всё так, да.

rumkin 7 июл 2008 в 02:42

А если я код напишу кроссбраузерный, включите его по-умолчанию?

rossomachin 7 июл 2008 в 02:45

Ммм, не понял вопрос.

rumkin 7 июл 2008 в 02:51

Ну, если я разработаю код, то сможите ли вы ключить его в код сайта, т.е. к Суперхабру прикрутите?

rossomachin 7 июл 2008 в 02:55

Прямо с ходу обещать не могу. Хотя и не вижу причин сказать нет. Сделайте

rumkin 7 июл 2008 в 03:04

*сможете

hlomzik 7 июл 2008 в 02:13

На СХ div'ы комментариев вложены друг в дружку? Это очень хорошо, если это так =)

rumkin 7 июл 2008 в 02:37

ul'ы

rossomachin 7 июл 2008 в 02:49

Реализовано на вложенных UL.

atomicxp 7 июл 2008 в 02:12

Логичней было бы не делать пробелы если внутри древосписка нет разделений ветвей, например:

1 коммент

 . 4 коммент

 . . 6 коммент

 . . . 7 коммент

 . 5 коммент

2 коммент

 . 8 коммент

 . . 9 коммент

3 коммент

То есть 6 и 7 коммент и 8 и 9 коммент отображать на одном уровне, пока кто-нибудь не допишет внутрь ответ.

rossomachin 7 июл 2008 в 03:16

Поставил букмарклет. Работает. Однако есть проблема с первыми комментариями в ветках, сильно уезжающими влево.
Весной как раз продумывали несколько похожих решений с возможностью автоматического и ручного сдвигания узких веток влево с учётом фактической ширины видимой части окна браузера и номера вложенности^* первого проблемного комментария.

^* — каким по счёту комментарий в данной ветке. Можно примерно прикинуть, что при 1024 px проблемы начинаются с 7-го комментария, при 1280 — с 11-го и т.д.(с числами могу и наврать, не помню точно).

rossomachin 7 июл 2008 в 03:19

Проще говоря, сдвигалка влево появляется возле первого проблемного комментария. По клику на неё влево уезжает не вся ветка, а лишь то, что находится ниже (ну и сам коммент, к которому присобачена сдвигалка).

rumkin 8 июл 2008 в 21:56

А, как быть с вложенными комментами, которые и тоже оказались зажатыми? Отключать для них сдвигалку? Или и их сдвигать? А если свернули родительский комментарий, их тоже сворачивать?