Ricardo Amaral
Ricardo Amaral
@bdbch A lot of tests are breaking because the new implementation doesn't seem to play well with `cy.type()`. Tried to replace with `realType()` ([ref](https://github.com/dmtrKovalenko/cypress-real-events)), and while I was able to...
> This change does really feel like an edge case and I'm not super confident in the change. For what it's worth, we've been using this change on [Todoist](https://todoist.com) for...
@nperez0111 Any thoughts on how to better fix this issue?
> Separately, I took a look at what remirror does (they seem to just not apply the text in this situation), I noticed that they are trying to preserve the...
> I think that the main goal here should be to not crash the editor and a secondary goal can be to get it to be marked up properly. I...
> Alright, I'm convinced by your argument. I wasn't trying to convince you per se 😅, just sharing my point of view. Ultimately, this is your call, of course 👍
@nperez0111 The test pipeline is still running, but I should've fixed all Cypress tests. It required the usage of [`cypress-real-events`](https://github.com/dmtrKovalenko/cypress-real-events) for typing instead of using Cypress's default `type` (which are...
> Thanks for this, didn't realize that you had a library in mind that worked for this. It's what we also use internally to solve issues like this (i.e. when...
@nperez0111 @bdbch @svenadlung Finally got all tests to pass, feel free to review.
@nperez0111 Same as this one, latest releases says this is fixed, but issue is still opened.