Rewriting most remaining MUI usage #2905
Conversation
olemartinorg
added
the
kind/other
…n AltinnSummaryTable
… not printing (whoops, I just wanted to test that out, didn't mean to keep it)
# Conflicts: # src/layout/OrganisationLookup/OrganisationLookupComponent.tsx
…in between the table and cell
src/components/AltinnSpinner.tsx
Outdated
There was a problem hiding this comment.
Det finnes en Spinner i Designsystemet allerede. Skulle vi heller brukt den? 😄 Vet at den er brukt flere andre steder i hvert fall.
There was a problem hiding this comment.
Ja, jeg har ikke sett på CircularProgress og gjort noe med den enda (derav 'most' i tittelen på PRen). Helt enig på generell basis at det er best å bruke designsystemet sine komponenter. 🙌
| prop={props.summaryDataObject[name]} | ||
| /> | ||
| export const AltinnSummaryTable = ({ summaryDataObject }: IAltinnSummaryTableProps) => ( | ||
| <table className={classes.table}> |
There was a problem hiding this comment.
Hvorfor ikke Designsystemet Table-komponent? 😄
There was a problem hiding this comment.
Fordi denne tabellen, rent utseendemessig, er heeelt basic. Det er såvidt det er en tabell engang. Jeg prøvde først å bare lage denne visningen med noen div-elementer og css grid, men det ble mye knot. Alt ble veldig mye enklere når jeg bare gjorde den om til en helt vanlig table. Med designsystemet måtte jeg ha overstyrt massevis av styling for å få den til å se ut som en simpel tabell (altså gitt at den skulle se lik ut), så det ville jo vært mot hensikten.
Jeg synes godt vi kan bytte ut denne greia med noe annet i fremtiden, men jeg er usikker på om tabellen fra designsystemet er riktig det heller. Men ingen poeng i å endre utseendet på den nå i forbindelse med fjerning av MUI.
| ) : ( | ||
| <List classes={{ root: classes.list }}> | ||
| <ul className={classes.list}> |
There was a problem hiding this comment.
Her også, hvorfor ikke Designsystemet sin List? 🙈
There was a problem hiding this comment.
Hmm, her undersøkte jeg ikke like nøye, men basert på utseendet til denne tror egentlig man like godt kunne ha brukt p-tagger eller noe. 😅 Med andre ord, den har nærmest ikke noe utseende, så jeg tror designsystemet bare hadde vært i veien (gitt at den skal se lik ut som før).
| margin-bottom: 0.75rem; | ||
| border-radius: 0; | ||
| background-color: #e3f7ff; | ||
| box-shadow: 0 0 4px rgba(0, 0, 0, 0.25); |
There was a problem hiding this comment.
Tenker jeg dropper å lage en variabel på akkurat den. Den blir brukt nøyaktig 2 plasser, og bare i denne filen. Tidligere hadde den en variabel for hele box-shadow, men jeg ser ikke helt poenget.
|
Description
I had some time between issues, so I started pulling in one end to see how many of our MUI imports I could rewrite in a fairly short amount of time. I think this should be most of the low-hanging fruit.
Related Issue(s)
Verification/QA
kind/*label to this PR for proper release notes grouping