feat: participants table redesign#322
Conversation
|
The preview deployment for Eventownik Frontend Preview is ready. 🟢 Open Preview | Open Build Logs | Open Application Logs Last updated at: 2026-05-21 17:52:08 CET |
There was a problem hiding this comment.
Infinite scroll: Nie wykorzystujemy wirtualizacji. Gdy powiedzmy że załadowanych jest wszystkich 500 wierszy to wydajność resizingu kolumn bardzo cierpi. Choć ogólnie z taką ilością wierszy scrollowanie działa płynnie, no to właśnie przy resizingu zaczyna się wszystko zacinać. Chciałbym zobaczyć refaktor z użyciem TanStack Virtual - powinno pomóc.
UI: Nie ma fade outu na ostatnim widocznym rzędzie jak na designie.
Konflikty: Trzeba rozwiązać.
be6b54b to
57ba630
Compare
maciejkrol18
left a comment
There was a problem hiding this comment.
Fajnie zrobiona wirtualizacja, teraz o wiele płynniej działa resize'owanie.
Pozostałe komentarze nie do konkretnych linijek/plików
- Cały czas (przynajmniej u mnie) pokazuje się taki błąd przy wejściu na tabelę:
- Dalej nie ma tego fade outu na dole jak na designie
- Zarówno w poziomie i pionie nie ma customowych scrollbarów ze
ScrollArea, tylko są te defaultowe przeglądarki
maciejkrol18
left a comment
There was a problem hiding this comment.
Oba poniższe problemy naprawiłem u siebie na szybko cofając stan participants-table.tsx na commit (df43642) - dalsze commity mają te problemy ale jako że był forcepush to nie jestem pewien czy to info coś daje
Zbugowane scrollowanie
W przypadku dużego datasetu jak wydarzenie z 500 uczestnikami, zaczyna się bugować scrollowanie w pionie - przy próbie scrolla cały czas przeskakuje. Problem nie występuje dla wydarzenia z 10 uczestnikami, gdzie wówczas też jest scroll. Nie dotyczy scrolla w poziomie.
2026-04-18.21-15-50.mp4
Scrollbary ze ScrollArea
Jak widać na filmiku wyżej, dalej ich nie ma i są te defaultowe z przeglądarki
| const row = rows[virtualRow.index]; | ||
|
|
||
| return ( | ||
| <TableRow key={row.id}> |
There was a problem hiding this comment.
Niedziałający scrolling jest spowodowany niepoprawną wirtualizacją rzędów tabeli
<TableRow key={virtualRow.key} data-index={virtualRow.index} ref={rowVirtualizer.measureElement}>818dae0 to
938bc57
Compare
|
@ShadowCatP Jeszcze tylko dwie mniejsze rzeczy:
|
maciejkrol18
left a comment
There was a problem hiding this comment.
- Przycisk z edycją jest niewidoczny w light modzie
- Przy wystarczająco dużej ilości kolumn, po doscrollowaniu do prawej przyciski edycji zostają w miejscu zamiast dojść w pełni do końca
- W przypadku gdy nie mamy wybranych żadnych uczestników, przy przycisku od wysyłki maili powinien podobny tooltip jak przy bulk edicie - tam jest "Najpierw zaznacz uczestników do edycji", tu można by było zrobić "Najpierw zaznacz odbiorców wiadomości"
6772082 to
393a108
Compare
I still need to adjust css style of things, but the functionality is finished. Also Imo participant details page should be moved to a separate issue, because there is still not enough info on how it will function and it is strictly related to the table itself.