Pull to refresh

Comments 37

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

This version didn't pass full review because of the following issues:

1) This completely freezes my browser.

Что-то тут не согласовывается.
Так я и прошу помочь с локализацией этой проблемы. Но что-то мне подсказывает, что не ExtJS в этом виноват.
А что именно вам подсказывает?
Судя по описанию проблемы, очень похоже на то, что «доски пытались стругать с помощью топора». Для подобного расширения вряд ли понадобится все «богатство палитры компонентов с ExtJS».
Попробуйте для сравнения сделать тоже с помощью другой JS либы, благо их существует масса.
Подсказывет например тот факт, что с сообществом более 100000 человек и многолетней историей разработки эта библиотека хотябы не должна фризить браузеры на популярных платформах. Кроме того, ни у кого я этого не замечал. Значит причины какие-то локальные.

>Попробуйте для сравнения сделать тоже с помощью другой JS либы, благо их существует масса.
К сожалению я не знаю ни одной подобной либы. Особенно, чтобы интерфейс, построенный на них, не выглядил как под Win 3.1.
Это великолепно! Как я ждал редактора закладок в панельном интерфейсе просто не передать! Даже думал разобраться с написанием плагинов под Far manager и запилить свой редактор. Лед тронулся! Ура.

Есть правда пока один существенный минус — я больше пользуюсь Хромом, и все закладки у меня там. Но начало положено.

Протестирую обязательно. Спасибо.
Я тоже долго его ждал. Вы оказались терпеливее )
Думал про то, чтобы сделать версию для хрома, но там ведь тоже сначала api и местные особенности изучить нужно, что не быстро. Если кто-то знающий возьмется, готов помочь, чем смогу.
Работает. Вот некоторые соображения:

1. Текущую строку на активной панели я бы выделил. Например другим цветом фона строки. (см. пример)
2. С сортировкой неоднозначность. Видимо по умолчанию сортируется по дате добавления, хотя это не очевидно.
3. Лично мне не хватает обычной сортировки по именам. Причем желательно сначала все каталоги потом все ссылки.
4. Не знаю как устроены записи в закладках, но мне была бы полезна функция упорядочивания списка по алфавиту (как описал выше), с возможностью сохранить эту сортировку в сам список закладок. (надеюсь понятно объяснил)
5. Горячие клавиши из нижнего меню я бы привел в соответствие некоторым стандартам для панельных менеджеров:
— F1 — окно помощи (как минимум описание горячих клавиш или целиком диалог помощи «?»)
— F2 —
— F3 — открыть ссылку во вкладке
— F4 — редактирование закладки (shift + f4 — создание закладки)
— F5 — копирование на соседнюю панель
— F6 — перенос на соседнюю панель
— F7 — создать папку
— F8 — удалить (папку / файл)
— F9 —
— F10
В дополнение как вариант:
Enter — открыть текущую ссылку во вкладке
Ctrl-Enter — открыть текущую ссылку в новом окне

5. Перемещение элемента из одной панели в другую выглядит нелогичным если на разных панелях открыта одна и та же папка (по моему мнению никаких действий происходить не должно, или выводить сообщ. что копируем в туже папку)
6. Для себя я бы изменил немного вывод информации в строках (на широком экране пропадает много места, и строк наоборот не хватает) — название и адрес выводил бы в одну строку просто в разных колонках (см. пример)
7. к папкам бы сделал примечания (см. пример)
8. к элементам интерфейса сделать подсказки, типа атрибута title (например я не понял что делает кнопка с треугольником, которая в верхней строке).
9. добавить элемент ввода строки для поиска фильтрации списка (желательно учитывать не только название закладки, но и метки и подробное описание)
10. добавить всплывающую подсказку с полным описанием закладки

