Open
Conversation
|
The preview deployment for Eventownik Frontend Preview is ready. 🟢 Open Preview | Open Build Logs | Open Application Logs Last updated at: 2026-03-01 15:20:09 CET |
maciejkrol18
reviewed
Mar 2, 2026
Member
There was a problem hiding this comment.
Refactor na nową wersję dnd kita i ogólnie kod zajebiście zrobiony ale mam kilka uwag:
- Są teraz konflikty ale nieskomplikowane, malina dorobił tłumaczenia tam i dlatego są, trzeba naprawić
- Na telefonie nie można zeswipeować w dół strony z palcem na kafelku, teraz jeszcze się da powiedzmy obok dać palec ale po mergu PR z responsywnoscia dashboarda kafelki będą na całą szerokość
- Myślę że sam kursor z łapką to może być trochę za mało dla niektórych żeby się zorientować że te kafelki są przesuwalne, dodałbym jakąś subtelną ikonkę na dole kafelka (Amelka mówi że niby by było lepiej w lewym górnym), coś takiego (grip horizontal z lucide z muted kolorem):
edit: na telefonie jest jednak możliwość reorderowania ale nie działa w devtoolsach na symulacji touch inputu
Member
Author
edit: yup, devtoolsy nie do końca to emulują |
Member
Zgadzam się, miałem na myśli to że ta ikonka była by żeby zasygnalizować że jest taka możliwość ogólnie a łapać wciąż można by było za całość |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.