Closed dacin21 closed 2 years ago
Des Weiteren sollten wir uns überlegen, von tslint
zu eslint
zu migrieren. eslint
scheint mehr linting-Möglichkeiten zu haben; insbesondere kann eslint --fix
viele kleinere linting-errors automatisch lösen (d.h. man verbringt z.B. weniger Zeit damit, code-style Fehler wie z.B. fehlende oder überschüssige Kommas zu beheben).
Versuch 1:
oh great, now we have do deal with stuff like this: https://github.com/mui-org/material-ui/issues/20812
-> I just nuked all the /es/ usage, seems to work just fine with the regular packages.
Kaputte Dinge, die mir aufgefallen sind:
color: rgba(0, 0, 0, 0.54);
font-size: 0.75rem;
font-weight: 500;
[TableRow] Remove the fixed height CSS property. The cell height is computed by the browser using the padding and line-height.
Bei tables die in jeder Zeile mindestens einen button (z.B. zum duplizieren oder archvieren) haben, ist dies kein Problem, da der button genügend hoch ist. Bei anderen, z.B. bei den nicht-verrechneten Projekten, ist dies aber ein Problem.
height: 48px;
bei der TableRow?.MuiButtonBase-root {
macht komisches CSS.4px 56px 4px 24px;
sein, ist überall 16px
).
4px
haben wir wieder das Problem, dass es keine minimale Zeilenhöhe mehr mehr gibt. Besser wäre wohl padding: 13px 56px 13px 24px;
.height: 48px
per MUI css setzen.font-size: 0.875rem;
anstelle von font-size: 0.8125rem;
)
Beim update von @types/classnames
zu
"@types/classnames@^2.2.6":
- version "2.2.10"
- resolved "https://registry.yarnpkg.com/@types/classnames/-/classnames-2.2.10.tgz#cc658ca319b6355399efc1f5b9e818f1a24bf999"
- integrity sha512-1UzDldn9GfYYEsWWnn/P4wkTlkZDH7lDb0wBMGbtIQc9zXEQq7FlKBdZUn6OBqD8sKZZ2RQO2mAjGpXiDGoRmQ==
+ version "2.3.1"
+ resolved "https://registry.yarnpkg.com/@types/classnames/-/classnames-2.3.1.tgz#3c2467aa0f1a93f1f021e3b9bcf938bd5dfdc0dd"
+ integrity sha512-zeOWb0JGBoVmlQoznvqXbE0tEC/HONsnoUNH19Hc96NFsTAwTXbTqb8FMYkru1F/iqp7a18Ws3nWJvtA1sHD1A==
+ dependencies:
+ classnames "*"
kommt folgender Fehler:
/var/frontend/src/form/fields/Select.tsx
(11,24): Could not find a declaration file for module 'classnames'. '/var/frontend/node_modules/classnames/index.js' implicitly has an 'any' type.
Try `npm install @types/classnames` if it exists or add a new declaration (.d.ts) file containing `declare module 'classnames';`
evtl wegen der neuen classnames "*"
dependency?
Reporter:
Daniel R.
Wir sind auf reactInzwischen gibt es react16.5.2
, was auch schon 3 Jahre alt ist.16.14.0
oder sogar17.0.2
. Edit: Gemäss yarn sind wir schon auf react 16.13, also ist es nicht ganz so schlimm.Bei material-UI sind wir auf version
3.2.2
, inzwischen gibt es aber version4.12.3
(für react 17). Man müsste schauen, ob alle packete schon für react 17 zur Verfügung stehen und allenfalls einen Ersatz finden.