что постоянно слушать keydown на документе, да еще и в отдельном компоненте – не очень хорошая затея.
Если бы слушать клавиатуру нужно было бы многим компонентам, то, конечно, стоило бы сделать отдельный сервис и из него эмитить ивенты, но пока только одному компоненту нужно слушать клавиатуру, то можно и так. Но опять таки – пугает что я слушаю клавиатуру на всем документе, а не только на компоненте.
Нужно отписки добавить. Плюс слушать на документе - не вижу ничего страшного
Отписки – unsubsribe обсервабла на ngOnDestroy?
Или отписку на HostListener?
На закрытии модалки отписаться просто и всё
Так от чего отписаться-то? От HostListener или keydowns$?
А смысл?
А он отпишется автоматом?
Не знаю. А зачем?
Чтоб память не текла?
Неоднозначная фраза 😂
А может?
Думаю да
Обсуждают сегодня