Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update material-ui packages to mui/material namespace #70

Closed
5 tasks
vgr opened this issue Nov 30, 2022 · 9 comments
Closed
5 tasks

Update material-ui packages to mui/material namespace #70

vgr opened this issue Nov 30, 2022 · 9 comments
Labels
kind/user-story Used for issues that describes functionality for our users. status/draft Status: When you create an issue before you have enough info to properly describe the issue.

Comments

@vgr
Copy link
Contributor

vgr commented Nov 30, 2022

Description

The current material-ui dependencies in src/frontend is not compatible with the newer versions of React.
altinn-studio has alraedy updated their dependencies to use @mui/material namespace.
Altinn/altinn-studio#9318

Additional Information

No response

Tasks

No response

Acceptance Criterias

  • All usage of material-ui is replaced with mui/material
  • No regressions - local manual tests with app + running unit tests
@vgr vgr added kind/user-story Used for issues that describes functionality for our users. status/draft Status: When you create an issue before you have enough info to properly describe the issue. labels Nov 30, 2022
@annerisbakk
Copy link
Member

@acn-sbuad Er hele eller deler av denne fremdeles aktuell?

@acn-sbuad
Copy link
Contributor

acn-sbuad commented Feb 2, 2024

@acn-sbuad Er hele eller deler av denne fremdeles aktuell?

@annerisbakk Don't know. We would need to check with some frontenders.

@annerisbakk
Copy link
Member

@framitdavid Kan du vurdere denne?

@framitdavid
Copy link

@framitdavid Kan du vurdere denne?

Jeg kan ta en vurdering på denne Ila dagen. 🤓

@framitdavid
Copy link

framitdavid commented Feb 2, 2024

@annerisbakk hvis du vil ha min objektive mening her, så burde vi ikke bruke tid på å endre dette for alle komponenter som bruker material i dette prosjektet, men heller erstatte material i alle komponenter med @digdir/designsystemet.

Det kan være at noen komponenter ikke kan migreres over til designsystemet fordi designsystemet ikke har en tilsvarende komponent. Her kan man vurdere å fortsette å bruke material enn så lenge, eller om man skal lage sin egen komponent som er en sammenslåing av flere komponenter fra designsystemet for å komme et steg videre. (Evt. bidra inn til designsystemet om det lar seg gjøre).

Jeg ser at det ikke er veldig hyppige commits i frontend her, så usikker på hvem som evt. "eier" prosjektet og har ressurser til å vedlikeholde og gjøre en god opprydding her? Jeg mener hvertfall at det er lite hensiktsmessig å oppdatere material videre, når vi skal over på vårt eget designsystem. 😊

Hva med å lukke denne og lage en ny issue som går på å kartlegge omfang med å migrere fra material til digdir/designsystemet? Etter man har kartlagt kan man lage nye issues med konkrete tiltak som må gjøres for å ro det i land.

@annerisbakk
Copy link
Member

@SandGrainOne Høres for meg ut som en sak vi kan lukke, basert på tilbakemelding fra @framitdavid

@SandGrainOne
Copy link
Member

@annerisbakk Ja, vi kan låse på den versjonen vi har nå, men det kan ikke være slik til all evighet. Noen må en eller annen gang ta den jobben David foreslår.

@annerisbakk
Copy link
Member

@SandGrainOne Da kan vi lukke denne etter å ha opprettet en ny issue som analyserer jobben beskrevet over.

@annerisbakk
Copy link
Member

@SandGrainOne Lukker denne og oppfølging gjøres her #242

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/user-story Used for issues that describes functionality for our users. status/draft Status: When you create an issue before you have enough info to properly describe the issue.
Projects
None yet
Development

No branches or pull requests

5 participants