You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We often fail on search tests as sometimes the result of the serach appears after some time. It would be good to have option to check current state of the field (inactive, searching, found, not-found).
Currently we only blink it with some class to show red color for a few miliseconds.
Actual Behavior
How it works in COOL (red blink works only one time after we type something wrong and before we were in "found" or "clear" state) searchcool-2024-12-31_11.08.26.webm
Example how it works in browser's search field on a webpage, where red color is present until we blur/search again: loremfirefox-2024-12-31_10.42.58.webm
I propose to add classes for all the mentioned states:
inactive-search (empty / out of focus)
searching (started, waiting for response form server)
found
not found
Change possible ony on user action, not after timeout. This will allow to check it's state in cypress to make tests more stable.
Also we can style all of these states if needed.
Describe the Bug
We often fail on search tests as sometimes the result of the serach appears after some time. It would be good to have option to check current state of the field (inactive, searching, found, not-found).
Currently we only blink it with some class to show red color for a few miliseconds.
Actual Behavior
How it works in COOL (red blink works only one time after we type something wrong and before we were in "found" or "clear" state)
searchcool-2024-12-31_11.08.26.webm
Example how it works in browser's search field on a webpage, where red color is present until we blur/search again:
loremfirefox-2024-12-31_10.42.58.webm
I propose to add classes for all the mentioned states:
Change possible ony on user action, not after timeout. This will allow to check it's state in cypress to make tests more stable.
Also we can style all of these states if needed.
cc: @pedropintosilva @banobepascal what do you think?
The text was updated successfully, but these errors were encountered: