uralplan / psi-dev

Automatically exported from code.google.com/p/psi-dev
0 stars 0 forks source link

Подсветка в xml-консоли #479

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Прилагаю патч для подствеки синатксиса в 
xml-консоли. В составе архива патч на 
имеющиеся файлы и непосредственно класс 
подсветки синтаксиса.

Известные пробелмы:

1. не отличаются внешне входящие и 
исходящие станзы
2. цвета зашиты в файл xmlhighlighter.cpp
3. нет подсветки в окне набора станзы (так 
как его фон белый, а в консоли черный, то 
читабельным сделать оба не получилось 
ввиду предыдущей проблемы)

Original issue reported on code.google.com by tishka17 on 26 Dec 2011 at 5:22

Attachments:

GoogleCodeExporter commented 8 years ago
Скриншот

Original comment by tishka17 on 26 Dec 2011 at 6:04

Attachments:

GoogleCodeExporter commented 8 years ago
ну прикольно конечно, но нафига?
я вот ни разу не заморачивался по поводу 
подсветки.
другое дело, если бы это была часто 
используемая фича.

Original comment by rion4ik on 26 Dec 2011 at 7:04

GoogleCodeExporter commented 8 years ago
Лично я использую консольку при разработке 
бомбуса, да и другим разработчикам было бы 
удобно читать подсвеченный xml. Естественно, 
обычным юзерам, которые консоль видят раз в 
жизни, это не нужно.

Хочу заметить, что это не фичереквест, а 
патч (-;

Original comment by tishka17 on 26 Dec 2011 at 7:15

GoogleCodeExporter commented 8 years ago
Мне нравится. Иногда заглядываю в консоль 
слабо что понятно. А тут сразу все видно.

Original comment by drizt@land.ru on 26 Dec 2011 at 7:22

GoogleCodeExporter commented 8 years ago
> 1. не отличаются внешне входящие и 
исходящие станзы
вот это действительно плохо.

и да, при написании софта надо 
ориентироваться на пользователя, а не на 
разработчика.
совершенно другое дело, если бы подобные 
фичи оформлялись в виде плагинов. я одобряю 
необходимый рефакторинг кода для этого. 
тогда можно было бы сделать не только 
клевую xml консоль, а целый xmpp редактор.

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

Original comment by rion4ik on 26 Dec 2011 at 7:44

GoogleCodeExporter commented 8 years ago
tishka17, подсветка - это хорошо, но то, что не 
отличаются цвета - настолько большой минус, 
что перечеркивает плюсы. Мне, например, 
важнее сразу видеть, входящая станза или 
исходящая. Распарсить я ее и без подсветки 
могу))

Original comment by wadea...@gmail.com on 26 Dec 2011 at 8:02

GoogleCodeExporter commented 8 years ago
tishka17, клёвый патч! тока как то различать бы 
входящие и исходящие станзы, по цвету 
таймстампов, или кавечек, что ли :)

Original comment by maksim.maj on 28 Dec 2011 at 10:07

GoogleCodeExporter commented 8 years ago
И где обещанная новая версия патча?

Original comment by vladimir.shelukhin on 2 Feb 2012 at 9:18

GoogleCodeExporter commented 8 years ago
разраб свалил. всем пофиг. багу можно 
закрывать.

Original comment by rion4ik on 29 Aug 2013 at 6:13