---

Гумеров Артур Римович
Ссылка на макеты https://www.figma.com/design/eZRVhhqJoof1vqcM2f4cmc/TR-DS---Atoms?node-id=15410-1490&t=LqEtYunD0hImxKzs-0
13:54

Малей Александр Юрьевич
Гумеров Артур Римович
Ссылка на макеты https://www.figma.com/design/eZRVhhqJoof1vqcM2f4cmc/TR-DS---Atoms?node-id=15410-1490&t=LqEtYunD0hImxKzs-0
запросил доступ
13:55
Гумеров Артур Римович добавил(а) пользователя Чивилев Михаил Иванович, 13:57
Гумеров Артур Римович добавил(а) пользователя Степанов Максим Сергеевич, 14:11
Гумеров Артур Римович
Завёл единую фиче ветку feature/TRADERADAR-3401-icon-switch
Палец вверх
1
14:12
Старые иконки тут src\components\Icons\index.tsx
Новые иконки src\uikit\Icon\Icon.tsx
При замене удаляем файл со старой иконкой
изменено14:13
Гумеров Артур Римович добавил(а) пользователя Архипов Валерий Иванович, 14:57
Шабанов Александр Николаевич
@all А кто какие иконки будет заменять?
16:38
Комментарии
Архипов Валерий Иванович
По буквам может разбить, что бы не запутаться?
16:49
Шабанов Александр Николаевич
Архипов Валерий Иванович
По буквам может разбить, что бы не запутаться?
можно так
18:18
20.08.2026
Шабанов Александр Николаевич
@all Кто-нибудь уже начал? Я могу взять все на букву A
Минус
2
11:01
Гумеров Артур Римович
Архипов Валерий Иванович
По буквам может разбить, что бы не запутаться?
Единственный момент - не у всех есть доступ до всех виджетов, поэтому лучше начать с виджетов своего стрима
11:02
но можно и по буквам идти, просто если туда нет доступа то передавайте конкретно эту иконку другому
11:03
После замены надо проверять замену во всех местах где была старая. Но это очевидно, мне кажется
изменено11:03
Шабанов Александр Николаевич
Я начну тогда делать на букву A. Если будут проблемы, отпишусь, передам
Рукопожатие
1
11:05
26.08.2026
Гумеров Артур Римович добавил(а) пользователя Шкиндер Дмитрий Владимирович, 18:00
27.08.2026
Гумеров Артур Римович добавил(а) пользователя Шашков Артем Михайлович, 12:45
Степанов Максим Сергеевич

вопрос legacy иконки обязательно складывать в папку в папке?)
17:26
Гумеров Артур Римович
Степанов Максим Сергеевич

вопрос legacy иконки обязательно складывать в папку в папке?)
Изображение
нет, это просто legacy
а те которые мы заменяем это legacy legacy :D
17:27
от icons/migration тоже избавимся
17:27
их внедряли до того как поняли что нужно делать lucide
17:28
Степанов Максим Сергеевич
это промежуточный этап - все легаси будет в этой папке?
17:28
Гумеров Артур Римович
Степанов Максим Сергеевич
это промежуточный этап - все легаси будет в этой папке?
те которые заменяем просто удаляем, никуда перекладывать не нужно
17:29
если я тебя правильно понял
17:29
т.е. сейчас заменим все иконки src\components\Icons
Потом заменяем иконки из src\uikit\Icon\migration
17:31
Степанов Максим Сергеевич
ну вот я щас попробовал с буквы Z 
и сразу наткнулся на иконку которая отличается от lucide
я так понял мы старую иконку оформляем в новом Icon компонент но в легаси формате
17:31
Гумеров Артур Римович
Степанов Максим Сергеевич
ну вот я щас попробовал с буквы Z  и сразу наткнулся на иконку которая отличается от lucide я так понял мы старую иконку оформляем в новом Icon компонент но в легаси формате
о, интересно
17:31
скинь сюда и тегни Мишу, он подключится
17:32
такого не должно быть, по идее
17:32
а, поправка - флаги и логотипы не меняются, поэтому часть иконок в src\components\Icons останутся. Перенесём их чуть позже
изменено17:32
Степанов Максим Сергеевич

снизу lucide
17:34
@Чивилев Михаил Иванович что делать с иконкой
17:36
Гумеров Артур Римович
Степанов Максим Сергеевич

снизу lucide
Изображение
а что смущает, что они по-разному выглядят?
17:37
Чивилев Михаил Иванович
Не понял если честно. Не хватает какой то иконки в люсиде или что?
17:38
Степанов Максим Сергеевич
да, то что иконки разные смущяет
я конечно поменяю, но потом кто-нибудь придет и будет ругаться
17:38
так что пусть ругаются на тебя 
Улыбка до ушей
1
17:38
Чивилев Михаил Иванович
Так. То есть квадратной иконке соответствует круглая эта вот?
17:39
Гумеров Артур Римович
да, меняем, это согласовано с Анатолием)
17:39
Чивилев Михаил Иванович
А куда круглая делась
17:39
Степанов Максим Сергеевич

