David Sancho

Results 138 comments of David Sancho

Oh, that seems massive. Do we have any issue on rescript for this? Seems something that these bindings can include out of the box, but seems a little wrong to...

I'm happy to contribute to it, give me a little bit of time and should be simple to add. Thanks!

I ported a ppx from omp to ppxlib and this trick was implemented there. Haven't struggled with the issue myself. Repo: https://github.com/ml-in-barcelona/react-jsx-ppx > Also, have you seen the difference in...

Pretty old issue, but it's fixed in https://github.com/reasonml/reason/issues/2300 When we publish a new version (probably a patch, will ping back here)

Hi, box-shadow (same as text-shadow) deserves a better error messaging here. Bot single values and multiple values are supported, but they need to be expressed with full form. ``` box-shadow:...

I'm having similar issues here. Tried to workaround it without forking de yUndoPlugin. Tried to replicate the behaviour inside a codesandbox, but I'm afraid isn't the same flow: https://codesandbox.io/s/tiptap-react-editor-forked-s7hvdk?file=/src/App.tsx Running...

I'm not sure if it's still relevant, but meanwhile, locations are a little broken between the Reason parser, and the JSX ppx... This still happens.

Not sure how's worth adding an external like renderWithReturn considering that most APIs might change for React 17/18. I will close now since it seems like isn't being needed, but...

From #460, I think the issue name should be "genType don't annotate dependencies" or similar. --- I wanted to share one bug that I have been facing using genType and...

That's a little harder in Reason than it might sound, we can't change the type definition of a function based on a value. Possible solutions like GADTs or moving this...