Feliz.MaterialUI icon indicating copy to clipboard operation
Feliz.MaterialUI copied to clipboard

updated to the newest material-ui v4.12

Open Linschlager opened this issue 2 years ago • 11 comments

Resolves #85

  • Fixed the tool to account for the new bugs in the docs
  • Ran the generator over the newer docs
  • Updated the urls to account for the legacy status
  • Made some small adjustments to get the Generator work on non-Windows

Linschlager avatar Jan 11 '22 18:01 Linschlager

Unfortunately @Shmew hasn't been active on here or Twitter since very early July 2021. It's possible something has happened to him :-/

JustinWick avatar Jan 19 '22 01:01 JustinWick

Is there any way to use the updated version in this branch in a project that uses paket and yarn?

JustinWick avatar Jan 19 '22 01:01 JustinWick

Maybe we should fork it to Feliz.MUI and work on bindings for V5?

marcpiechura avatar Jan 19 '22 07:01 marcpiechura

@Linschlager: Thoughts here?

I'd love @marcpiechura's suggestion, but at this point even just figuring out how to get this into a dependent project's package managers would unblock me :)

JustinWick avatar Jan 22 '22 04:01 JustinWick

@JustinWick . you could build the pull request branch locally and reference the dll directly, instead of the nuget package and then update the npm package separately.

marcpiechura avatar Jan 22 '22 06:01 marcpiechura

@Linschlager did you update the baseUrl? Because I think https://material-ui.com will now point to the api docs for v5. The legacy version 4 is still available at: https://v4.mui.com/.

halcwb avatar Jan 22 '22 10:01 halcwb

@Linschlager did you update the baseUrl? Because I think https://material-ui.com will now point to the api docs for v5. The legacy version 4 is still available at: https://v4.mui.com/.

@JustinWick Yes, I did. (HtmlCache.fs in a few places) - Although I may have missed some occurrences

Maybe we should fork it to Feliz.MUI and work on bindings for V5?

@marcpiechura I think this would be the only sustainable option for moving forward. The transition to v5 should be relatively straightforward as they kept most of the structure in their docs (apart from the styling, but that's irrelevant for the transformer). When I find some time in the next week, I'll try to get a POC going on my fork

Linschlager avatar Jan 22 '22 10:01 Linschlager

@Linschlager I think if you change this, all references will be correct:


let refresh =
  async {
    let baseUrl = "https://v4.mui.com" //"https://material-ui.com"

...

This is in the HtmlCache.fs.

halcwb avatar Jan 22 '22 11:01 halcwb

@Linschlager I think if you change this, all references will be correct:


let refresh =
  async {
    let baseUrl = "https://v4.mui.com" //"https://material-ui.com"

...

This is in the HtmlCache.fs.

Not fully. Common.fs also needs to fix the relative links so the crawler can fetch the files correctly

Linschlager avatar Jan 22 '22 12:01 Linschlager

Any plans to continue this? I'd be using v5 now if this were available!

I'd help out, though I know less than nothing about modern JS frameworks, I'm a backend/embedded guy, the last time I got paid to make a GUI for a webpage was 20 years ago.

JustinWick avatar Feb 21 '22 23:02 JustinWick

Any plans to continue this? I'd be using v5 now if this were available!

I'd help out, though I know less than nothing about modern JS frameworks, I'm a backend/embedded guy, the last time I got paid to make a GUI for a webpage was 20 years ago.

@JustinWick I have a WIP Mui5 version on my branch, though it will probably take a while for me to finalize fixing all the special cases Not sure it makes sense to publish this updated package if the end-goal is to upgrade to Mui5 anyway and a lot of things are going to change

Linschlager avatar Feb 22 '22 09:02 Linschlager

Closing this in favor of the MuiV5 "PR"

Linschlager avatar Jan 15 '24 13:01 Linschlager