crimaniak / dirty-on-steroids Goto Github PK

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

Нужно пробовать оптимизировать производительность, возможно путем перевода селекторов с jquery на xpath.

var lastCheckinTimestamp = d3.localStorGetItem('lastCheckinTimestamp', 0 );

возвращает undefined

Надо как-то решить, что мы включаем по умолчанию, а что оставляем по умолчанию выключеным. Может быть на подсайте servicepack провести опрос?

При переходе в пост, за которым слежу:

1. автоматического перехода на первый новый коммент не происходит,
2. в индикаторе комментов справа в позициях «следующий мой» и «следующий новый» отсвечивают нули. При движении скроллом индикация проявляется, возврат скроллом обратно её уже не обнуляет.
   Opera 12.11 x64, Win7 x64

мне нужно посмотреть при возможности, похоже что-то с таймингом

у нас есть скрипт разворачивающий новые?

типа такого
span class="b-all_new_comments_link">2 комментария</span

Или мне надо как-то это дело добавить?

Думаю как минимум на первых порах эта фича будет востребована и необходима. По сути это где-то прямо в ядре можно сделать, ибо оно же вызывает run каждого модуля. Ну и вывод наверно можно на саму страницу, а не консоль. Хотя польза вывода на страницу сомнительна, если только совсем пользователям смотреть на то, что у них тормозит сп.

С дерти: http://servicepack.d3.ru/comments/367424/#8275075
Можно добавить возможность выбора стиля (как для новых комментов).

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

Пример из модуля hide.inbox.users:

    var insidePost = ( document.location.href.indexOf('comments') > -1 );
    var insideInbox = ( document.location.href.indexOf('inbox') > -1 );

Новые иконки стали вроде лучше, но читабильность ужасная (нет контраста). И слишком похоже на хабр. Может мы лучше сопрём что-то с нового д3? Как стрелочки для голосования по карме?

сабж не работает, по крайней мере скроллер не обновляется

При прокрутке далеко вся страница оказывается выделенной

Сейчас вот опять не работает уже фиг знает сколько времени. А драгоценное время загрузки скрипта жрёт.
Даже похоже, что он удалён со своего сервера.

Для того, чтобы избежать такого:

                var module = d3.getModule("Навигация по новым");
                if (module != null) {
                    module.countItems.call(module);
                    module.newPosition.call(module);
                }

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

Нужно проапдейтить readme в соответствии с изменившимся api. Также желательно сделать шпаргалку по наиболее полезным функциям и методам (возможно, с примерами использования).

В целях улучшения производительности работы модулей нужно помимо обработчика для одного комментария/поста сделать обработчик группы комментариев/постов. Цель — минимизировать количество вызовов сложной логики.

Сейчас часто сложные рассчеты по всем комментариям/постам вызываются при добавлении одного комментария/поста. То есть, обработка занимает время O(n²) вместо O(n).

Обработчик для группы может представлять собой функцию onUpdate, которая будет триггериться после того, как вся группа комментариев/постов уже добавлена, либо функции onCommentsBatch(comments), onPostsBatch(posts), которые будут триггериться после группового добавления сущностей и в которые будут передаваться обновленные сущности.
Мне первый вариант по некоторым причинам нравится больше.

По поводу реализации. Пока в голову приходит только clearTimeout/setTimeout при каждом новом посте.

1. Выключаем картинки в постах
2. Листаем кнопками новых до поста с выключенной картинкой.
3. Раскрываем картинку в посте.
4. Пытаемся пролистать наверх — упс!

Настройки Модуля "Показывать favicons доменов":
"Активировать" – включено.
"Показывать иконки": "при наведении"
Наводим на заголовок статьи.
Иконка показывается выше заголовка и обрезана в верхней части.
Firefox 16.0.2, Chrome 23.0.1271.91 m, Opera 12.11 (Windows 7 x32)
Скриншот:

Сделать продвинутый фильтр комментариев, предполагаемые опции:

