wrong value using Switch or TextField(!) in combination with updateOn: 'blur'  #1362

Activity

1. tsonevn

   self-assigned this

   on Jun 4, 2018
2. tsonevn commented on Jun 4, 2018

   

   tsonevn

   on Jun 4, 2018

   Contributor

   More actions

   Hi @gms1,
   Thank you for the sample project.
   I tested this case while using the Switch component with '@angular/forms'. Indeed the valueChanges method will not be executed on first Switch value change. For further info, please keep track on the issue.
3. tsonevn

   added

   bug

   backlog

   help wanted

   forms

   on Jun 4, 2018
4. gms1 commented on Jun 10, 2018

   

   gms1

   on Jun 10, 2018

   Author

   More actions

   Hi @tsonevn,

   regarding Switch and other controls, I wondered how an "updateOn: blur" could work, as long as there are no focus/blur events from the underlying framework for these controls.
   However, I am getting a similar behavior using Textfield, although focus/blur events are fully supported for this control.

   In the repo there is now a new folder 'testfield-updateOnBlur' with which the problem with TextField can be reproduced.

   Should I open a new issue for TextField?
   For Switch / Slider, ... this feature may not be that important, but with TextField we are now forced to unnecessarily validate all typos, or to implement a workaround every time that bypasses the standard validation
5. gms1

   changed the title [-]wrong value using Switch in combination with updateOn: 'blur' [/-] [+]wrong value using Switch or TextField(!) in combination with updateOn: 'blur' [/+] on Jun 11, 2018
6. gms1 commented on Jun 11, 2018

   

   gms1

   on Jun 11, 2018

   Last edited by gms1

   Author

   More actions

   added TextField to the title

   however, it is possible that the TextField-issue already has a solution:
   #1230

   see ControlValueAccessor

     * On blur (or equivalent), your class should call the registered function to allow
      * the forms API to update itself:
      *
      * ```ts
      * host: {
      *    '(blur)': '_onTouched()'
      * }
      * ```
   
7. gms1 commented on Jun 11, 2018

   

   gms1

   on Jun 11, 2018

   Author

   More actions

   verified: #1230 solves the TextField-part of this issue!

   👍React with 👍1Reacted by Nikolay Tsonev
8. tsonevn

   removed their assignment

   on Jun 22, 2018