image
Поясню некоторые пункты:
2) Сортировка по индексу (внутреннему в Firefox), чтобы порядок совпадал с порядком в стандартном интерфейсе.
3) К сожалению, прикрутить сейчас ещё какую-то сортировку достаточно трудно, поэтому не могу ничего обещать по этому поводу.
5) Enter и так работает, как вы описали.
5.2) Никакого копирования не происходит, а перемещение внутри папки, одна из основных задач, решаемых расширением. И возможно это потому, что элементы упорядочены.
6) Как по мне, то на одной строке названия и ссылки мешают друг другу.
7) Это выглядит полезно, но встает вопрос производительности. При 500 закладках папка уже загружает секунду или больше, что вызывает некоторый дискомфорт. Если при этом пройтись по всем подпапкам, то в однопоточной среде javascript можно получить просто неработоспособное расширение. Вообще конечно надо бы тесты провести, но по этой же причине Total commander по умолчанию не считает размер папок и уж тем более не делает это в основном потоке.
8) Честно говоря я думал, что после первого нажатия будет понятно, что она открывает в соседней панели ту же папку, что и в текущей. Но возможно, действительно нужно добавить подсказку.
9) Были планы по фильтрации, но тут примерно такая же ситуация, что и с сортировкой — надо менять архитектуру. В общем, не в ближайшем будущем, но появлением этой функциональности возможно.
1) вместо console.log, делайте как и принято в ExtJS 4.2:
// if (Ext.isDefined(Ext.global.console)) {
Ext.global.console.log('Console log');
}
//
2) content\about\esc.js — там у вас работа напрямую с innerHTML. Вообще окно сделано как то не так, раз уж используете экст :(
1) Просто эта версия заточена конкретно под Firefox >= 21, поэтому консоль по идее должна быть.
2) В окне about контент загружается в iframe, в котором ExtJS не подключается, поэтому есть немного самодеятельности, но опять же по совету со stackoverflow.
Вопрос в том, что все, что находится в тэгах debug последний senchacmd вырезает из релиза. незачем зазря это все сыпать в консоль
Как я писал в статье, мне пришлось делать простую замену SenchaCMD, т.к. мой проект она не смогла собрать. Кроме того, в консоль сейчас почти ничего не сыпется, за исключением пары ворнингов, по которым можно узнать что-то полезное при потенциальной ошибке у конечного пользователя.
Попробовал почитать код… Как то вы непривычно используете экст… Непонятно как senchacmd это компилирует… Хотя возможно вы стационарно натравливаете
Ответил выше. Она это не компилирует )
да я все понял по исходникам, такое оно точно не компилирует. сегодня вечером посмотрю, если никто до меня не поможет
Буду очень признателен.
Отличная штука! Еще было бы неплохо отдельные «диски» для истории и табов. Чтобы можно было туда-сюда копировать всякое.
Была такая мысль. Но сейчас я реализовал только то, чего именно мне не хватало, ввиду отсутствия фидбека. А пожелания можно высказывать в issue tracker, особо популярными могу заняться, как будет время.
А почему не
F2 — Rename
F3 — View
F4 — Edit
F5 — Copy
F6 — Move
F7 — New Folder
F8 — Delete
?
1) Закладки отличаются от файлов по набору основных операций над ними. Например копировать закладку мне не приходилось никогда в жизни и не знаю зачем это нужно (на крайний случай есть теги).
2) Зачем пропадать F1?
3) Открытие закладки в новом табе/окне частая операция, следовательно для неё нужны горячие клавиши, но вот место и сами F-клавиши ограничены.
В общем постарался расположить на F* самое основное.
Да, но люди пришедшие с двухпанельных файл менеджеров (я не скажу за все, но хотя бы Тотал Коммандер) будут чертыхаться. Было бы очень удобно иметь те же хоткеи. Я-то сам не буду пользоваться менеджером закладок, я не так много их делаю, просто чтобы понравиться пользователям, я бы советовал, хотя бы сделать переключение хоткеев на вышеуказанный режим.
Результат получился полезный, но я такую задачу решал бы, разбросав списки закладок по подпапкам, чтобы в каждой размер списка ссылок был не более экрана.: )

Догадка, где искать место подвешивания скрипта. Вы не запускали скрипт на Маке? Есть большая вероятность, что тестеры используют макбуки, а там поведение браузера может отличаться от Windows и Linux.
Пробовал на хакинтоше с рутрекера. Там всё работало нормально. Полноценный мак сейчас взять негде.
Могу я завтра на своем попробовать. Отпишусь в личку.
Все прекрасно работает, ничего не тормозит. Подробнее описал в личке.
В консоли оказалось три ошибки, которые под катом
image
Спасибо за тестирование.
У меня тоже есть такие ошибки, но, кажется это скорее ошибки со стороны браузера. Т.е. я не понимаю, что он этим хочет сказать. И если бы описанные ошибки действительно бы были, то внешне это было бы заметно, поэтому на них просто не стоит обращать внимание.
К слову сейчас еще проверил на OS X 10.7, которая на старом минике и тоже все прекрасно. Не знаю уж как ваше расширение могло подвесить браузер тестера, так как даже на старом слабеньком минике все летает. Это на мой макбук эйр с ssd еще можно было списать, но как уже написал, проверил и заведомо тормозной вариант.
Т.к. ни у кого так и не завис браузер, то я отправил на очередную проверку в текущем виде, в надежде, что просто попадется другой редактор. Может на этот раз прокатит.
Я бы на вашем месте перед повторной отправкой сделал легкий рефакторинг и попросил ревьювера указать, по-возможности, на конкретные фрагменты кода, вызвавшие сомнения.
Ну и конечно же пожелаю удачи и, как результата, публикации расширения.
Некоторую работу над замечаниями из письма я провел ещё до статьи на хабр. В частности экранирование html-символов в окне about. Да на eval проверил, чтобы где попало не использовался. Поэтому частично требования редакторов удовлетворены.
Если в этот раз при проверке что-то зависнет, то придется в irc с редакторами общаться. Думаю это единственный способ от них добиться ответа.
За пожелание спасибо, удача мне пригодится )
Если мне попадается интересная статья, которую сейчас некогда читать, то я складываю закладку прямо в корень, чтобы не забыть, а потом забываю и про саму статью и про закладку.

Странный подход. По моему субъективному мнению в закладки нужно складывать исключительно то, к чему нужно будет возвращаться снова и снова. А для описанного случая («интересная статья, которую сейчас некогда читать») больше подходят списки отложенного к прочтению — например, Pocket, который я использую сам и всем всячески рекомендую. Для удобства пользования сервисом и расширение для того же Firefox есть, и возможность почитать отложенные статьи на планшетике тоже имеется.
Надо посмотреть, что оно из себя представляет. Я о нем до этого просто не слышал.
Рекомендую посмотреть. Не обязательно на Pocket, есть множество альтернатив. Просто мой выбор пал на Pocket, потому и советую именно его. Но ключевая фраза — «список отложенного к прочтению».
Подобные сервисы специально заточены для соответствующих целей, а потому гораздо более удобны, особенно если использовать все их возможности. Например, уже упомянутое приложение под Android позволяет ко всему прочему настроить единообразный режим отображения и убрать все ненужное во время чтения оформление. Я например читаю светлый текст на черном фоне. После прочтения статья помечается как прочитанная и за нее можно забыть (никаких чисток как в случае с закладками производить не нужно).
Есть возможность запустить расширение при отключенной панели меню (Menu Bar)?
Можно добавить в закладки ссылку chrome://totalbookmarks/content/ и запускать через неё
Sign up to leave a comment.

Articles