Выделение при наличии в регексе автоматически сгенерированного текста

[Oleg Sychev]

Originally reported on Google Code with ID 226

Необходимо стандартизовать поведение инструментов авторинга при выборке пустоты.

Ситуация 1: a()b, выбрана пустая подмаска.
В дереве обязательно выбирается и подмаска и пустота, только пустоту выбрать нельзя.
В графе выделяется внутренний узел пустоты, без подмаски.
В описании не выделяется ничего вообще, хотя надпись для пустоты и подмаски присутствует
 (?!)

Ситуация 2: (ab|cd|)ef  Выбрано в тексте вторая |
В дереве ситуация адекватная
В графе и описании выделена только пустота.

Reported by oasychev on 2013-09-15 17:24:15

[Oleg Sychev]

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

Валерий, кому это править?

Reported by oasychev on 2013-09-16 21:45:52

[Valeriy Streltsov]

Я думаю стоит сначала переделать дескрипшен на такой же режим работы как и остальные
тулзы (каждый раз запрос-ответ + проверка кэша), а там посмотрим.

Reported by vostreltsov on 2013-09-16 21:57:21

[Oleg Sychev]

Согласен, только вопрос на все тулзы должен быть один.

Я вот думаю, не добавит ли производительности если тестинг тул при обновлении будет
единым запросом с остальными тоже...

Reported by oasychev on 2013-09-16 21:59:07

[Oleg Sychev]

* не вопрос - запрос

Reported by oasychev on 2013-09-16 21:59:28

[Valeriy Streltsov]

Выделение пофиксил, остается переделать сам дескрипшен на выдачу упрощенной разметки
и отдачу ответов на каждый запрос

Reported by vostreltsov on 2013-09-17 10:37:25

[Oleg Sychev]

Валерий - вам осталось поправить чтобы в лексере учитывалось смещение и координаты были
по регексу пользователя сразу.

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

Reported by oasychev on 2013-09-19 20:24:35

[Oleg Sychev]

Есть проблемы при выдаче синтаксических ошибок - она неоткрытые скобки лепит к самой
первой открывающей и выдает отрицательные координаты....
Надо наверное в таком случае в самом узле ошибки переставить координаты к следующей
открывающей скобке...

Reported by oasychev on 2013-09-23 19:47:53

[Oleg Sychev]

Иванов - изменения в графе выглядят красиво, но реализованы криво - мало ли что и как
будет автоматически добавлено, нельзя дублировать поведение именно exact именно в данный
момент времени. 
Есть возможность по координатам или другим полям определить добавленные узлы. Уточните
у Стрельцова условие.

Терехов - попробуйте сделать выделение добавленных узлов как у Иванова - не лучше будет,
чем сейчас?

Пахомов - а у вас когда выделение добавленных узлов будет?!

Reported by oasychev on 2013-10-01 18:57:06

[Former user]

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

Reported by grvlter on 2013-10-01 19:36:34

<hr> * Attachment: 1.PNG<br>

[Oleg Sychev]

Вопрос по рисунку - к Терехову и Стрельцову - в такой ситуации лучше иметь фон самих
добавленных узлов серым или белым? Белый заметнее, но вызывает неправильные ассоциации
с главным регексом. У графа такой проблемы не стоит - там ассерты метками на дугах
показываются...

Reported by oasychev on 2013-10-01 19:55:00

[Former user]

Мне кажется серый лучше - он подчёркивает особенность узлов. Белый фон будет показывать
схожесть автоматически сгенерированных узлов и остальных узлов дерева...

Reported by grvlter on 2013-10-01 20:09:32

[Oleg Sychev]

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

Reported by oasychev on 2013-10-01 21:11:32

[Oleg Sychev]

Теперь работает во всех инструментах.

Reported by oasychev on 2013-10-18 18:09:11 - Status changed: Done