Skip to content

feat(tree): filtration (#UIM-14) #195

Merged
merged 12 commits into from
Aug 16, 2019
Merged

Conversation

lskramarov
Copy link
Contributor

No description provided.

@lskramarov lskramarov force-pushed the feature/UIM-14_filtering-in-tree branch from a47bf42 to 8d97043 Compare July 31, 2019 14:05
@lskramarov lskramarov requested a review from mikeozornin August 7, 2019 11:45
@lskramarov lskramarov self-assigned this Aug 7, 2019
@lskramarov
Copy link
Contributor Author

@mikeozornin можно посмотреть.

И нужны стили для "найденного/подсвеченного" текста. Пока сделано примитивно болдом...

@lskramarov lskramarov force-pushed the feature/UIM-14_filtering-in-tree branch from 60d5e3a to 0eff7c0 Compare August 8, 2019 11:52
Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Иногда ломается регистр:
http://d.mikeozornin.ru/mrlRUP
http://d.mikeozornin.ru/qP8tlF

http://d.mikeozornin.ru/AQKtHe
Жирность ломает цвет букв, кажется так лучше не делать. Цвет цветом, жирность жирностью.

http://d.mikeozornin.ru/ckBymR
Двоеточие показывается, но не ищется. Это почему так?

Остальные вопросы скорее к примеру.

Поле поиска должны быть с cleaner'ом.

http://d.mikeozornin.ru/eSDq71
Давай сделаем так, чтобы ноды с одинаковыми именами не были идентичны и не выбирались все сразу. Ну или сделаем ноды без одинаковых имен. Одна и та же нода в трех местах дерева это не ок. Так только в Сиеме и то это уже убирают.

И поиск сделать регистронезависимым.

@lskramarov
Copy link
Contributor Author

http://d.mikeozornin.ru/ckBymR
Двоеточие показывается, но не ищется. Это почему так?

так задумано, поиск по данным т.е. именам, а : задано в шаблоне и не является именем.

И поиск сделать регистронезависимым.

Поправил. + это можно настроить по месту...

Остальные вопросы скорее к примеру.

на особенности дев примера можешь не обращать внимание, поскольку он меняется. В сторибуке будет 'хороший' пример.

Остальное поправил, можно смотреть.

@lskramarov
Copy link
Contributor Author

Добавил тесты, если проверка пройдет нормально то можно посмотреть и влить.

@lskramarov lskramarov changed the title feat(tree): filtration (#UIM-14) WIP feat(tree): filtration (#UIM-14) Aug 15, 2019
@lskramarov lskramarov force-pushed the feature/UIM-14_filtering-in-tree branch from 05e1196 to 2f2a0cf Compare August 16, 2019 11:32
@lskramarov lskramarov merged commit 7fe6423 into master Aug 16, 2019
@lskramarov lskramarov deleted the feature/UIM-14_filtering-in-tree branch August 16, 2019 14:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants