search

positive-js / mosaic

Angular components ✨ 📦🐣 ✨
https://mosaic.ptsecurity.com
MIT License
142 stars 40 forks source link

feat(tree-select): fix highlightCorrectOption (#UIM-607) #790

Closed oburdasov closed 2 years ago

oburdasov commented 2 years ago

@lskramarov в tree-option.component.ts не проставляется hasFocus = true из-за того, что focusOrigin === 'program' и соответственно нет подсветки. image

Не уверен, почему при 'program' не нужна подсветка, Но пришлось поменять focusOrigin на 'keyboard' в highlightCorrectOption(). Что скажешь?

lskramarov commented 2 years ago

@lskramarov в tree-option.component.ts не проставляется hasFocus = true из-за того, что focusOrigin === 'program' и соответственно нет подсветки. image

Не уверен, почему при 'program' не нужна подсветка, Но пришлось поменять focusOrigin на 'keyboard' в highlightCorrectOption(). Что скажешь?

да, ок. Только еще нужно добавить тест, на это поведение.

oburdasov commented 2 years ago

@lskramarov добавил