Skip to content

fix(evo-datepicker): Исправить флоу инициализации и триггер событий изменений (v8)#1299

Merged
harlamenko merged 4 commits intomasterfrom
fix/evo-datepicker-initializing-v8
Mar 12, 2026
Merged

fix(evo-datepicker): Исправить флоу инициализации и триггер событий изменений (v8)#1299
harlamenko merged 4 commits intomasterfrom
fix/evo-datepicker-initializing-v8

Conversation

@harlamenko
Copy link
Copy Markdown
Collaborator

  1. Исправлена инициализация flatpicker-а: значение по умолчанию берется из control value, вместо config.defaultDate.
    1.1. значения синхронизированны
    1.2. дефолтное значение теперь не нужно прописывать в двух местах
    1.3. визуал дейтпикера соответствует значение его контрола
  2. Исправлени логика CVA (в частности убран onChange из writeValue)

1. `writeValue` — убраны вызовы propagateChange/onChange. writeValue
     теперь только синхронизирует flatpickr с входящим значением формы,
      не уведомляя форму обратно (контракт CVA).
  2. flatpickr `onChange` колбэк — вызывает propagateChange напрямую
     вместо writeValue, чтобы пользовательское действие корректно
     уведомляло форму без рекурсии.
  3. `setDateFromInput(date, triggerChange = true)` — добавлен параметр
      triggerChange. Все программные вызовы (ngAfterViewInit,
     updatePickerIfNeed, ngOnChanges) передают false, чтобы не
     тригерить flatpickr onChange.
  4. `isValueExist()` — учитывает pendingValue до создания flatpickr,
     чтобы шаблонные @if были стабильны в рамках одного CD-цикла.
@harlamenko harlamenko force-pushed the fix/evo-datepicker-initializing-v8 branch from 43b7f89 to 6bed8a6 Compare March 10, 2026 14:37
@harlamenko harlamenko marked this pull request as ready for review March 10, 2026 14:57
@harlamenko harlamenko merged commit 8152206 into master Mar 12, 2026
4 checks passed
@harlamenko
Copy link
Copy Markdown
Collaborator Author

🎉 This PR is included in version 8.21.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants