search

cca-io / rescript-mui

ReScript bindings for MUI
MIT License
250 stars 52 forks source link

Namespace: MaterialUi -> Mui #174

Closed cknitt closed 2 years ago

cknitt commented 2 years ago

IMHO we should change the namespace from MaterialUi to Mui in 3.0.0 for the following reasons:

  1. Nice and short 🙂
  2. Material-UI rebranded to MUI
  3. Now that we are using the namespace feature in bsconfig.json, it is not possible anymore for the user to "alias" the MaterialUi module to Mui himself using include MaterialUi.
  4. 3.0.0 already has other breaking changes anyway.
fhammerschmidt commented 2 years ago

Done, but leaving it open so that we do not forget to mention it in the docs.

fhammerschmidt commented 2 years ago

Closing this, I put it already into the Draft PR (#177).