feat: hassu 898 nopeutukset #318
Conversation
src/hooks/useHassuTable.tsx
Outdated
| import { Column, TableState } from "react-table"; | ||
|
|
||
| interface Test<D extends object> { | ||
| columns: readonly Column<D>[]; | ||
| data: readonly D[]; | ||
| initialState: Partial<TableState<D>> | undefined; | ||
| } | ||
|
|
||
| export const useHassuTableStabilizer = <D extends object>(tableOptions: Test<D>): HassuTableProps<D> => { | ||
| const [tableProps, setTableProps] = useState<HassuTableProps<D> | null>(null); | ||
|
|
||
| const newTableProps = useHassuTable<D>({ | ||
| tableOptions, | ||
| useRowSelect: true, | ||
| }); | ||
|
|
||
| useEffect(() => { | ||
| if (!tableProps) { | ||
| setTableProps(newTableProps); | ||
| } | ||
| }, [tableProps, setTableProps, newTableProps]); | ||
|
|
||
| if (!tableProps) return newTableProps; | ||
| return tableProps; | ||
| }; |
There was a problem hiding this comment.
Tonne on tainnut jäädä testailulta tyypille nimeks Test.
Toiseks tää stabilizer nyt raa'asti vaan estää tablePropsien päivityksen (ja HassuTablen rendauksen) vaikka tälle hookille annetut propsit muuttusikin. Mun käsityksen mukaa useHassuTable -hookkiin jos lisäis memoizausta muutamaan kohtaan niin sais vähennettyä renderaus kertoja kuitenkaan estämättä sen uudelleen renderausta. Testailin lisäämällä memoizointia tableHooks, defaultTableOptions, tableProps ja returnattavaan arvoon. Näin sain ne moneen kertaan renderoinnit estettyä.
Kannattaisiko tää ennemmin tehdä sillä tavalla?
There was a problem hiding this comment.
Muuten tää PR on mun mielestä OK 👍
There was a problem hiding this comment.
Keskusteltiin, että teen oman PR:n näistä useHassuTable-muutoksista ja riisutaan ne tästä PR:stä kokonaan pois
…ta luetaan nyt projektin tiedoista.
…margin-hommat meni huonosti.
…saa koko ajan uudet propsit, eli renderöityy turhaan uudestaan, jos checkboxia klikataan.
…uvaiheen vuorovaikutuksessa
ValheKouneli
force-pushed
the
feature/HASSU-898-nopeutukset
branch
from
769bc8f
to
ebfef01
Compare
Korjaus siihen, miten kunnan nimi haetaan suunnitteluvaiheen vuorovaikutuksessa. Nyt kunnan s.postin kirjoittaminen ei jumita.
Purkkakorjaus useHassuTableen, jotta se ei rendaa uudestaan ja uudestaan.
Refaktorointi vuorovaikutuslomakkeen Suunnitelmat ja Aineistot -osiolle.