rescript-webapi

rescript-webapi copied to clipboard

Look at ways to reduce code generation

1

Tasks already logged: - #3 (removes `asHtmlDocument` function, and probably a few others) - #7 (reduces need for 'convert' functions) - #30 (the generated JS for `bs-fetch` that we imported...

TheSpyder

Convert SelectionDirection and SelectionMode in HTMLInputElement

1

>Our long term aim is to reduce the runtime code - when this library was written polymorphic variants were not strings. We can't convert existing modules until we do a...

illusionalsagacity

Passing `Headers.t` to `RequestInit.make` and `Request.t` to `fetch`

1

I'm not sure about how this would be done but it would be great if I could pass `Headers.t` to `RequestInit.make` and `Request.t` to `fetch`. This is possible in the...

tom-sherman

The type of `observe` is currently an open object: https://github.com/tinymce/rescript-webapi/blob/18d9d553201d575fa8e194d3370298b556b28747/src/Webapi/Dom/Webapi__Dom__MutationObserver.res#L5 To make this type safe, we can use `@obj` to create the object using optional arguments: https://github.com/tinymce/rescript-webapi/blob/5219ef22571c7e79ac221d049c1ad19546270c99/src/Webapi/Webapi__Blob.res#L29-L30 There are 7...

TheSpyder

I have only used [changesets](https://github.com/atlassian/changesets) on one other project but I was very happy with it. In particular, the github actions to automate publishing releases (and pre-releases) are fantastic.

TheSpyder

I'm not sure what this looks like but it sounds like a good idea.

TheSpyder

Update Fetch API

4

The new fetch API is just a copy of `bs-fetch` with a bit of re-arranging to separate out the inner modules (this aids tree shaking). There are far better ways...

TheSpyder

In many cases, ReScript syntax is unambiguous enough that a trailing `_` is no longer required in places it was for ReasonML and OCaml syntax. A codebase search for `_:`...

TheSpyder

Merge Document and HTMLDocument

1

It turns out browsers have been doing this for a while: https://bugs.chromium.org/p/chromium/issues/detail?id=238368 It would remove one of the sources of type conversion hassle given our target market for this library.

TheSpyder

Should we namespace the library with `"namespace": "Webapi"`?

4

This is an open question, based on some experimental code I wrote (branch [namespace](https://github.com/tinymce/rescript-webapi/tree/namespace)) and a [discussion in the ReasonML discord](https://discord.com/channels/235176658175262720/235199190781788160/849536037185126402). I found a way to make webapi compile in...

TheSpyder