Skip to content

feat: participants table redesign#322

Open
ShadowCatP wants to merge 77 commits into
mainfrom
feature/participants-table-redesign
Open

feat: participants table redesign#322
ShadowCatP wants to merge 77 commits into
mainfrom
feature/participants-table-redesign

Conversation

@ShadowCatP
Copy link
Copy Markdown
Member

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.

@solvro
Copy link
Copy Markdown

solvro Bot commented Mar 5, 2026

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

@ShadowCatP ShadowCatP linked an issue Mar 5, 2026 that may be closed by this pull request
@maciejkrol18 maciejkrol18 changed the title Feature/participants table redesign feat: participants table redesign Mar 7, 2026
@ShadowCatP ShadowCatP marked this pull request as ready for review March 14, 2026 13:46
Copy link
Copy Markdown
Member

@maciejkrol18 maciejkrol18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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ć.

Comment thread src/app/dashboard/events/[id]/participants/table/core/columns.tsx
Comment thread src/hooks/use-participants-table.ts Outdated
Comment thread src/app/dashboard/events/[id]/participants/table/core/participants-table.tsx Outdated
@ShadowCatP ShadowCatP force-pushed the feature/participants-table-redesign branch from be6b54b to 57ba630 Compare April 2, 2026 18:08
@ShadowCatP ShadowCatP requested a review from maciejkrol18 April 2, 2026 18:08
Copy link
Copy Markdown
Member

@maciejkrol18 maciejkrol18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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ę:
Image
  • Dalej nie ma tego fade outu na dole jak na designie
Image
  • Zarówno w poziomie i pionie nie ma customowych scrollbarów ze ScrollArea, tylko są te defaultowe przeglądarki
Image

Comment thread src/app/dashboard/events/[id]/participants/table/core/participants-table.tsx Outdated
Comment thread src/app/dashboard/events/[id]/participants/table/core/columns.tsx
@ShadowCatP ShadowCatP requested a review from maciejkrol18 April 17, 2026 21:33
Copy link
Copy Markdown
Member

@maciejkrol18 maciejkrol18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Comment thread src/app/dashboard/events/[id]/participants/table/core/columns.tsx Outdated
Copy link
Copy Markdown
Member

@maciejkrol18 maciejkrol18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scrollbary już działają

const row = rows[virtualRow.index];

return (
<TableRow key={row.id}>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Niedziałający scrolling jest spowodowany niepoprawną wirtualizacją rzędów tabeli

<TableRow key={virtualRow.key} data-index={virtualRow.index} ref={rowVirtualizer.measureElement}>

@ShadowCatP ShadowCatP force-pushed the feature/participants-table-redesign branch from 818dae0 to 938bc57 Compare May 2, 2026 15:00
@ShadowCatP ShadowCatP requested a review from maciejkrol18 May 2, 2026 15:00
@maciejkrol18
Copy link
Copy Markdown
Member

maciejkrol18 commented May 2, 2026

@ShadowCatP Jeszcze tylko dwie mniejsze rzeczy:

  • Dodać import uczestników. Jest już eksport, przycisk do importu powinien być obok. Definicja endpointu tutaj. Z info które dostałem, powinien normalnie działać. Oczekuje form data z plikiem xlsx. (będzie jako osobny task)
  • Zablokować przycisk z wysyłaniem maili jeśli nie ma wybranych uczestników, tak jak to działa z bulk deletem

Copy link
Copy Markdown
Member

@maciejkrol18 maciejkrol18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Przycisk z edycją jest niewidoczny w light modzie
image
  • 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
Image
  • 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"

@ShadowCatP ShadowCatP requested a review from maciejkrol18 May 11, 2026 18:57
@ShadowCatP ShadowCatP force-pushed the feature/participants-table-redesign branch from 6772082 to 393a108 Compare May 19, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redesign listy uczestników

3 participants