снизу lucide
Изображение
А что не так? Они одинаковые же
17:40
Гумеров Артур Римович
Чивилев Михаил Иванович
А что не так? Они одинаковые же
одна островатая, другая кругловатая видимо
17:40

@Степанов Максим Сергеевич ты же отсюда брал, да?
https://www.figma.com/design/eZRVhhqJoof1vqcM2f4cmc/TR-DS---Atoms?node-id=14558-32751&p=f&t=1y6HszdSozsDFzBQ-0
Палец вверх
1
17:40
Комментарии
Чивилев Михаил Иванович
Ну у люсида все иконки со скруглениями
17:41
Гумеров Артур Римович

это ещё лайтовые изменения, бывают и посильнее)
изменено17:41
Чивилев Михаил Иванович
Если бы они были 1 в 1 одинаковые какой смысл был бы менять одно на другое 
изменено17:41
Гумеров Артур Римович
Чивилев Михаил Иванович
Если бы они были 1 в 1 одинаковые какое смысл был бы менять одно на другое 
ну стандартизация компонента только если, у нас сейчас это зоопарк. Это уже рефакторинг внутренний конечно тогда был бы
изменено17:42
Чивилев Михаил Иванович
Не. Смотрите. Если вы считаете что какая то иконка не подходит приходите ко мне. Я не могу учесть вообще весь контекст всех стримов и всех продуктов
17:42
Я же по файлу этому соотносил а не по контексту в системе
17:43
Если надо я или другую подберу или новую нарисую
17:43
(Но лучше подобрать)
17:43
Гумеров Артур Римович
если мы сейчас заменим то потом заменить достаточно быстро будет
17:43
Чивилев Михаил Иванович
Степанов Максим Сергеевич

снизу lucide
Изображение
Вот эти имхо вообще 1 в 1
17:43
Гумеров Артур Римович
мы ещё и дедупликацию иконок делаем, очень нужное дело
изменено17:43
Чивилев Михаил Иванович
Ну да. Их там из 160 стало 92 вроде
17:44
Степанов Максим Сергеевич
фигму я чет упустил, все вижу что тут сопоставление сделано.
зря шороху навел)
17:44
а кнопки типа IconButton меняем?
17:48
Гумеров Артур Римович
Степанов Максим Сергеевич
а кнопки типа IconButton меняем?
от IconButton будем отказываться, делаем в <Button просто
17:49
там есть пропертя вроде бы
17:49
Степанов Максим Сергеевич
будем или прямо сейчас менять?
17:49
Гумеров Артур Римович
давай сразу
17:49
если будут проблемы то пиши, подумаем
17:49
в новой дизайн системе ведь нет отдельного IconButton, @Чивилев Михаил Иванович верно же?
17:49
просто button с разными настройками?
17:50
Чивилев Михаил Иванович
На данный момент да
17:51
Он нам нужен?
17:51
У компонента через настройки выключается текст
17:52
Гумеров Артур Римович
Чивилев Михаил Иванович
Он нам нужен?
не вижу смысла, зачем заниматься поддержкой двух очень похожих компонент
17:53
Шабанов Александр Николаевич
Гумеров Артур Римович
от IconButton будем отказываться, делаем в <Button просто
Я просто на Button заменяю
17:55
Степанов Максим Сергеевич
я чет мож туплю, но в IconButton есть проверти active - чтобы типа фокус был
а у новой это как делается?
17:55

17:56
Гумеров Артур Римович
ссылка на макеты с маппингом https://www.figma.com/design/eZRVhhqJoof1vqcM2f4cmc/TR-DS---Atoms?node-id=15410-1490&t=LqEtYunD0hImxKzs-0
фичеветка куда всё сливаем feature/TRADERADAR-3401-icon-switch
страница чтобы занимать иконки https://wiki.moex.com/pages/viewpage.action?pageId=962108192
Старые иконки тут src\components\Icons\index.tsx
Новые иконки src\uikit\Icon\Icon.tsx
При замене удаляем файл со старой иконкой
17:58
Гумеров Артур Римович закрепил(а) сообщение, 17:58
Гумеров Артур Римович
Гумеров Артур Римович
ссылка на макеты с маппингом https://www.figma.com/design/eZRVhhqJoof1vqcM2f4cmc/TR-DS---Atoms?node-id=15410-1490&t=LqEtYunD0hImxKzs-0 фичеветка куда всё сливаем feature/TRADERADAR-3401-icon-switch страница чтобы занимать иконки https://wiki.moex.com/pages/viewpage.action?pageId=962108192 Старые иконки тут src\components\Icons\index.tsx Новые иконки src\uikit\Icon\Icon.tsx При замене удаляем файл со старой иконкой
завёл страницу в конфлюенсе, впишите туда над чем работаете
17:58
Комментарии
Степанов Максим Сергеевич
я чет мож туплю, но в IconButton есть проверти active - чтобы типа фокус был а у новой это как делается?
это свойство нужно чтобы без наведения активировать подсказку типа?
17:59
Степанов Максим Сергеевич
Гумеров Артур Римович
завёл страницу в конфлюенсе, впишите туда над чем работаете
это старая вики?
17:59
Гумеров Артур Римович
Степанов Максим Сергеевич
это старая вики?
да
17:59
доступа нет?
17:59
Степанов Максим Сергеевич
или наоборот? у меня доступа нет на редактирование
изменено17:59
Гумеров Артур Римович
Степанов Максим Сергеевич
или наоборот? у меня доступа нет
сейчас закажу быстренько
17:59
Посмотрел, мне кажется active это аналог !disabled у обычной Button
18:04
Комментарии
Степанов Максим Сергеевич
Гумеров Артур Римович
Посмотрел, мне кажется active это аналог !disabled у обычной Button
не, disabled еще вырубает - нельзя нажать
18:05
Гумеров Артур Римович
Это свойство используется только в 4 местах
18:06