Использовать относительное/абсолютное значение
Количество опций
Не скрывать мои
Не скрывать ответы на мои
Не скрывать родителей
Не скрывать картинки
Показывать селектом / линками
Полностью прятать / делать прозрачными

добавить в модуль ignorator

http://servicepack.d3.ru/comments/365697/#8283022

Uncaught TypeError: Object [object HTMLDivElement] has no method 'getElements'

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

Когда приходишь на последний комментарий и он новый.

Нужно уходить на base64 стринги в картинках, пока старый дёрти не грохнули. Особенно важно для скроллка и тултипа.

Каждый запрос d3.storage.get() у меня занимает 50-70 миллисекунд в опере.
В хроме - 0.
В FF пока не могу проверить.
Запись в опере - 0

Не знаю, чем вызвано: то ли самой оперой, то ли тем, что опера у меня - основной браузер, и много чего в local storage лежит, а хром только из коробки.

В итоге время выполнения модулей наполовину - это получение данных из storage.

Прошу проверить на своих браузерах или сказать, если это что-то известное.

Проверял так:
добавил в d3:

...........
runTimeTotal: 0,
buildMode: '@buildMode@',
getFromStorageTime:0,
getFromStorageTimes:0,
.......

        get: function(key, defaultValue)
        {
                        var start = new Date().getTime();
            var value = localStorage.getItem(key);
                        var end = new Date().getTime();
                        d3.getFromStorageTime += end - start;
                        d3.getFromStorageTimes ++;
            return value === null ? defaultValue : value;
        },

.......
if (console) console.log('get from storage time: ' + d3.getFromStorageTime + 'ms');
if (console) console.log('get from storage times: ' + d3.getFromStorageTimes + 'times');

Тресход должен выпиливать посты из массивов d3.posts или comments, иначе посдчёт не сходится

вот так выглядит

http://servicepack.d3.ru/comments/365697/#8270204

Модуль: hide.posts.by.rating.js

Теперь с ним всё плохо, можно открывать несколько окон для ответа и они создаются реалтайм. Надо заметно переделывать всё. К тому же, нужен предпросмотр для написания новых инбоксов. Возможно это будет отдельно, ибо опять всё везде по разному называется.

Я тут про эпичный баг реквест: http://servicepack.d3.ru/comments/376382/#new

Может реально использовать это https://gist.github.com/1143845? код короче будет немного

можно добавить стрелочку вниз из д3 темы после ссылки. Только не убейте favicons!

Подсасывать новую версию раз в сутки с кешированием в localStorage

Comments processor for module 'Навигация по новым' registered. Processing time 9161

Возможно, виноват мой последний фикс f544585

http://servicepack.d3.ru/comments/385620/

На основе рейтинга комментариев, как обсуждалось.

Subh

Похоже что пагинаторов нигде не будет, а значит все плагины работающие с постами/комментариями должны уметь работать с появляющимися нодами.

Так как явно будут общие части с d3, надо будет сделать иерархию и вынести их в общего предка.
Обновлять content/dirty.js больше не планирую, после создания content/lepra.js он будет удален.

В связи с изменением дерти тултип вместо количества комментов показывает undefined.

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

С дерти:

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

http://d3.ru/comments/384774/#8273275

Хоть баг и не наш, надо бы пофиксить.

http://labs.d3.ru/comments/381125/

прикрутим? весч дельная по-моему

Subj

1. Delete obsolete modules and code
2. Rename "extra" folder to "modules".
3. Create subfolders for modules by type, such as "content", "social", "other"
4. Create folder "libs" and move libraries, such as jquery.js there
5. Create folder "build" for build-related scripts

Возможно, нужно добавить иконку.

См. скриншот.
Должно быть d3.ru/user/username, сейчас ссылка ведет на subsite.d3.ru/user/username

Тут есть глубокий оптимизатор

https://developers.google.com/speed/articles/compressing-javascript