Bruno Dias

Results 313 comments of Bruno Dias

> > > > > > @jingyuexing Can you revert those formatting changes, please? > > > > Tested on macOS. > > Maybe you can create a action Yeah...I...

We are going in the right direction, just made some comments. It must be the last change. Obrigado pelo PR.

Awesome!! :tada: Can you include a test for this, please?

It looks great, but we also need to add an unit testing for this feature. Let me know if you need anything...

All unit tests are here https://github.com/reactjs/react-modal/tree/master/specs

@cyruscuenca @C0DE-RUNNER @kaichii @exaucae Have you looked in the resultant style for the components involved?

Hmmm...I have seen this before. Using react-modal in this case is not the best option. Your solution looks great, I'd recommend extract this code to its own library or maybe...

@btraut @bokibo Can you provide the actual error that this version is throwing, please?

This is a typescript thing...it's very sad. I don't know how the react version is injected on the react-modal type definition, so I can't help. If you want to give...