@Малей Александр Юрьевич одно из этих место это твой код, подскажи как оно точно работает
Грустно
1
18:06
Если не разберёмся то меняем IconButton на Button в тех местах где не используется свойство active
изменено18:08
Шабанов Александр Николаевич
Как я понял это свойство держит кнопку в состоянии pressed
18:09
Гумеров Артур Римович
Шабанов Александр Николаевич
Как я понял это свойство держит кнопку в состоянии pressed
хм, а зачем нам такое
18:09
Кстати уточнение - все 4 места использования  это код Саши)
Ждём его ответа
изменено18:11
Шабанов Александр Николаевич
Можно же это в новую Button перенести, если это нужно
18:11
Гумеров Артур Римович
Шабанов Александр Николаевич
Можно же это в новую Button перенести, если это нужно
угу, надо только понять надо ли нам такое
18:12
Степанов Максим Сергеевич

все очень сложно с этими иконками 
18:14
Гумеров Артур Римович
космический лифт, альфабанк, план эвакуации, штурвал, хоровод и гусеница
изменено18:15
Степанов Максим Сергеевич

все очень сложно с этими иконками 
Изображение
плохо читаются?
18:15
Степанов Максим Сергеевич
Гумеров Артур Римович
космический лифт, альфабанк, план эвакуации, штурвал, хоровод и гусеница
рэп?
Бицепс
1
Солнечные очки
1
18:16
Степанов Максим Сергеевич

все очень сложно с этими иконками 
Изображение
нет, цвет stroke у нее не тот, отсутствуют классы у иконки на которые завязан цвет
18:16

просто на первой же иконке все пошло как-то не гладко
Очень смешно
1
18:17
Гумеров Артур Римович
Степанов Максим Сергеевич
нет, цвет stroke у нее не тот, отсутствуют классы у иконки на которые завязан цвет
ну это же 1 раз поменять в общем компоненте иконок же?
18:17
с Мишей сейчас согласуем и всё
18:18
@Шабанов Александр Николаевич а у тебя проблем не было с цветом?
18:18
Степанов Максим Сергеевич

просто на первой же иконке все пошло как-то не гладко
Изображение
typical trade radar :)
18:18
Степанов Максим Сергеевич
но мне уже надо бежать, ппоробую позже либо завтра че-то сделать
Рукопожатие
1
18:19
Шабанов Александр Николаевич
Гумеров Артур Римович
ну это же 1 раз поменять в общем компоненте иконок же?
Погодите, там же currentColor
18:19
Гумеров Артур Римович
может быть стили из старого места что-то переопределили 
18:19
Шабанов Александр Николаевич
Гумеров Артур Римович
@Шабанов Александр Николаевич а у тебя проблем не было с цветом?
Были в хэдере виджета из-за общих стилей хэдера, я поэтому добавлял туда отключение стилей
18:21
Степанов Максим Сергеевич
ага, да, я нашел, вроде ок
18:21
Гумеров Артур Римович
Степанов Максим Сергеевич
ага, да, я нашел, вроде ок
надо всего лишь использовать временно реализованное свойство чтобы после того как подправим задеприкейтить его и снова смигрировать))
Истукан
1
18:22

надо медальки выдавать разных степеней
18:23
Валера вон участник вторых циклических войн
18:23
Степанов Максим Сергеевич
Скоро я так понимаю галактическая битва за редизайн
18:24
Гумеров Артур Римович
Степанов Максим Сергеевич
Скоро я так понимаю галактическая битва за редизайн
это на новый год!)
18:25
Понедельник
Степанов Максим Сергеевич
МРы мы куда скидываем?
https://gitlab-dev.tech.moex.com/info-trade-terminal/moex-terminal-front/-/merge_requests/5171
не стал менять IconButton на Button, получится слишком много каскадных изменений, тк в Button надо этот флаг добавлять или state - чтобы как в макетах, везде править тесты и тд. Нужно отдельной задачей
15:55
Гумеров Артур Римович
Переслано от  Гумеров Артур Римович понедельник, 15:57
Мры для замены иконок:
15:57
Степанов Максим Сергеевич
сливаем же в общую ветку 3401?
15:59
Гумеров Артур Римович
Степанов Максим Сергеевич
сливаем же в общую ветку 3401?
да, наберём пачку и отдадим на тест