Total Commander

[yozhik]

[olegvsk]

yozhik
Мы имеем концепцию "Все настройки в одном файле (json)", вполне понятную и близкую и мне и, думаю, большинству пользователей. Аналог тех же стилей VCL, LookAndFeel в Java и тем несчётного числа IDE и редакторов. Проблема установки шрифта мне кажется надуманной, поскольку шрифт устанавливается для темы за несколько секунд.
Вот именно этого мне и не хватало в существующих средствах просмотра для TC - наглядной настройки тем и легкого назначения этих тем разным типам файлов (помимо простого назначения файловых ассоциаций, без копания в инишниках). Соответсвенно, плагин написан прежде всего по моим представлениям, как это должно работать, и менять что-то в архитектуре для меня смысла нет. Наверное, многие предпочтут другой подход, но, как заметил наш остроумный коллега выше, привычные альтернативы - есть. Так что продолжу дорабатывать плагин в сложившемся виде, конструктивная критика по-прежнему приветствуется.

[yozhik]

olegvsk
Ну, вот, надеюсь, точки над «и» мы расставили

Пара наблюдений:

Косячок в Theme-редакторе: Editor -> Common backgroung

Во время настройки цвета в Theme-редакторе: начинаю менять цвет -> активируется кнопка Save -> чтобы сохранить последние текущие изменения, нажимаю Save; а чтобы сбросить последние текущие изменения, закрываю редактор по Close или Esc. Может быть удобнее было бы добавить маленькую кнопку сброса, которая делала бы то же самое без закрытия редактора: сбрасывала изменения к состоянию до активации кнопки Save. Как-то так:

Т.е. если Save нажал, то уже не сбросишь, а если ещё не нажал, то можно. Это было бы удобнее в случае, когда настраиваешь элементы друг за другом по очереди, чтобы после каждой неудачной попытки не переоткрывать редактор.

В Theme-редакторе, в выпадающих списках цветов сейчас не понятно значение пункта None. Если его выбрать, то элемент окрашивается в чёрный цвет:

Но чёрный — это не None. Чёрный — это отсутствие света, но не цвета. Не знаю что тут предложить. Для чего вообще замышлялся этот None?

Далее, в Editor -> Common Background (далее CB) None вроде бы оправдан, но если открыть этот список и снова выбрать тот же None, то все зависимые от Common элементы *Background зальются чёрным с именем None. Может было бы очевидней, если бы вместо None в CB значилось <mixed>. Т.е. если во всех зависимых от CB элементах цвет одинаковый, то в CB отображается этот цвет, а если хоть в одном зависимом элементе другой цвет, то в CB — <mixed>.
_________________
Amo ergo sum

[olegvsk]

[Orion9]

yozhik
После такого разбора философий как-то и шутить больше не хочется

Вы правильно выразились: ваш пост - памятник коструктивизму. И разумеется, мы все ждем продолжения! ) Мне, например, интересен разбор хайлайтеров и самой темы подсветки, поскольку я не стал бы сводить проблему "на нет", говоря, что эта настройка занимает несколько секунд. Может, для знающего человека, который постоянно с этим работает и имеет большой опыт, это и выглядит довольно просто. Но новичку или обычному пользователю придётся попробовать и пощупать каждый параметр в отдельности, чтобы посмотреть, где и как отражается его изменение; побаловаться с цветами, чтобы найти для себя приемлемые; сохранить и еще потом не раз откатить всю тему, применив ее на других файлах и т.д и т.п. Какие же это секунды? Тут часами можно возиться. И именно это я имел в виду, когда говорил о танцах.

А что касается философии глобального шрифта, то она тут явно напрашивается, хотя и не является большой проблемой в действительности. Скорее просто "неудобством" при переключении на другие темы.

Опция Open any files - безусловно облегчила жизнь, но всё-равно считаю, что она должа быть включена по-умолчанию. Или нужна детект-строка (за шпору спасибо).

Кроме того, предложенной вами кнопки отмены мало - здесь нужна полноценная и широкоизвестная система дефолтных настроек с пометками измененных параметров и возможности отката каждого из них по отдельности. Почему автор компонента сразу это не сделал - остается только гадать, наверное, ему просто было лень. А вот автору плагина можно сказать только спасибо, что вообще решился взяться за это дело. И спешить тут не нужно. Если проект будет постоянно развиваться, то, думаю, со временем мало кому будет интересно Cuda уехал текст.)

[Orion9]

olegvsk
Поймал при переключении между кодировками UTF8 и ANSI на простом ini-файле:

[olegvsk]

[Orion9]

olegvsk

[olegvsk]

[Orion9]

[Orion9]

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

Более того, в контексте определенного хайлайтера многие параметры основной темы становятся просто избыточными и декоративными, поскольку не имеют никакого отношения к этому хайлайтеру. Самый яркий пример - обычный ini-файл, файл синтаксиса которого содержит всего 5 правил подсветки (SubRules 4, KeyList 1). Но чтобы изменить эти правила, нужно открыть всю тему и при этом увидеть кучу нерелевантных и сбивающих с толку параметров, типа AssemblerComment, AssemblerReservedWord, CharacterForeground, DirectiveForeground и т.д.

Поскольку в окне редактирования темы не предусмотрен фильтр по правилам конкретного синтаксиса, процесс создания отдельной темы для него не представляется такой уж лёгкой и простой задачей. Сейчас все это выглядит довольно неудобно и ненаглядно. Вероятно, здесь нужна отдельная прослойка в виде редактора темы в контексте определенного хайлайтера на основе выбранной (эталонной) темы. Все-равно одна (эталонная) тема не может подойти всем хайлайтерам, а следовательно их так и так придется создавать кастомно, чтобы полностью удовлетворить все потребности.

[olegvsk]

[Orion9]

[Destiny]

А почему для случая, если что-то не найдено в поиске - не выскакивает окошко с сообщением, что ничего не найдено? Неужели лучшим и оптимизированным решением является то, что сейчас сделано - вывод в статус-бар инфы, что "буббу": не найдено?

[olegvsk]