fix(form-field): background-color for input in select, tree-select and dropdown (#DS-4725)#1398
Conversation
lskramarov
changed the title
🚨 E2E tests failedReview the report for details. 💡 Comment |
|
Visit the preview URL for this PR (updated for commit df09822): https://koobiq-next--prs-1398-iyfwhzvr.web.app (expires Sat, 07 Mar 2026 17:25:11 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c |
…d dropdown (#DS-4725)
🚨 E2E tests failedReview the report for details. 💡 Comment |
|
/approve-snapshots |
|
🔄 Updating snapshots. |
|
✅ Snapshots updated! |
| ) | ||
| .subscribe((focusedItem) => this.keyManager.updateActiveItem(focusedItem as KbqDropdownItem)); | ||
|
|
||
| this.search?.inOverlay.set(true); |
There was a problem hiding this comment.
может будем устанавливать значение через провайдеры?
providers: [{kbqFormFieldDefaultOptionsProvider({inOverlay:true})}]в таком случае, вместо model можно input использовать
readonly inOverlay = input(!!this.defaultOptions?.inOverlay, { transform: booleanAttribute });There was a problem hiding this comment.
какие преимущества это дает ?
There was a problem hiding this comment.
не нужно инжектить форм-филд и переопределять его свойства напрямую, вместо этого оперируем токенами и не нарушаем принцип инверсии зависимостей (классы должны зависеть от абстракций, а не от реализации)
There was a problem hiding this comment.
formField не инжектится, это же контент, который тут фактически есть и мы его донастраиваем (конкретный/один инстанс, не весь класс), а через провайд мы изменим поведение всем дочерним классам..
Для бага этого фикса более чем (максимально точечный фикс), а так да, хорошее место подсветил, спасибо!
3 participants
No description provided.